[wp-trac] Re: [WordPress Trac] #5369: jQuery should be used to fade alerts

[WordPress Trac]

#5369: jQuery should be used to fade alerts
----------------------------+-----------------------------------------------
 Reporter:  JeremyVisser    |        Owner:  JeremyVisser
     Type:  enhancement     |       Status:  assigned    
 Priority:  low             |    Milestone:  2.5         
Component:  Administration  |      Version:              
 Severity:  minor           |   Resolution:              
 Keywords:                  |  
----------------------------+-----------------------------------------------
Comment (by JeremyVisser):

 A side-effect for this patch: Fat.fade_all() is no longer called unless
 there is a {{{.alert.fade}}} present, so if you manually create a {{{<div
 id="mymessage" class="fade-ff0000 alert">}}} (i.e. a red fading alert), it
 won't fade anymore. You could always call
 {{{jQuery(".alert#mymessage").vkfade("ff0000")}}} but that's inferior to
 just using FAT.

 Also, I strongly believe the "fade" class is unsemantic, and the "alert"
 class for alerts should be just used, and the fade should be implied. For
 red fades, an "error" class could be added.

 Could FAT be rewritten to use jQuery.animate()?

 Ironically, I'm currently -1 to my own patch in its current state.

-- 
Ticket URL: <http://trac.wordpress.org/ticket/5369#comment:4>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software