Bug 87466 – select multiple box (size>1) is not displayed if no options are listed. (testcase)

First Last Prev Next No search results available Search page

Bug 87466:

select multiple box (size>1) is not displayed i...

Product:	konqueror
Component:	khtml
Status:	RESOLVED
Resolution:	FIXED
Target:	---

Version:	unspecified
Priority:	NOR
Severity:	normal

Votes:

Description:
Opened: 2004-08-18 22:15 Last Changed: 2004-10-15 17:19:06

Version: (using KDE Devel) Installed from: Compiled sources Compiler: gcc 3.2.2 OS: Linux A select element with size>1 collapses to a thin line if no options are listed, where it should be displayed empty. The select boxes in the search page at http://www.jobcentreplus.gov.uk/cms.asp?Page=/Home/Customers/Searchforajob are empty at startup, but are then filled using javascript. There are other problems with this page (it doesn't work in mozilla either, but at least the select boxes appear right to start with) but this prevent me from working those out. testcase: the second select element is not displayed, whereas the first and third are. <html><head></head><body> <P>Select size 1 <select SIZE="1"></select> <P>Select size 2 <select SIZE="2"></select> <P>Select size 2 with options <select SIZE="2"> <option value="1">1</option> <option value="2">2</option> <option value="3">3</option> </select> </body></html>

Comment #1 Sebastian Nohn 2004-09-14 10:19:03

verified. another example: [...] <select style="width: 300px" name="addAspect" size="15" multiple="multiple"> </select> [...] <select style="width: 300px" name="selAspect" size="15" multiple="multiple"> <option value="15">aol.de</option> <option value="11">google.com</option> [...] </select> [...]

Comment #2 Stephan Kulow 2004-10-15 17:19:06

CVS commit by coolo: don't assume listboxes have items CCMAIL: 87466-done@bugs.kde.org M +7 -3 render_form.cpp 1.279 --- kdelibs/khtml/rendering/render_form.cpp #1.278:1.279 @@ -1028,4 +1028,8 @@ void RenderSelect::layout( ) p = p->next(); } + if ( !height ) + height = w->fontMetrics().height(); + if ( !width ) + width = w->fontMetrics().width( 'x' ); int size = m_size;

Platform:	Compiled Sources
OS:	Linux
Keywords:

People

Reporter:	richard missenden
Assigned To:	Konqueror Developers
CC:

Related actions

Note

You need to log in before you can comment on or make changes to this bug.

Attachments
Add an attachment (proposed patch, testcase, etc.)

Depends on:
Blocks:
	Show dependency tree - Show dependency graph

First Last Prev Next No search results available Search page

Actions

Reports

My Account