incorrect overflow handling selectmenu[multiple] inside fieldset #6077

Closed
cautionbug opened this Issue Jun 5, 2013 · 3 comments

Comments

Projects
None yet
3 participants
@cautionbug

i think this one's legit:
http://jsfiddle.net/3yEUu/6/

When a SELECT[multiple] is inside a FIELDSET, text-overflow is not applied to truncate with an ellipsis.

The only (seemingly) relevant CSS i can find is:

.ui-select .ui-btn-text { text-overflow: ellipsis; }

DOM Paths (starting from data-role="collapsible" tags):

div.ui-collapsible > div.ui-collapsible-content > h3 > div.ui-select > a.ui-btn > span.ui-btn-inner > span.ui-btn-text > span
fieldset.ui-collapsible > div.ui-collapsible-content > div.ui-select > a.ui-btn > span.ui-btn-inner > span-ui-btn-text > span

It appears the H3 as a descendant plays a critical role somewhere in JS? However, i tried H3 inside the FIELDSET. It's correctly turned into a title, but the HTML is the same as the other FEILDSET.

EDIT:
jQM 1.3.1 / jQ 1.10.1, just for clarification. (also tried 1.9 in the fiddle, same result)

EDIT 2:
May want to change title. i realized after that i'd specified "collapsible" fieldset. The issue isn't dependent on collapsible.

@jaspermdegroot

This comment has been minimized.

Show comment
Hide comment
@jaspermdegroot

jaspermdegroot Jun 8, 2013

Member

JS Bin with latest code: http://jsbin.com/ayumal/32/edit

Member

jaspermdegroot commented Jun 8, 2013

JS Bin with latest code: http://jsbin.com/ayumal/32/edit

@ghost ghost assigned jaspermdegroot Jul 2, 2013

jaspermdegroot added a commit that referenced this issue Jul 2, 2013

Core CSS: Unset browser default min-width for fieldset. Fixes #6077.
The min-width was causing issues in combination with white-space nowrap to get an ellipsis.
@thojo

This comment has been minimized.

Show comment
Hide comment
@thojo

thojo Jul 20, 2013

Fix does't work in Firefox 22! IE10, Opera 12.16 OK.

thojo commented Jul 20, 2013

Fix does't work in Firefox 22! IE10, Opera 12.16 OK.

@jaspermdegroot

This comment has been minimized.

Show comment
Hide comment
@jaspermdegroot

jaspermdegroot Jul 20, 2013

Member

Thanks @thojo, will look into Firefox.

Member

jaspermdegroot commented Jul 20, 2013

Thanks @thojo, will look into Firefox.

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