JIRA > FLUID-3225

[Joan Garcia Vila]

Hi jacob.

Do I have to work in FLUID-3225?. It's an interesting thing remove duplicated event calls.
If so, please let me know.

If not, please, this a "request for JIRAs" to work in.

thanks in advance,
cheers,
joan.


--> Missatge original de Jacob Farber (jacob.farber at utoronto.ca) per a Joan García Vila enviat el 05/10/2009 19:33:17

Hello Joan,<br/>I'm sorry to bring this up again, but it seems like there was a small bug when using the tab key to get outside the code view Tabs. I think the bug is part of something bigger - http://issues.fluidproject.org/browse/FLUID-3225 - and so I couldn't come up with a super graceful work around. Instead, I had to remove the unSelect event and put that functionality inside the onSelect event instead.<br/><br/>One more little point is I just did a small tidy up of the code... it turns out we don't need to say "fluid.activate()" and all the activatable code setup when we can just fire the "click()" event bound to each tab.<br/><br/>I hope this makes sense!<br/>Jacob<br/><br/><br/>-----Original Message-----<br/>From: Colin Clark [mailto:colinbdclark at gmail.com] <br/>Sent: Sunday, October 04, 2009 10:30 AM<br/>To: Joan Garcia Vila<br/>Cc: Jacob Farber; fluid-work at fluidproject.org<br/>Subject: Re: JIRA > Accessibility for demos portal > ask for review (3)<br/><br/>Hi Joan,<br/><br/>I've reviewed your patch and it looks good. I went ahead and applied  <br/>it to 1.1.2.<br/><br/>While reviewing, I also noticed that we need to take the anchors  <br/>inside each Tab out of the keyboard tab order. I created a one-line  <br/>fix for this and committed.<br/><br/>We're all done with FLUID-3197. Thanks for all your help!<br/><br/>Colin<br/><br/>On 1-Oct-09, at 4:47 AM, Joan Garcia Vila wrote:<br/><br/>> Hi.<br/>><br/>> This is an "Ask for review" mail.<br/>> Applied Colin's 1 and 2 suggestions to the code.<br/>><br/>> Jira: http://issues.fluidproject.org/browse/FLUID-3197<br/>><br/>><br/>> Thanks in advance,<br/>> cheers,<br/>> joan.<br/><br/>---<br/>Colin Clark<br/>Technical Lead, Fluid Project<br/>http://fluidproject.org<br/><br/>