[PR #4265] [MERGED] Enhanced Responsive Image Manager #6333

Closed
opened 2026-02-05 10:29:34 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/4265
Author: @ssddanbrown
Created: 5/24/2023
Status: Merged
Merged: 5/29/2023
Merged by: @ssddanbrown

Base: developmentHead: image_manager_responsive


📝 Commits (10+)

  • e467324 Updated image manager to use grid-based css
  • 6c91e09 Image manager: Redesigned header bar(s)
  • dc6133c Image manager: added ability to trigger load more via scroll
  • 946c9ae Image manager: supported a tabbed interface on mobile
  • 89d6d86 Image manager: extracted lang text, updated anims and search cancel
  • 9ff7c97 Image manager: Added extra detail below image edit form
  • e3c4a9d Added the ability to replace existing image files
  • f78c063 Fixed bad /api docs redirection on sub path
  • cd4b612 Image update API: added update image file ability
  • 948e95e Updated test to align with image manager HTML changes

📊 Changes

24 files changed (+538 additions, -173 deletions)

View changed files

📝 app/Api/ApiDocsController.php (+8 -0)
📝 app/Uploads/Controllers/GalleryImageController.php (+1 -1)
📝 app/Uploads/Controllers/ImageController.php (+27 -7)
📝 app/Uploads/Controllers/ImageGalleryApiController.php (+6 -1)
📝 app/Uploads/ImageRepo.php (+27 -15)
📝 app/Uploads/ImageService.php (+11 -3)
📝 lang/en/common.php (+1 -0)
📝 lang/en/components.php (+7 -0)
📝 lang/en/errors.php (+1 -0)
📝 resources/js/components/dropzone.js (+4 -0)
📝 resources/js/components/image-manager.js (+51 -7)
📝 resources/js/components/tabs.js (+44 -5)
📝 resources/sass/_animations.scss (+1 -1)
📝 resources/sass/_components.scss (+82 -29)
📝 resources/sass/_forms.scss (+52 -0)
📝 resources/sass/_layout.scss (+4 -0)
📝 resources/sass/_lists.scss (+4 -0)
📝 resources/sass/styles.scss (+0 -38)
📝 resources/views/pages/parts/image-manager-form.blade.php (+55 -12)
📝 resources/views/pages/parts/image-manager-list.blade.php (+9 -6)

...and 4 more files

📄 Description

Bit of a refresh for the image manager.

Related to #2535

Todo

  • Clean the search and filter buttons to be floating at the top of the image grid window, within the same row.
  • Add more detail when image selected (Creator, uploaded to) (Consider permissions).
  • Add ability to replace an image file.
    • Add this to the API also.
  • Update layout to work for mobile.
  • Auto load pages on scroll at bottom of view, if "load more" is active.
  • Test usability via keyboard/screen reader.
  • Test dark mode styles.
  • Remove image-manager-auto-show dev script in pages/edit.blade.php.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/BookStackApp/BookStack/pull/4265 **Author:** [@ssddanbrown](https://github.com/ssddanbrown) **Created:** 5/24/2023 **Status:** ✅ Merged **Merged:** 5/29/2023 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `development` ← **Head:** `image_manager_responsive` --- ### 📝 Commits (10+) - [`e467324`](https://github.com/BookStackApp/BookStack/commit/e4673246589974284295e7b8e3479f1d1a6fea1c) Updated image manager to use grid-based css - [`6c91e09`](https://github.com/BookStackApp/BookStack/commit/6c91e09c73d8ed1d34223b886866a1ac78f94bae) Image manager: Redesigned header bar(s) - [`dc6133c`](https://github.com/BookStackApp/BookStack/commit/dc6133c4c4e67c1403dd98f42c86331b8a2170d8) Image manager: added ability to trigger load more via scroll - [`946c9ae`](https://github.com/BookStackApp/BookStack/commit/946c9ae80442bb3b413d0af44545f432e06f781c) Image manager: supported a tabbed interface on mobile - [`89d6d86`](https://github.com/BookStackApp/BookStack/commit/89d6d862fa6ab81df54f7c699f7ad2ac28e36370) Image manager: extracted lang text, updated anims and search cancel - [`9ff7c97`](https://github.com/BookStackApp/BookStack/commit/9ff7c97911fb274f94d5d9cb9be2fbd0f8299785) Image manager: Added extra detail below image edit form - [`e3c4a9d`](https://github.com/BookStackApp/BookStack/commit/e3c4a9d167af8a92b288bcc100347d52bff102bc) Added the ability to replace existing image files - [`f78c063`](https://github.com/BookStackApp/BookStack/commit/f78c0635eee694e3deeb362c4d49ce6a3bcbe906) Fixed bad /api docs redirection on sub path - [`cd4b612`](https://github.com/BookStackApp/BookStack/commit/cd4b612019991aee8d0af2332d59ee0e57fa9eea) Image update API: added update image file ability - [`948e95e`](https://github.com/BookStackApp/BookStack/commit/948e95e1ad89ed9e20e37cef2233f2387c634d10) Updated test to align with image manager HTML changes ### 📊 Changes **24 files changed** (+538 additions, -173 deletions) <details> <summary>View changed files</summary> 📝 `app/Api/ApiDocsController.php` (+8 -0) 📝 `app/Uploads/Controllers/GalleryImageController.php` (+1 -1) 📝 `app/Uploads/Controllers/ImageController.php` (+27 -7) 📝 `app/Uploads/Controllers/ImageGalleryApiController.php` (+6 -1) 📝 `app/Uploads/ImageRepo.php` (+27 -15) 📝 `app/Uploads/ImageService.php` (+11 -3) 📝 `lang/en/common.php` (+1 -0) 📝 `lang/en/components.php` (+7 -0) 📝 `lang/en/errors.php` (+1 -0) 📝 `resources/js/components/dropzone.js` (+4 -0) 📝 `resources/js/components/image-manager.js` (+51 -7) 📝 `resources/js/components/tabs.js` (+44 -5) 📝 `resources/sass/_animations.scss` (+1 -1) 📝 `resources/sass/_components.scss` (+82 -29) 📝 `resources/sass/_forms.scss` (+52 -0) 📝 `resources/sass/_layout.scss` (+4 -0) 📝 `resources/sass/_lists.scss` (+4 -0) 📝 `resources/sass/styles.scss` (+0 -38) 📝 `resources/views/pages/parts/image-manager-form.blade.php` (+55 -12) 📝 `resources/views/pages/parts/image-manager-list.blade.php` (+9 -6) _...and 4 more files_ </details> ### 📄 Description Bit of a refresh for the image manager. Related to #2535 ### Todo - [x] Clean the search and filter buttons to be floating at the top of the image grid window, within the same row. - [x] Add more detail when image selected (Creator, uploaded to) (Consider permissions). - [x] Add ability to replace an image file. - [x] Add this to the API also. - [x] Update layout to work for mobile. - [x] Auto load pages on scroll at bottom of view, if "load more" is active. - [x] Test usability via keyboard/screen reader. - [x] Test dark mode styles. - [x] Remove image-manager-auto-show dev script in `pages/edit.blade.php`. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 10:29:34 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#6333