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 ([email protected]) 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:[email protected]] <br/>Sent: Sunday, October 04, 2009 10:30 
AM<br/>To: Joan Garcia Vila<br/>Cc: Jacob Farber; 
[email protected]<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/>
_______________________________________________________
fluid-work mailing list - [email protected]
To unsubscribe, change settings or access archives,
see http://fluidproject.org/mailman/listinfo/fluid-work

Reply via email to