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.

Dropzone: Polished image manager elements

- Added file placeholder for non-image uploads.
- Added use of upload limits.
- Removed upload timeout variable.
- Added pass-through and usage of filetypes.
- Extracted some view text to language files and made use of existing
  text.

.vscode folder is added in .gitignore file (#4197)

Squash of 7 commits.

---------

Co-authored-by: esakkiraja100116 <redacted>

Dropzone: Developed ux further

- Added image manager button for uploads.
- Added image manager placeholder sidebar text for guidance.
- Improved dropzone layer styling.
- Removed old dropzone styles.
- Got success events and auto-hide working.
- Updated upload items to animate out.

Dropzone: started on design/ui of uploading

- Added new wider target handling.
- Updated upload item dom with design and seperate "landing" zone.
- Added new helper for simple dom element creation.

Dropzone: Swapped fetch for XHR for progress tracking

Started custom dropzone implementation

Merge pull request #4191 from tigsikram/fix-api-docs-timestamp

Fix timestamp in API docs example response

Fix timestamp in API docs example response

Merge branch 'feature/mail-verify-peer' into development

Mail: updated peer verify option name and added test

Added clojure code language option

For #4112

Merge branch 'wkhtmltopdf-env-example' into development

Updated complete env wkhtml text and added advisory

Added advisory to start to refer to docs for full details.
Updated added WKHTMLTOPDF option text.

Updated php deps

Edits from reviewing public events page

Documented public JS events used

Related to #4179

Merge pull request #4181 from BookStackApp/js_formatting

Added standard JS formatting via ESLint

ESLINT: Added GH action and details to dev docs

ESLINT: Addressed remaining detected issues

ESLINT: Started inital pass at addressing issues

Ran eslint fix on existing codebase

Had to do some manual fixing of the app.js file due to misplaced
comments

Added code formatting standard via eslint

Merge pull request #3617 from BookStackApp/codemirror6

Upgrade to codemirror 6

CM6: Further fixes/improvements after testing

- Updated event naming to be "cm6" when codemirror-specific.
- Removed cm block border in md editor to prevent double bordering.
- Updated copy handling to fallback to execCommand.

CM6: Fixed a range of issues during browser testing

- Fixed some keybindings not running as expected, due to some editor
  defaults overriding or further actions taking place since the action
  would not indicate it's been dealt with (by returning boolean).
- Fixed spacing/border-radius being used on codeblocks on non-intended
  areas like the MD editor.
- Fixed lack of BG on default light theme, visible on full screen md
  editor.
- Fixed error thrown when the user does not have access to change the
  current editor (Likely non-cm related existing issue)

CM6: Added tabbing, fixed dark mode border in WYSIWYG

CM6: Updated for popup editor, added new interface

New simple interface added for abstraction of CM editor in simple
use-cases, just to provide common actions like get/set content, focus
and set mode.

CM6: Got WYSIWYG code blocks working

Required monkey-patch to work around potential codemirror issue with
shadowdom+iframe usage.
Also updated JS packages to latest versions.

CM6: Aligned styling with existing, improved theme handling

Updated cm6 theme handling to allow extension via API

Uses our custom event system, uses methods that take callables so that
internal dependancies can be passed.

Added cm6 strategy for splitting and dyn. loading langs

Split out legacy modes to their own dynamically imported bundle to
reduce main code bundle size.

Addressed existing cm6 todos

- Updated clipboard handling
  - Removed old clipboard package for browser-native API.
- Updated codemirror editor events to use new props for new data types.

Cleaned up change handling in cm6 editor action handling

Finished update pass of all md editor actions to cm6

Updated another range of actions for cm6

Made a start on updating editor actions

Got md shortcuts working, marked actions for update

Got markdown editor barely functional

Updated content sync and preview scoll sync to work.
Many features commented out until they can be updated.

Merge branch 'v23.02-branch' into development

Updated translator attribution before release v23.02.3

Updated translations with latest Crowdin changes (#4131)

Updated translations with latest Crowdin changes (#4131)

Fixed tag numbering in last commit

Fixed click issue with tag suggestions in safari

Updated selectable elements to be divs instead of buttons since Safari
akwardly does not focus on buttons on click.
Also standardised keyboard handling to our standard nav class.
Also addressed empty tag values showing in results.
For #4139

Fixed error upon user delete with no migration id

Fixes #4162

Cleaned up old token services

Add WKHTMLTOPDF to .env.example.complete

Merge branch 'v23.02-branch' into development

Fixed delete role failing with no migrate role provided

For #4128

Updated php deps

Fixed issue with user delete ownership not migrating

Caused by input not being part of the submitted form.
Updated test to ensure the input is within a form.
For #4124

Updated php deps

Merged latest crowdin changes

Fixed delete role failing with no migrate role provided

For #4128

Updated php deps

Fixed issue with user delete ownership not migrating

Caused by input not being part of the submitted form.
Updated test to ensure the input is within a form.
For #4124

Allow a user to disable peer check when using TLS/STARTTLS

This is useful when developing and on Docker setups. Despite setting
encryption to null, if a server supports STARTTLS with a self-signed
certificate, the mailer try to upgrade the connection with STARTTLS.

Added own twig/smarty packages for cm6 lang support

Merge branch 'development' into codemirror6

Merge pull request #4103 from BookStackApp/image_api

Image API Endpoints

Added examples, updated docs for image gallery api endpoints

Added phpunit tests to cover image API endpoints

Started Image API build

Updated php deps

Merge pull request #4099 from BookStackApp/permissions_api

Content-Permissions API Endpoints

Added content-perms API examples and docs tweaks

Tweaked content permission endpoints, covered with tests

Started build of content-permissions API endpoints

Removed bookstack wording instances in color setting options

Updated translations with latest Crowdin changes (#4066)

Fixed language locale setting issue

Attempted to access an array that had been filtered and therefore could
have holes within, including as position 0 which would then be
accessed.
Also added cs language to internal map

Related to #4068

Fixed caching issue when running tests

Updated translations with latest Crowdin changes (#4025)

Updated php deps and translaters in prep for v23.02

Added updated_at index to pages table

This has a large impact on some areas where latest updated pages are
shown, such as the homepage for example.

Added caching to the loading of system roles

Admin system role was being loaded for each permission check performed.
This caches the fetching for the request lifetime.

Fixed old deprecated encoding convert on HTML doc load

Merge pull request #4062 from BookStackApp/settings_perf

Changed the way settings are loaded

Changed the way settings are loaded

This new method batch-loads them from the database, and removes the
cache-layer with the intention that a couple of batch fetches from the
DB is more efficient than hitting the cache each time.

Changed autosave handling for better editor performance

This changes how the editors interact with the parent page-editor
compontent, which handles auto-saving.
Instead of blasting the full editor content upon any change to that
parent compontent, the editors just alert of a change, without the
content. The parent compontent then requests the editor content from the
editor component when it needs that data for an autosave.

For #3981

Made page-save HTML formatting much more efficient

Replaced the existing xpath-heavy system with a more manual traversal
approach. Fixes following slow areas of old system:
- Old system would repeat ID-setting action for elements (Headers could
  be processed up to three times).
- Old system had a few very open xpath queries for headers.
- Old system would update links on every ID change, which triggers it's
  own xpath query for links, leading to exponential scaling issues.

New system only does one xpath query for links when changes are needed.
Added test to cover.

For #3932

Increased attachment link limit from 192 to 2k

Added test to cover.
Did attempt a 64k limit, but values over 2k significantly increase
chance of other issues since this URL may be used in redirect headers.
Would rather catch issues in-app.

For #4044

Updated language files to remove literal "1" values

This is to encourge the ":count" values to be used instead of 1s in the
translated variants so that non-pluralised languages are hardcoded with
"1"s in their content, even when not used in a singular context.

For #4040

Merge pull request #4051 from BookStackApp/roles_api

User Roles API Endpoint

Fixed unselectable checkbox role form options

Added role API responses & requests

Also applied other slight tweaks and comment updates based upon manual
endpoint testing.

Updated test to have reliable check ordering

Set order to role permissions API response

Aded roles API controller methods

Altered & updated permissions repo, and existing connected
RoleController to suit.
Also extracts in-app success notifications to auto activity system.
Tweaked tests where required.

Added tests for not-yet-built role API endpoints

Added esbuild bundle inspection metafile