Merge pull request #4318 from devdot/improve-json-debug-exception

Change JsonDebugException to Responsable interface

Merge branch 'development' of github.com:BookStackApp/BookStack into development

Users API: Fixed incorrect created_at date on index endpoint

For #4325

Merge pull request #4322 from BookStackApp/comments_in_editor

Added read-only comments listing into page editor

Comments: Fixed failing tests due to unset template variable

Comments: Added read-only listing into page editor

Change JsonDebugException to Responsable interface

In all other exceptions, when a Response is supposed to be returned,
the Responsable interface is used instead of render.

Merge branch 'fix-api-404' into development

Exceptions: Added some types, simplified some classes

During review of #4291

CSS: Reduced styles used in export formats

Extracted many main page content styles to own scss partial.
Styles could do with a more general clean-up.

Closes #4303

Updated fonts to be defined via CSS variables

Exports system remains separate due to lacking css variable support.

Refactor notify exception to clean up api exception handling

Refactor exception handling by using interface

Merge pull request #4301 from BookStackApp/css_color_variables

CSS: Updated status colors to be CSS variables, Added dark variants

CSS: Updated status colors to be CSS variables, Added dark variants

Needed some level of harcoding though due to callouts using colors,
which can't be css colors as DOMPDF won't understand these.
Use css variables elsewhere and added new dark variants to fit a bit
better.

Page Drafts: Added new "Delete Draft" action to draft menu

Provides a way for users to actually delte their user drafts where
required.
For #3927

Added test to cover new endpoint.

Makes update to MD editor #setText so that new selection is within new
range, otherwise it errors and fails operation.

Added activity text for each activity type

Ensures some sensible text is always in webhook text data.
Also aligned some notification reporting to use centralised activity
system instead of custom success events.

For #4216

Search: Updated popular items query, load parent book for chapters/pages

Primarily intended to show parent book for chapters when moving/copying
pages, since the default parent selector interfaces, which used the
entity-selector search endpoint, would run this popular query when no
term was present as a default backup.

For #4264

Permissions: Updated guest user handling so additional roles apply

Previously additional roles would only partially apply (system or "all"
permissions). This aligns the query-handling of permissions so that
additional roles will be used for permission queries.

Adds migration to detach existing roles as a safety precaution since
this is likely to widen permissions in scenarios that the public user
has other roles assigned already.

For #1229

API: Fixed misaligned image datetime format

For #4294

Fixed error on pages without comments

Merge pull request #4286 from BookStackApp/comment_threads

Comment threads

Comments: Updated reply-to and general styling

Updated reply inidicator to fit with new nesting system, only showing on
view when nest within nesting structure.

Updated the general design to be a bit cleaner and better adapt on
mobile.

Tested on FF+Chrome, inc. dark mode.

Comments: Added visual nesting limit, added nesting test

Comments: Updated to show form in expected location

Includes a change of create response to use a branch as a template.

Fix "HTTP 500 on not found" bug  #4290

Add test showing the "HTTP 500 on not found" bug

Comments: updated component and split out code

Split out comment component code so single-comment actions (delete, edit) are handled within their own compontent.
Modernised existing component code.

Comments: Updated to show as nested threads

Initial functional implementation, a lot of tweaking and adapting to be
done.

Page display pointer: Considerably improved accessibility

- Updated pointer to move within content DOM so that you can back-focus
  into the pointer if desired.
- Added new "Section select mode" which toggles focusabiltiy for main
  content sections, with ability to show pointer via enter press on
  these.
- Updated pointer with proper input/button labelling.

Tested via orca screen reader on Firefox/Fedora/Gnome.
For #3975

Chapters API: Allowed move via book_id property

Aligns it with pages and with the book_id property already being part of
the API.
For #4272.

Altered ldap_connect usage, cleaned up LDAP classes

Primarily updated ldap_connect to avoid usage of deprecated syntax.
Updated tests and service to handle as expected.
Cleaned up syntax and types in classes while there.

Closes #4274

Merge pull request #4265 from BookStackApp/image_manager_responsive

Enhanced Responsive Image Manager

Removed forced initial image manager display

Image manager: Improved screen reader usage

Added extra labels, or removed duplicate info, to improve screen reader
ux after testing via gnome/fedora/firefox screen reader usage testing.

Image manager: cleaned up style changes, dark mode support

- Updated tab handling to be smarter on initial tab selection, to first
  target non-hidden tab panels where they may be handled server-side.
- Extracted contained search box handling styles to _forms.scss, after
  merging with image-manager-specific styles since this is only usage of
  contained variant.
- Aligned focus handling on image manager UI elements.

Updated test to align with image manager HTML changes

Image update API: added update image file ability

Fixed bad /api docs redirection on sub path

Direct route redirect does not seem to go via standard URL generator so
misses off generation via base URL.

Added the ability to replace existing image files

- Updated UI with image form dropdown containing delete and replace
  image actions.
- Adds new endpoint and service/repo handling for replacing existing
  image.
- Includes tests to cover.

Image manager: Added extra detail below image edit form

Image manager: extracted lang text, updated anims and search cancel

- Updated search cancel to only show when a search is active.
- Updated gallery image load animation to be much faster.

Image manager: supported a tabbed interface on mobile

Makes interface relatively usable now on mobile sizes.
Required updating of tab handling to support tabs being active at only
mobile screen sizes, include change on resize, upon support for
potentially nested tab usage.
Tab component will now search within sensible depths for finding its own
tabs and panels to control.

Image manager: added ability to trigger load more via scroll

Image manager: Redesigned header bar(s)

Updated image manager to use grid-based css

Merge pull request #4262 from BookStackApp/command_cleanup

Command cleanup & alignment

Updated command classes to include "Command" in name

Aligned command class code

- Aligned usage of injecting through handler.
- Aligned handler return type.
- Aligned argument and arg desc format.
- Aligned lack of constructor.

Covered untested commands with testing

Merge pull request #4247 from BookStackApp/controller_cleanup

Revised `app/` folder layout/structure

Fixed failing references after controller/file reshuffle

Updated translation attribution before v23.05.2

Updated php deps

Updated translations with latest Crowdin changes (#4239)

Merge pull request #4252 from BookStackApp/cli_update_2

Updated System CLI

Merge pull request #4254 from BookStackApp/code_active_line

Updated code view block line highlighting to only show on focus

Allowed attachment drag via main text link

Enables easier sorting and dragging of box into content.
Related to #591

Allowed button-based multi-file uploads

Likely something that worked via dropzone before.
This adds support for our custom dropzone file handling.
Related to #4241

Updated code view block line highlighting to only show on focus

The default 1st line highlighting confused users when existing on
read-only blocks as it was not clear this represented the active line.
This changes the highlight to only show when the block is focused upon.

Updated System CLI

- Fixed a range of additional issues involving symlinks.
- Fixed incorrect app locating relative to system cli.

Cleaned up namespacing in routes

Also moved home controller and moved controllers up a level in http.

Played around with a new app structure

Updated translator attribution before release v23.05.1

Merge pull request #4229 from BookStackApp/cli-update

Updated system CLI

Updated system CLI

- Fixed wrong env details being used on restore.
- Updated update-url on restore actually work.
- Added better support for symlinked locations.
- Added warning against updating in docker-like (non git controlled)
  environments.

Updated translations with latest Crowdin changes (#4211)

Fixed code block line-number bar showing in exports

Also fixed in print view.
Likely crept in during CM6 changes.

For #4215

Updated php deps, formatted command changes

Prevented shorcuts activating when in codemirror areas

For #4227

WYSWIYG code blocks: copied head styles into shadow root

Currently only link-based styles are made available in the shadow root
code editor environment, this adds normal styles to apply any user-added
via custom head content.

Fixes #4228

Fixed added padding around hr tags in details blocks

Due to manual handling & wrapping of non-block content in details block
not taking hr elements into account.
For #3963

Added WYSWIYG editor code editor cancel focus return

Focus now returns to the editor properly when you quit out the code
editor without saving.
This also sets the return location to be correct on normal saving (Would
sometimes jump to the end of the document).

For #4109.

Added force option for update-url command

Includes test to cover.
Closes #4223

Added mostodon badge in readme, updated CLI

Set page include limit to be 3 as expected instead of 4

Updated translator attribution pre v23.05 release

Updated translations with latest Crowdin changes (#4163)

Updated CLI & PHP deps, added gitignore for local composer

Fixed broken favourites in code editor

Merge pull request #4206 from BookStackApp/system_cli

Added System CLI

Added system cli, and created backups directory

Added sessionindex to SAML2 single logout request to idp

related to  #3936

Added oidc_id_token_pre_validate logical theme event

For #4200

Merge branch 'esakkiraja100116/development' into development

Review of #4202, Rolled out to other searches, added testing

suggesstion issue fix (#4175)

Adding a video { width: 100%; }  (#4204)

* Adding a video { width: 100%; }

This is to prevent that videos included in pages don't exceed the page border

* Reverting precedent commit

* Adding a video { max-width: 100% } instead

Review of #4192, Fixed formatting and added test

changed PageContent.php to accept nested includes (#4192)

* changed app/Entities/Tools/PageContent.php to accept nested include levels. Tested it and it works.

* changed recommendations

This loop is now only around parsePageIncludes and bugfixes the space indentation.

* Update PageContent.php

fix spaces

Fixed sort urls with no params not building full path

The provided partial path would be return which may not resolve to the
full URL when used on systems like those hosting BookStack on a
sub-path.
Fixes #4201

Merge pull request #4193 from BookStackApp/custom_dropzone

Custom dropzone implementation

Fixed linting and failing test issues from dropzone work

Dropzone: Adjusted styles for dark mode

Removed dropzone from package and attribution list

dropzone: Addressed existing todos, cleaned attachment ux

Updated dom layout of attahcments to prevent nested dropzones (No issue
but potential to be one) and updated edit form dropzone handling so the
dropzone item card was not as distracting.

Updated attachments to work with new dropzone

- Fixes existing broken attachment edit tabs.
- Redesigns area to move away from old tabbed interface.
- Integrates new dropzone system, for both addition and edit.

Image manager: fix upload control for drawing, updated styles

- Tightened image manager styles to address things that looked akward.
- Prevented visiblity/use of upload controls for drawings.
- Updated dropzone to use error handling from validation messages.

Merge branch 'development' of github.com:BookStackApp/BookStack into development

Fixed broken shortcut hint overlay

Also updated event handler usage to use abort controller while there.