Bug 14214

Summary: [S60] Dropdown not opened if it has size defined.
Product: WebKit Reporter: Hong Zhao <hong.a.zhao>
Component: WebCore Misc.Assignee: Nobody <webkit-unassigned>
Status: CLOSED FIXED    
Severity: Normal Keywords: PlatformOnly
Priority: P2    
Version: 523.x (Safari 3)   
Hardware: S60 Emulator   
OS: S60 3rd edition   
Attachments:
Description Flags
Webcore doesn't support multiple size in dropdown list.
zalan: review-
WebCore doesn't support multiple visible items in the dropdown list in 3.2.
none
Make IsEnable() inside KWQListBox derived from its parent QWidget in 3.1m.
zalan: review+
Fix bug in dropdown list not opened if it has size defined. zalan: review+

Description Hong Zhao 2007-06-18 09:09:21 PDT
TSW ID# PNIO-73UHXL

Steps to reproduce:

- Steps to reproduce:

1. Go to http://waplabdc.nokia-boston.com/browser/users/xbasic/xhtml_forms/select/select_size.xhtml 

2. Click dropdown 

- Actual outcome:
Dropdown doesn't open.

- Expected outcome:
Dropdown opens

- Other info:
Dropdown in question has size="2", dropdowns without size definition seem to work OK.
Comment 1 Hong Zhao 2007-06-18 10:26:11 PDT
Created attachment 15103 [details]
Webcore doesn't support multiple size in dropdown list.
Comment 2 Hong Zhao 2007-06-18 10:32:23 PDT
Created attachment 15104 [details]
WebCore doesn't support multiple visible items in the dropdown list in 3.2.
Comment 3 zalan 2007-06-20 13:59:52 PDT
Comment on attachment 15103 [details]
Webcore doesn't support multiple size in dropdown list.

this patch changes behavior of single select with size>1 by diverting it form m_useListBox = true to false. The issue here is that when the renderselect constructs ListBox instead of ComboBox, then the activate event does not get through. That's what  needs to be fixed here. (you can check it out by coming up with testcase with <select multiple="multiple">. it'll construct a ListBox and that listbox cannot be activated either). so even if this patch was applied, the multiselect  testcase would still not be working.)
Comment 4 Hong Zhao 2007-06-22 06:25:16 PDT
Created attachment 15181 [details]
Make IsEnable() inside KWQListBox derived from its parent QWidget in 3.1m.
Comment 5 Hong Zhao 2007-06-22 06:58:39 PDT
Created attachment 15183 [details]
Fix bug in dropdown list not opened if it has size defined.
Comment 6 zalan 2007-06-25 08:22:05 PDT
Comment on attachment 15181 [details]
Make IsEnable() inside KWQListBox derived from its parent QWidget in 3.1m.

r=me
Comment 7 zalan 2007-06-25 08:30:51 PDT
Comment on attachment 15183 [details]
Fix bug in dropdown list not opened if it has size defined.

r=me
Comment 8 zalan 2007-06-25 08:45:08 PDT
r23765 -> 3.1m
r23766 -> ccb
Comment 9 Bradley Morrison 2008-04-09 11:39:41 PDT
Bulk closing of all s60 platform bugs. 

Sorry for the noise!
Comment 10 Joel Parks 2011-03-21 11:53:31 PDT
re-purposing InTSW keyword for use by QtWebkit team