fnagel / jquery-ui

A fork of jQuery UI: WIP branches, legacy Selectmenu Widget (branch: selectmenu) and an accessible version of jQuery UI Tabs (outdated, branch: tabs accessability)
Other
588 stars 141 forks source link

Spaces in optgroup label causes duplications #236

Closed asaworld closed 12 years ago

asaworld commented 12 years ago

I saw someone else reported this 2 years ago and you said that you could not recreate it. The below code causes this issue to appear (optgroups are not grouped). Remove the spaces in the optgroup label and it works just fine. I only downloaded this library a few days ago.

<select name="data[sport]" class="selectmenu" id="sport">
<optgroup label="My Sports">
<option value="Kayaking">Kayaking</option>
<option value="Rock climbing">Rock climbing</option>
</optgroup>
<optgroup label="Other Sports">
<option value="Surfing">Surfing</option>
<option value="Skateboarding">Skateboarding</option>
</optgroup>
</select>

Thanks, Asa

fnagel commented 12 years ago

Thanks for your contribution! Ive tested your code in a fiddle (http://jsfiddle.net/GXtpC/475/) but it worked as expected.

Did you find the old issue?

fnagel commented 12 years ago

Any feedback on this issue?

asaworld commented 12 years ago

Still seeing this happen. I have put together the simple code in jsfiddle. Can you see this happen?

http://jsfiddle.net/GXtpC/497/

fnagel commented 12 years ago

You are using outdated code. Please update!

fnagel commented 12 years ago

Any feedback on this issue?

asaworld commented 12 years ago

You are right, I think you can close this one.

On Mon, Aug 6, 2012 at 2:04 PM, Felix Nagel notifications@github.comwrote:

Any feedback on this issue?

— Reply to this email directly or view it on GitHubhttps://github.com/fnagel/jquery-ui/issues/236#issuecomment-7536252.

fnagel commented 12 years ago

Thanks for your feedback! Closed as solved.