Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Option rendering, specifically when dealing with SelectItemGroups, is too generic #420

Closed
eclipse-faces-bot opened this issue Jul 8, 2008 · 17 comments

Comments

@eclipse-faces-bot
Copy link

HTML spec states that optgroups are valid for select elements, however, the
current option rendering language in the renderkit docs applies optgroup to all
cases where SelectItems are rendered.

Environment

Operating System: All
Platform: All

Affected Versions

[1.2]

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
Reported by @rlubke

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
Move to unscheduled target milestone

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
Prepare to delete "spec" subcomponent.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
kito75 said:
Changed subcomponent to Components/Renderers.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
cat1

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
rk

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
These are valid 2.0 Rev a issues

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:

On Mon, 03 May 2010 09:26:16 -0700, Ryan Lubke ryan.lubke@oracle.com said:

RL> I believe the root of the problem is that option rendering is generic to
RL> all components that use
RL> SelectItems, however, optgroup wouldn't be valid for checkboxes or radio
RL> buttons - per the HTML spec.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
take ownership.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
fix checked in.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
Created an attachment (id=234)
fix for this bug, first iteration.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
File: diffs.txt
Attached By: @edburns

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
rogerk said:
changelog

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
Marked as fixed on Monday, June 21st 2010, 9:45:27 am

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@manfredriem said:
Closing resolved issue out

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
This issue was imported from java.net JIRA JAVASERVERFACES_SPEC_PUBLIC-420

@eclipse-faces-bot
Copy link
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant