JIRA > FLUID-3197 > Accessibility for demos portal > ask for review (2)
Joan Garcia Vila
jgarciavila at uoc.edu
Wed Sep 30 09:25:35 UTC 2009
I will review and fix the JIRA FLUID-3197 submitted patch.
I start JIRA FLUID-3121 task.
Many thanks for the startup support (collin and michelle).
--> Missatge original de Colin Clark (colinbdclark at gmail.com) per a Joan García Vila, Jacob Farber enviat el 30/09/2009 00:54:08
Great work on this patch, I'm impressed! You've definitely got all the
necessary functionality in here, making our demo portal more accesible.
Jacob went ahead and committed the patch, but I think he was a bit
hasty in his review. There are a couple of small changes I'd like to
see in place before we add this patch to the Infusion 1.1.2 release.
They shouldn't take you long at all, and will improve our code and
reduce the number of repetitive changes you'll have to make.
1. Try adding the tabindex attribute to each of the Tab containers in
container shouldn't be in the tab order. In general, it's best to add
both ARIA and tabindex change in code, not in the markup.
2. A tiny detail: Can you make sure we've got a line break and the
standard 4 spaces between script tags in each HTML file?
Jacob, can you back your commit of Joan's patch out and commit his new
patch when he implements these small changes?
As for suggestions for new issues to work on:
This issue is a bit more subtle than the last one I gave you, but it
should be doable. In short, all of our components have a basic
contract with the user: all components must be rooted in a container
element. That container can be specified as anything "jQueryable:" a
jQuery instance, an element, or a selector string.
In the Reorderer, we have a couple of cases where containers are
passed around without first calling fluid.container() on them or
wrapping them as jQuery instances. It's a minor fix, but something
that will require a bit of thought. The user who filed this issue was
pretty comprehensive, so it should be straightforward.
Once again, great work Joan and it's nice to have you on board!
On 29-Sep-09, at 1:42 PM, Jacob Farber wrote:
> Thanks for the patch Joan, looks great. Better still, using the
> keyboard I found a bug in the code I wrote.
> I think its time to change the demo portal code into component-
> friendly code J
> From: fluid-work-bounces at fluidproject.org [mailto:fluid-work-bounces at fluidproject.org
> ] On Behalf Of Joan Garcia Vila
> Sent: Tuesday, September 29, 2009 12:07 PM
> To: Joan García Vila; colinbdclark at gmail.com
> Cc: fluid-work at fluidproject.org
> Subject: RE: JIRA > Accessibility for demos portal > ask for review
> The major changes are in:
> change in the ul html tag..
> --> Missatge original de Joan García Vila (jgarciavila at uoc.edu) per
> a "colinbdclark at gmail.com" enviat el 29/09/2009 18:03:08
> - JIRA:
> Hello Collin and Justin, this is a review request for the patch
> posted to the previous jira url.
> I've modified the in order to place the role and the aria state
> (selected) to the tabs created as a response of the XmlHttpRequest.
> Hope the code would be good enough.
> Please, could you assign me new JIRAS.
> Many thanks in advance,
Technical Lead, Fluid Project
More information about the fluid-work