- Issue created by @dineshkumarbollu
- First commit to issue fork.
- Merge request !6753Issue #3423384: Extra Close icon in Messages โ (Closed) created by govind_giri_goswami
- Status changed to Needs review
4 months ago 10:50am 23 February 2024 - ๐ฎ๐ณIndia govind_giri_goswami
changes in closeMessage() function, which adds a close button to a message container if one doesn't already exist, and hides the message when the close button is clicked
- Status changed to Needs work
4 months ago 2:57pm 23 February 2024 - ๐บ๐ธUnited States smustgrave
Would be good to add solution to the issue summary.
- ๐ฎ๐ณIndia shreyal999
The code should have indentation or line gap as in the previous version for code clarity.
- ๐ฎ๐ณIndia Gauravvv Delhi, India
Gauravvvv โ made their first commit to this issueโs fork.
- ๐ท๐บRussia kostyashupenko Omsk
Wondering what is the cause of that fact that
once()
in that case doesn't actually work (works 2 times) - Status changed to RTBC
4 months ago 6:18am 4 March 2024 - ๐ฎ๐ณIndia adwivedi008
I tested the MR mentioned in #4, and it fixes the issue
So moving the issue to RTBC - ๐บ๐ธUnited States andy-blum Ohio, USA
nod_ โ credited andy-blum โ .
- ๐บ๐ธUnited States mherchel Gainesville, FL, US
nod_ โ credited mherchel โ .
- ๐ฎ๐ณIndia Ruturaj Chaubey Pune, India
nod_ โ credited Ruturaj Chaubey โ .
- ๐ซ๐ทFrance nod_ Lille
when bigpipe is enabled, the status message comes from a bigpipe/ajax command.
The message commands adds a new message using Drupal.theme('message') it calls olivero implementation of the theme function that adds a close button. Nothing is using once at this point. Then when the page loads, behaviors are called, and a second close button is added to the message. Once is working as expected it's just not at the right place.
Theme functions should be returning plain strings instead of objects, that would have prevent the double initialization.
Fix makes sense in this situation.
- Status changed to Fixed
4 months ago 10:16am 4 March 2024 Automatically closed - issue fixed for 2 weeks with no activity.