Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove spurious ; characters from html #1200

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jrmhaig
Copy link

@jrmhaig jrmhaig commented Feb 27, 2025

Identify the bug

The file list display on the multi file upload component broke between version 3.3.1 and 3.4.0.

Here is the display in version 3.3.1:

Screenshot 2025-02-27 at 13 29 16

and here it is in 3.4.0 (and later):

Screenshot 2025-02-27 at 13 30 36

I have traced it back to d7c1dab where the construction of a block of html was altered.

Fixes #1199

Description of the change

I have removed some ; characters that were left in during some previous linting.

Verification process

I have been unable to test using this branch but the change appears relatively simple.

Release notes

  • Fix the display of uploaded files in the multi-file upload component.

@jrmhaig jrmhaig requested a review from a team as a code owner February 27, 2025 14:15
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Broken display of uploaded files
1 participant