[PR #755] [MERGED] Fixes issues with gif images not animating #5708

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

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/755
Author: @Abijeet
Created: 3/18/2018
Status: Merged
Merged: 3/24/2018
Merged by: @ssddanbrown

Base: masterHead: bug/gif-image-740


📝 Commits (2)

  • bf8dddd Not resizing gif images.
  • 2cfcbe0 Fixes an issue with handling of large image files.

📊 Changes

2 files changed (+14 additions, -1 deletions)

View changed files

📝 app/Services/ImageService.php (+13 -0)
📝 resources/assets/js/vues/components/dropzone.js (+1 -1)

📄 Description

Close #223

Also fixes issues with improper error message when file is too large.


🔄 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/755 **Author:** [@Abijeet](https://github.com/Abijeet) **Created:** 3/18/2018 **Status:** ✅ Merged **Merged:** 3/24/2018 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `master` ← **Head:** `bug/gif-image-740` --- ### 📝 Commits (2) - [`bf8dddd`](https://github.com/BookStackApp/BookStack/commit/bf8dddd99c6adec29c58e30697980f858c494667) Not resizing gif images. - [`2cfcbe0`](https://github.com/BookStackApp/BookStack/commit/2cfcbe0a3cd6700b562a81fd46b71549667d048a) Fixes an issue with handling of large image files. ### 📊 Changes **2 files changed** (+14 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `app/Services/ImageService.php` (+13 -0) 📝 `resources/assets/js/vues/components/dropzone.js` (+1 -1) </details> ### 📄 Description Close #223 Also fixes issues with improper error message when file is too large. --- <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:14:51 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#5708