Doubleclick handling for <select> tag (#71427)

[rwlbuis at xs4all.nl]

Hi Germain,

> mmh, you are speaking about converting createListBox to __khtml, right?

Yes.

> That's also a QScrollView, so we have code to specifically leave those
> out,
> just like textarea.
> see KHTMLView::drawContents, RenderReplaced::paintWidget and maybe a few
> other
> places.

Right. Well David and SadEagle gave me a few pointers, and I patched
khtml enough so for the KListBox the painting and mouse events seem
to be handled correctly now, and in the same way as for any other widget,
i.e. it is handled in KHTMLView::eventFilter like for other render
widgets. In fact paint and some key events are untouched, while mouse
events are handled and not given back to the KListBox event handler, but
they reach it anyway using the dom event mechanism.

Now from my pov this seems pretty nice, but I still wonder about some
things. First whether there are any scenarios, ie. other events that need
testing for the KListBox case? Since I check for QScrollView derivatives,
are there any widgets that get affected by the new code? And finally
mozilla also seems to handle events on the options in the select, I think
khtml should do that too, but I have not started on implementing that yet.

>> > I do have some code for more efficient widget painting in my tree - to
>> > get textarea to work, but it's basically the same mechanism. Not sure
>> it
>> > would help.
>>
>> I also noticed some problems with textarea painting, but can't remember
>> the
>> exact bug report number. Anyway, would be great if you fixed that :)
>
> I still have some oddities of event handling and refresh upon scrolling
> with
> those, but I'll post the code soon... maybe someone will be able to help
> resolving the remaining issues.

I think there is a good chance of that :)
Cheers,

Rob.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix71427-2.diff
Type: text/x-diff
Size: 3679 bytes
Desc: not available
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20041002/bcdddbeb/attachment.diff>