snoopdave commented on code in PR #245:
URL: https://github.com/apache/roller/pull/245#discussion_r4569217004
##########
app/src/main/webapp/WEB-INF/jsps/tiles/messages.jsp:
##########
@@ -19,8 +19,9 @@
<script>
$(document).ready(function () {
- // Remove the alert box after 10 seconds
- $(".alert").delay(10000).slideUp(200, function() {
+ // Remove the action message boxes below after 10 seconds. Only these:
other
+ // alerts on the page, such as error boxes in dialogs, must stay.
+ $("#messages, #errors").delay(10000).slideUp(200, function() {
Review Comment:
🐞Claude Issue: **Important:** this tile is shared by every page that shows
messages, not only Categories. Narrowing the auto-hide from `.alert` to
`#messages, #errors` changes behavior elsewhere: any other `.alert` box that
used to fade after 10 seconds (for example in ThemeEdit, TemplateEdit,
MediaFileAdd or WeblogConfig) now stays. Check those pages, or keep the old
selector and exclude the dialog alerts with `.not()` instead, so the change
stays local to what this PR needs.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
