Opened 12 years ago

Closed 12 years ago

Last modified 11 years ago

#10923 closed defect (fixed)

Select: drop down sizing problems

Reported by: bill Owned by: bill
Priority: high Milestone: 1.5
Component: Dijit - Form Version: 1.4.0
Keywords: Cc:
Blocked By: Blocking:


There are two sizing problems with the Select drop down menu, which become a bit more apparent with the new claro theme but were actually present before.

  1. no hover effect on right edge of MenuItem

Notice how the right edge of the menu item doesn't have the blue hover effect:

hover effect problem

  1. horizontal scroll bar appears

horizontal scrollbar problem

In both cases the problem deals with sizing of the inner <table> node to match the size of the wrapper <div> node, where the wrapper <div> node's content-size will vary depending on the presence of a vertical scrollbar and the width of the scrollbar, which depends on the browser and OS.

Attachments (2)

short.png (7.1 KB) - added by bill 12 years ago.
hover effect problem
scroll.png (20.1 KB) - added by bill 12 years ago.
horizontal scrollbar problem

Download all attachments as: .zip

Change History (5)

Changed 12 years ago by bill

Attachment: short.png added

hover effect problem

Changed 12 years ago by bill

Attachment: scroll.png added

horizontal scrollbar problem

comment:1 Changed 12 years ago by bill

Resolution: fixed
Status: newclosed

(In [21669]) Various fixes/simplifications so that Select's dropdown's inner <table> node is neither too wide nor too narrow. Main fix is to use width: 100% on <table> rather than having browser specific code to gauge width of scroll bar.

I think this brings _HasDropDown in line with how ComboBox is dealing with it's drop down.

Fixes #10923 !strict.

comment:2 Changed 12 years ago by bill

(In [22248]) Fix regression from [21669], case problem causing unwanted horizontal scrollbar on webkit. Refs #10923, fixes #11205 !strict.

comment:3 Changed 11 years ago by bill

Component: DijitDijit - Form
Note: See TracTickets for help on using tickets.