Editors: Properly aligned edit area border radius

Editors: Adjusted new design for mobile and dark mode

Tested new design across FF, Chrome, and Gnome web (webkit)

Editors: Tightened up new design, adjusted for MD editor

Editor: Started toying with more singificant design update

Editor: Started attempts to improve design elements

Homepage: Made much nicer at ipad-like widths

Updated default homepage layout to be much nicer at ipad-like widths by
switching to css-column approach at those breakpoints.
Also neated top actions by switching to simpler flexbox layout.

Fixes #4596

Merge branch 'LawssssCat/development' into development

Layout: Restructured tri-layout for sidebar control

Restructured tri-layout grid system, so the sidebars are contained in
their own child grid system, mimicking the parent grid, so we can treat
them as part of the same parent scroll container at smaller screen
sizes for consistent scroll/sticky behavior.

Tested on Firefox, Chromium, Gnome Web and Safari (MacOS).

For #4394
Changes made during review of #4562

CI: Added path filtering to actions

In the hope we can make the CI runs a bit more efficient and energy
conscious, by only running when relevant files have changed.

Tags: Fixed enter press clearing field

For #4570

Print Styles: Removed use of seperate style sheet

Seemed a bit redundant and complicated, since we're only adding a few
extra styles. Just merged into main styles instead.

Print Styles: Fixed header/footer content showing

Extra bits were showing due to recent changes.
Done a quick pass through major display views.

Fixes #4594

Testing: Added favicon cleanup

Was leaving a changed favicon leaving other test to fail.

Added NETWAYS to readme sponsor list

Related to:
https://github.com/BookStackApp/website/commit/edceda5342a95bba1e95abae55a5d62dc1ba6e45

Merge branch 'basic-pwa-support' into development

PWA Manifest: Tweaks during review of PR #4430

- Updated to go through HomeController with the builder as a helper
  class.
- Extracted some reapeated items into variables in manifest.
- Updated background color to match those used by BookStack.
- Removed reference of icon.ico since its not intended to be used.
- Added tests to cover functionality.

Review of #4430

Merge pull request #4578 from BookStackApp/upload_handling

Improvements to file/image upload handling UX

JS: Fixed ESLint issues

Images: Rolled out image memory handling to image actions

- Moved thumnbail loading out of repo into ImageResizer.
- Updated gallery and editor image handling to show errors where
  possible to indicate memory issues for resizing/thumbs.
- Updated gallery to load image data in a per-image basis via edit form
  for more resiliant thumb/data fetching. Data was previously provided
  via gallery listing, which could be affected by failing generation
  of other images.
- Updated image manager double click handling to be more pleasant and
  not flash away the edit form.
- Updated editor handlers to use main URL when thumbs fail to load.

Images: Extracted out image resizing to its own class

Images: Added thin wrapper around image filesystem instances

Extracts duplicated required handling (Like path adjustment) out to
simpler storage disk instance which can be passed around.

Images: Started refactor of image service

To break it up.
Also added better memory handling to other parts of the app.

Image manager: Tweaked grid sizing to prevent massive items

Images: Added test to cover thubmnail regen endpoint

Images: Reverted some thumbnails to be on-demand generated

Added since we can't always be sure of future image usage, and in many
cases we don't generate ahead-of-time.
Also:
- Simplified image handling on certain models.
- Updated various string handling operations to use newer functions.

Thumbnails: Added OOM handling and regen endpoint

- Added some level of app out-of-memory handling so we can show a proper
  error message upon OOM events.
- Added endpoint and image-manager button/action for regenerating
  thumbnails for an image so they can be re-created upon failure.

Images: Updated to create thumbnails at specific events

Uploads: Added user-facing message for Laravel post limit handling

Uploads over the post max size Would previously error without a
clean user facing message. This catches that error to provide a
user friendly message, compatible with our common error handling.

Tested on image manager handling.
Added test to cover.

Remove Dumps

Editors: Updated entity link select to pre-fill with selection

Updated all uses across both editors, so the entity link selector popup
now initates a search with the selection text if existing.

For #4571

Exports: Fixed issues with book text export format

- Fixed missing page content for direct page children
- Fixed lack of book description.
- Fixed inconsistent spacing between items.
- Fixed lack of spacing between HTML items when HTML on same line.

For #4557

Header: Simplified, split and re-orgranised view file(s)

- Moved "common" template partials, that are only used in layouts, to
  layouts/parts folder.
- Simplified HTML structure of header template.
- Extracted logo and links from header template to simplify.
- Added header-links-start template for easier extension/customization
  without needing to override full list of links.
  - Added test to cover usage of this.

For #4564

Homepage: Added tags button to non-default home views

For #4558

Search: Added further backslash handling

Added due to now not being able to perform an exact search where
contains a trailing backslash.
Now all backslashes in exact terms are consided escape chars
and require escaping themselves.
Potential breaking change due to search syntax handling change.

Related to #4535.

Readme: Updated badges, sponsors and top links

Fix Reloading changes on dark mode switch

Merge branch 'basic-pwa-support' of https://github.com/GamerClassN7/BookStack into basic-pwa-support

New line fix

Update HomeController.php

Formating Fixes

Additional Tweaks and FIxes

Implementation of required changes

Merge branch 'basic-pwa-support' of https://github.com/GamerClassN7/BookStack into basic-pwa-support

Fix Dark theme

Merge branch 'BookStackApp:development' into basic-pwa-support

Search: Added support for escaped exact terms

Also prevented use of empty exact matches.
Prevents issues when attempting to use exact search terms in inputs for
just search terms, and use of single " chars within search terms since
these would get auto-promoted to exacts.

For #4535

Avatar Commend: Simplified and updated during review

During review of #4560.

- Simplified command to share as much log as possible across different
  run options.
- Extracted out user handling to share with MFA command.
- Added specific handling for disabled avatar fetching.
- Added mention of avatar endpoint, to make it clear where these avatars
  are coming from (Protect against user expectation of LDAP avatar sync).
- Simplified a range of the testing.
- Tweaked wording and code formatting.

fix Sidebar scrolling at mid-range sceen

fix: Actually check if we have correct data

feat: Artisan command for updating avatars for existing users

Dev: Reviewed and expanded on PHP testing docs

Merge branch 'tusharnain4578/development' into development

Notifications: Updated comment notif. prefs. test

Combined testcases, updated to use actual text strings, and set comments
setting via correct method.

Made during review of #4552

Merge pull request #4555 from BookStackApp/language_cleanup

Language cleanup

Locales: Removed a lot of existing locale handling

There was a lot of locale handling to get correct/expected date
formatting within the app.
Carbon now has built-in locale content rather than us needing to target
specific system locales.

This also removes setting locale via Carbon directly.
Carbon registers its own Laravel service provider which seems to
accurately pull the correct locale from the app.

For #4555

Update UserPreferencesTest.php

Added Testcases for preferences menu of Comment Notifications visibility when comments are enabled/disabled.

Locales: Fixed errors occuring for PHP < 8.2

Locales: More use of locale objects, Addressed failing tests

Locales: Performed cleanup and alignment of locale handling

- Reduced app settings down to what's required.
- Used new view-shared $locale object instead of using globals via
  config.
- Aligned language used to default on "locale" instead of mixing
  locale/language.

For #4501

Extracted icon helper, aligned container resolution

Also updated breadcrumb view composer to current standards.
Closes #4553

Merge pull request #4554 from BookStackApp/guest_user_cleanup

Guest control: Cleaned methods involved in fetching/handling

Removed redundant null check

Guest control: Cleaned methods involved in fetching/handling

- Moves guest user caching from User class to app container for
  simplicity.
- Updates test to use simpler $this->users->guest() method for
  consistency.
- Streamlined helpers to avoid function overlap for simplicity.
- Extracted user profile dropdown while doing changes.

Merge branch 'v23-08' into development

Fixed : Comment notification settings are visible even if comments are disabled

Added a UX condition to display comment notification settings, only if the user has enabled the comment notifications.

Comments: Fixed wrong identification of parent comment

Would cause comment reply notifications to not be sent to expected user.
Updated test to cover problem case.

For #4548

Cleanup Command: Allowed running non-interactively

For #4541

Dark Mode: Fixed not toggle action when dark by default

Added test to cover.
For #4543.

Langs: Added Uzbek, Updated translator attribution

For #4527

Merge branch 'Bajszi97/development' into development

Reviewed #4533, formatting and tweaks

- Updating formatting.
- Tweaked truncation to roughly match elipsis char to width used.
- Updated testing to use existing helpers, and ran check as admin user
  to avoid name conflicts.

Test comment creator name truncation

Truncate with three dots

Theme: Added handling for functions.php file load error

This adds specific handling for functions.php error loading to re-throw
errors wrapped in a more descriptive message, to make it clear the error
is due to an issue in their functions.php file.

Decided to throw and stop, rather than ignore & continue, to be on the
safe side in the event auth-level (or other security level) customizations
have been made via functions.php.

Adds test to cover.
Closes #4504

Trimmed name last resort

Notifications: Reorgranised classes into domain specific folders

Closes #4500

Views: Rolled out ID to similar recent activity lists

To allow for easier customization.
Related to #4530

Merge branch 'v23-08' into development

Comments: Fixed JS error when lacking commenting permissions

The page comments component would throw an error due to references to
form elements/content, when form elements may not exist due to
permisisons.

For #4531

Slack auth: Switched from community to laravel library

Tested locally before & after change, and looked at code to compare.
Nothing seen or experienced that should affect things, from testing all
is working as expected with no difference from before.
- Update composer requirement of socialite to that which included slack.
- Updated PHP depds while there.
- Updated format of socialite events to align with current documentation
  and to use class references instead of strings.

OIDC: Moved name claim option handling from config to service

Closes #4494

Fixed php8 compat issue, updated readme & templates

- Changed use of array spread since it was not supported in PHP8.0.
- Updated issue templates based to reduce less valueable fields, update
  some details, and try to help bug reports be more focused on bugs.
- Updated readme with peertube link and attribution advistory for
  translations PRs.

Simplified and aligned handling of mixed entity endpoints

Fixes #4444

Mail: changed default "MAIL_FROM" address

Used an "example.com" address so we're using a propoer reserved domain,
and to avoid these trying to be delivered to the main bookstackapp
domain.

Closes #4518

Merge pull request #4525 from BookStackApp/http_alignment

HTTP calling logic alignment

Removed use of HttpFetcher

- Fixed some existing issues in new aligned process.
- Manually tested each external call scenario.

Started aligning app-wide outbound http calling behaviour

Updated translations with latest Crowdin changes (#4512)

Last translation merge for possible continued v23.08 branch

Updated sponsors and dev version

Updated translations with latest Crowdin changes (#4506)

WYSIWYG: Fixed filtering issue causing broken page edits

Could error upon div elements without classes, including drawings.

Related to #4510 and #4509

Updated translator attribution before release v23.08.1

Updated translations with latest Crowdin changes (#4481)

Preferences: Fixed section screen flexibility

Improved wrapping and flex control to prevent button text force wrapping
to newlines.

For #4502

Watching: Prevent issues when watchable or user is deleted

- Adds filtering to the watched items list in notification preferences
  so that deleted (recycle bin) items are removed via query.
- Adds relations and logic to properly remove watches upon user and
  entity delete events, to old watches in database do not linger.
- Adds testing to cover the above.

Did not add migration for existing data, since patch will be close to
introduction, and lingering DB entries don't open a security concern,
just some potential confusion in specific potential scenarios.
Probably not work extra migration risk, although could add in future if
concerns/issues are found.

Related to #4499

Merge pull request #4497 from BookStackApp/notification_language

Notifications: User language for notification text

Merge pull request #4484 from omahs/patch-1

Fix typos

Testing: Added entity decode flag and phpunit env option

- Passed decode flags to provide consistent behaviour across PHP
  versions during testing.
- Added env option to prevent local option taking action in PHPunit
  tests.

Notifications: Aligned how user language is used

- This ensures content notifications are not translated to receiver
  language.
- This adds actual plaintext support for content notifications (Was
  previously just HTML as text view).
- Shares same base class across all mail notifications.
- Also cleaned up existing notification classes.

Future cleanup requested via #4501

Notifications: Add test to check notification language