[ICE-6041] TabIndex causes focus issue when toggling through selectOneRadio using keyboard - ICEsoft JIRA Issue Tracker

Details

Type: Bug
Status: Closed
Priority: Major
Resolution: Fixed
Affects Version/s: 1.8.2-EE-GA_P01
Fix Version/s: 1.8.3, 1.8.2-EE-GA_P02, 2.0.0
Component/s: ICE-Components
Labels:
None
Environment:
IE7/8, FF3.6.9

Workaround Exists:

Yes
Workaround Description:
Do not use tabIndex.

Description

The tabIndex attribute is causing a focus issue with the ice:selectOneRadio. When navigating the form using the keyboard and tabbing into an ice:selectOneRadio, focus is placed on the parent element and not the first radio item in the list. When you hit tab again, you proceed to the next component. However, hitting tab again places focus back onto the first radio item in the list. Test case attached. This issue does not occur when the tabIndex is removed.

Options
- Sort By Name
- Sort By Date
- Ascending
- Descending
- Download All

Attachments

Hide
sc9295.war

29/Sep/10 8:41 AM

6.68 MB

Tyler Johnson
META-INF/MANIFEST.MF 0.0 kB

WEB-INF/lib/backport-util-concurrent.jar 319 kB

WEB-INF/lib/commons-digester.jar 140 kB

WEB-INF/lib/FastInfoset.jar 285 kB

WEB-INF/lib/icefaces-comps.jar 1.96 MB

ICEfacesPage1.xhtml 1 kB

WEB-INF/lib/jxl.jar 708 kB

WEB-INF/.../krysalis-jCharts-1.0.0-alpha-1.jar 151 kB

WEB-INF/lib/commons-beanutils.jar 226 kB

WEB-INF/classes/Bean.java 0.6 kB

WEB-INF/faces-config.xml 0.6 kB

WEB-INF/lib/commons-fileupload.jar 56 kB

WEB-INF/lib/jsf-impl.jar 816 kB

index.jsp 0.1 kB

WEB-INF/lib/jsf-api.jar 350 kB

WEB-INF/lib/icefaces.jar 1.20 MB

WEB-INF/web.xml 4 kB

WEB-INF/lib/commons-logging.jar 52 kB

WEB-INF/lib/icefaces-facelets.jar 596 kB

WEB-INF/lib/commons-collections.jar 558 kB

WEB-INF/classes/Bean.class 1 kB

Download Zip
Show

sc9295.war

29/Sep/10 8:41 AM

6.68 MB

Tyler Johnson

Activity

Ascending order - Click to sort in descending order

Tyler Johnson created issue - 13/Sep/10 10:33 AM

Tyler Johnson made changes - 13/Sep/10 10:36 AM

Field	Original Value	New Value
Issue Type	Improvement [ 4 ]	Bug [ 1 ]
Workaround Description		<script> function focusRadio() { var radio=document.getElementById("form:selectRadio:_1" ); radio.focus(); } </script> <ice:selectOneRadio id="selectRadio" onfocus="focusRadio()" layout="lineDirection"> <f:selectItems value="#{bean.testList}" /> </ice:selectOneRadio>
Environment	IE7/8, FF3	IE7/8, FF3.6.9
Workaround Exists		[Yes]
Salesforce Case		[5007000000C5Duz]

Tyler Johnson made changes - 29/Sep/10 8:21 AM

Summary	Cannot toggle through selectOneRadio options using keyboard	TabIndex causes focus issue when toggling through selectOneRadio using keyboard
Description	When tabbing to a ice:selectOneRadio using the keyboard, focus is applied to the parent element and not the first radio item in the list. This behavior also occurs with the h:selectOneRadio component. This improvement should be added to the component but a very simple javascript workaround exists.	The tabIndex attribute is causing a focus issue with the ice:selectOneRadio. When navigating the form using the keyboard and tabbing into an ice:selectOneRadio, focus is placed on the parent element and not the first radio item in the list. When you hit tab again, you proceed to the next component. However, hitting tab again places focus back onto the first radio item in the list. Test case attached. This issue does not occur when the tabIndex is removed.

Tyler Johnson made changes - 29/Sep/10 8:22 AM

Workaround Description	<script> function focusRadio() { var radio=document.getElementById("form:selectRadio:_1" ); radio.focus(); } </script> <ice:selectOneRadio id="selectRadio" onfocus="focusRadio()" layout="lineDirection"> <f:selectItems value="#{bean.testList}" /> </ice:selectOneRadio>
Workaround Exists	[Yes]

Tyler Johnson made changes - 29/Sep/10 8:41 AM

Attachment

sc9295.war [ 12579 ]

Ken Fyten made changes - 29/Sep/10 10:31 AM

Assignee

Ken Fyten [ ken.fyten ]

Ken Fyten made changes - 29/Sep/10 10:31 AM

Fix Version/s

1.8.2-EE-GA_P02 [ 10226 ]

Tyler Johnson made changes - 29/Sep/10 10:59 AM

Workaround Description		Do not use tabIndex.
Workaround Exists		[Yes]

Ken Fyten made changes - 20/Oct/10 10:19 AM

Assignee Priority

Ken Fyten made changes - 29/Oct/10 11:45 AM

Fix Version/s		1.8.3 [ 10211 ]
Assignee	Ken Fyten [ ken.fyten ]	Yip Ng [ yip.ng ]

Ken Fyten made changes - 29/Oct/10 11:46 AM

Fix Version/s

2.0.0 [ 10230 ]

Repository	Revision	Date	User	Message
ICEsoft Public SVN Repository	#22989	Mon Nov 01 13:03:37 MDT 2010	yip.ng	~~ICE-6041~~: TabIndex causes focus issue when toggling through selectOneRadio using keyboard.
				Files Changed
				MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/renderkit/dom_html_basic/SelectManyCheckboxListRenderer.java MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/renderkit/dom_html_basic/RadioRenderer.java

Repository	Revision	Date	User	Message
ICEsoft Public SVN Repository	#22990	Mon Nov 01 13:15:30 MDT 2010	yip.ng	~~ICE-6041~~: TabIndex causes focus issue when toggling through selectOneRadio using keyboard.
				Files Changed
				MODIFY /icefaces2/trunk/icefaces/compat/core/src/main/java/com/icesoft/faces/renderkit/dom_html_basic/RadioRenderer.java MODIFY /icefaces2/trunk/icefaces/compat/core/src/main/java/com/icesoft/faces/renderkit/dom_html_basic/SelectManyCheckboxListRenderer.java

Hide

Permalink

yip.ng added a comment - 01/Nov/10 3:23 PM - edited

Revision: 22989

Modified : /icefaces/trunk/icefaces/core/src/com/icesoft/faces/renderkit/dom_html_basic/RadioRenderer.java
Modified : /icefaces/trunk/icefaces/core/src/com/icesoft/faces/renderkit/dom_html_basic/SelectManyCheckboxListRenderer.java

Revision: 22990

Modified : /icefaces2/trunk/icefaces/compat/core/src/main/java/com/icesoft/faces/renderkit/dom_html_basic/RadioRenderer.java
Modified : /icefaces2/trunk/icefaces/compat/core/src/main/java/com/icesoft/faces/renderkit/dom_html_basic/SelectManyCheckboxListRenderer.java

Show

yip.ng added a comment - 01/Nov/10 3:23 PM - edited Revision: 22989 Modified : /icefaces/trunk/icefaces/core/src/com/icesoft/faces/renderkit/dom_html_basic/RadioRenderer.java Modified : /icefaces/trunk/icefaces/core/src/com/icesoft/faces/renderkit/dom_html_basic/SelectManyCheckboxListRenderer.java Revision: 22990 Modified : /icefaces2/trunk/icefaces/compat/core/src/main/java/com/icesoft/faces/renderkit/dom_html_basic/RadioRenderer.java Modified : /icefaces2/trunk/icefaces/compat/core/src/main/java/com/icesoft/faces/renderkit/dom_html_basic/SelectManyCheckboxListRenderer.java

Hide

Permalink

yip.ng added a comment - 01/Nov/10 6:19 PM

Fixed.

Note that once tabbed to a radio button group, the standard way of navigating within the radio buttons in the group is using the arrow keys.

Show

yip.ng added a comment - 01/Nov/10 6:19 PM Fixed. Note that once tabbed to a radio button group, the standard way of navigating within the radio buttons in the group is using the arrow keys.

yip.ng made changes - 01/Nov/10 6:19 PM

Status	Open [ 1 ]	Resolved [ 5 ]
Resolution		Fixed [ 1 ]

Ken Fyten made changes - 03/Jan/11 4:19 PM

Status	Resolved [ 5 ]	Closed [ 6 ]
Assignee Priority	P1

People

Assignee:

yip.ng

Reporter:

Tyler Johnson

Votes:

0 Vote for this issue

Watchers:

0 Start watching this issue

Dates

Created:

13/Sep/10 10:33 AM

Updated:

03/Jan/11 4:19 PM

Resolved:

01/Nov/10 6:19 PM