New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix double "x" in close preview modal button #1312
base: develop
Are you sure you want to change the base?
Conversation
The close modal button currently has two 'X's on top of each other. Bootstrap provides the X as a properly sized SVG via the CSS background property so the 'x' within the HTML <button> is not needed.
@@ -10,7 +10,7 @@ | |||
<div class="modal-content"> | |||
<div class="modal-header"> | |||
<h5 class="modal-title"></h5> | |||
<button type="button" class="btn-close" data-bs-dismiss="modal" aria-label="Close">×</button> | |||
<button type="button" class="btn-close" data-bs-dismiss="modal" aria-label="Close"></button> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tried on my projectsand as well and it does work the same way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For some reason didn't help me. I mean, if I remove "x" via inspect - it works, but editing file_preview_modal.js still gives me the "x" in the page code. Tried to clear the cache/different browser without any luck.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you rebuild the project with gulp
or gulp prod
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, I didn't. Should I do that?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, otherwise you won't see changes made to file_preview_modal.js
.
Thank you for this great project!
The close button on the preview modal currently has two
X
's on top of each other. Bootstrap 5 modals provide theX
as a properly sized SVG via the CSSbackground
property so the'x'
text within the HTML<button>
is not needed.Before:
After:
I did a search in this repo's issues and found that this was reported once in #1225 but there is also an unrelated issue reported in that one so I didn't want to automatically close it with this PR. At least mentioning it here will link this PR to that issue.