Validation failures are hidden if in a collapsable section of the form #1388

New Issue

OVERLORD · 2026-02-05T00:45:42+03:00

OVERLORD commented

2026-02-05 00:45:42 +03:00

Originally created by @cnfw on GitHub (Oct 1, 2019).

Originally assigned to: @ssddanbrown on GitHub.

Description
When creating or editing a book or a shelf, if there is a validation failure in the collapsable parts of the form (cover image, tags), the error is not immediately obvious as the section is automatically collapsed. This leads to a bit of confusion as to why you've been brought back to the form rather than the content.

Steps To Reproduce
Steps to reproduce the behaviour:

Go to a book
Click on the edit button
Expand "Cover Image" section
Set cover image to something that will fail validation (such as file size too large, or unsupported format)
Click save
Observe that you remain on the form, but with the error message hidden
Expand "Cover Image" section to reveal the error.

Expected behaviour
My expectation would be one of the following:

Show the error at the top of the page
Highlight the Cover Image box in a suitable colour to indicate that there is a problem with that section
Have the section(s) containing the error(s) already expanded when the page loads.

Screenshots
N/A

Configuration:

Exact BookStack Version: v0.27.4
PHP Version: 7.3
Hosting Method (Nginx/Apache/Docker): Docker

Additional context
Add any other context about the problem here.

Originally created by @cnfw on GitHub (Oct 1, 2019). Originally assigned to: @ssddanbrown on GitHub. **Description** When creating or editing a book or a shelf, if there is a validation failure in the collapsable parts of the form (cover image, tags), the error is not immediately obvious as the section is automatically collapsed. This leads to a bit of confusion as to why you've been brought back to the form rather than the content. **Steps To Reproduce** Steps to reproduce the behaviour: 1. Go to a book 2. Click on the edit button 3. Expand "Cover Image" section 4. Set cover image to something that will fail validation (such as file size too large, or unsupported format) 5. Click save 6. Observe that you remain on the form, but with the error message hidden 7. Expand "Cover Image" section to reveal the error. **Expected behaviour** My expectation would be one of the following: - Show the error at the top of the page - Highlight the Cover Image box in a suitable colour to indicate that there is a problem with that section - Have the section(s) containing the error(s) already expanded when the page loads. **Screenshots** N/A **Configuration:** - Exact BookStack Version: v0.27.4 - PHP Version: 7.3 - Hosting Method (Nginx/Apache/Docker): Docker **Additional context** Add any other context about the problem here.

OVERLORD added the 🐛 Bug 💆 UX 💻 Front-End labels 2026-02-05 00:45:42 +03:00

OVERLORD closed this issue

2026-02-05 00:45:45 +03:00

OVERLORD commented

2026-02-05 00:45:52 +03:00

@ssddanbrown commented on GitHub (Oct 3, 2019):

Good find @cw1998, Thanks for reporting.

@ssddanbrown commented on GitHub (Oct 3, 2019): Good find @cw1998, Thanks for reporting.

OVERLORD commented

2026-02-05 00:45:57 +03:00

@dellamina commented on GitHub (Oct 30, 2019):

Hey @ssddanbrown I'd love to give some help in this one if you are not already working on it :)

@dellamina commented on GitHub (Oct 30, 2019): Hey @ssddanbrown I'd love to give some help in this one if you are not already working on it :)

OVERLORD commented

2026-02-05 00:46:01 +03:00

@ssddanbrown commented on GitHub (Dec 16, 2019):

Sorry @dellamina, Totally missed your message at the time.

I've now applied a patch for this in master, to be in the next release. I went for option 3 so that sections will now auto-expand on load if found to be containing an error message so that the error message is visible.

@ssddanbrown commented on GitHub (Dec 16, 2019): Sorry @dellamina, Totally missed your message at the time. I've now applied a patch for this in master, to be in the next release. I went for option 3 so that sections will now auto-expand on load if found to be containing an error message so that the error message is visible.

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/BookStack#1388