Hello, please add fadeOut function for flag-message class

file: flag.js
line 45:

$('.flag-message', $newLink).fadeIn();

Add new line:
setTimeout(function(){ $('.flag-message').fadeOut() }, 2000);

This will hide message text after 2 seconds, so it won't be near the flag all the time.

Thanks.

CommentFileSizeAuthor
#2 flag_fadeout.patch498 bytesquicksketch

Comments

Flying Drupalist’s picture

Flying Drupalist commented

Yeah I agree.

quicksketch’s picture

quicksketch
he/him
commented
Status: Active » Fixed
StatusFileSize
new flag_fadeout.patch498 bytes

I went with a 3 second delay instead of 2, since my messages can get pretty long and I'd like to be able to read the entire message twice before it disappears. I also added in the context of $newLink, so it only fades out the current link message, not all the link messages on the page (not that it matters much since there should only be one anyway, but it's more efficient). Thanks for the suggestion.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Kane’s picture

Kane commented

It seems like in one of the latest releases that patch wasn't included?