FLUID-110: Tab moves focus through all Lightbox images on IE7

Description

To reproduce:

• open http://build.fluidproject.org:8080/sakai-imagegallery-tool/component-templates/html/Lightbox.html
• repeatedly press tab

Expected:

• the Lightbox to act as one tab stop in the document

Actual:

• tab moves through all images: first the image, then the label for the image, then the next image, and so on

Environments

IE7
Windows XP

Comments

• Michelle D'Souza commented 2007-11-14T18:04:05.000-0500

  Removed the anchors in the thumbnails from the tabindex.

• Joseph Scheuhammer commented 2007-11-15T10:12:19.000-0500

  Adding a tabindex attribute with a value of "-1" does fix the problem. However, it does not gracefully degrade in the case where JavaScript is turned off. That is, there is no keyboard access to the links when JavaScript is disabled since the tabindex will continue to disallow it. This suggests that negative tabindex values should be added dynamically.

  An issue is that the elements that need the negative tabindex are not constant across orderable situations. In the Lightbox, the required elements are a pair of anchors within the thumbnails. In the uPortal tabbed navigation example, it is a single anchor. In the vertical list example, no element needs a negative tabindex. It is conceivable that within a form containing orderables that some of the form fields (inputs, buttons) should be taken out of the tab order. What should be removed is different in each case.

  All this suggests that there needs to a way to inform the Reorderer which elements, if any, require a negative tabindex – another parameter to its operation.

• Michelle D'Souza commented 2007-11-20T14:21:02.000-0500

  Graceful degradation is now in place. The anchors are removed from the taborder in the initLightbox function.