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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Flying Drupalist’s picture

Flying Drupalist CreditAttribution: Flying Drupalist commented

Yeah I agree.

quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch commented
Status: Active » Fixed
FileSize
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 CreditAttribution: Kane commented

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