Bug 142722

Summary: setting size value of object changes font size
Product: [Applications] konqueror Reporter: Rob Kaper <webmaster>
Component: khtml ecmaAssignee: Konqueror Developers <konq-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: aiacovitti
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: Testcase
new testcase

Description Rob Kaper 2007-03-09 11:09:07 UTC 
Version:           3.5.4 (using KDE 3.5.4, compiled sources)
Compiler:          gcc version 3.4.6
OS:                Linux (i686) release 2.6.17.13

When changing the size value of an object such as a <select> combobox in Javascript, Konqueror changes the font size of the options, but only when there is no size attribute specified for the <select>.

(Testcase follows.)
Comment 1 Rob Kaper 2007-03-09 11:11:32 UTC 
Created attachment 19917 [details]
Testcase
Comment 2 Harri Porten 2008-03-16 21:55:07 UTC 
Confirmed in 4.0 branch. Easier visible after a View|Enlarge Font btw.
Comment 3 Maksim Orlovich 2008-03-17 05:08:22 UTC 
There is a patch from vtokarev for this waiting for review @ kfm-devel
Comment 4 Germain Garand 2008-03-17 17:09:45 UTC 
Oops, right, I actually intended to reply and forgot...thanks for the reminder.
Comment 5 Germain Garand 2008-03-22 21:31:15 UTC 
SVN commit 788917 by ggarand:

apply patch by Slava Tokarev
fixing unwanted changes of form widgets font size

CCMAIL: tsjoker@gmail.com
BUG: 142722


 M  +2 -0      render_replaced.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=788917
Comment 6 Andrea Iacovitti 2012-05-03 13:17:13 UTC 
Created attachment 70828 [details]
new testcase
Comment 7 Andrea Iacovitti 2012-06-17 17:24:00 UTC 
Git commit 95ccc85ae312453bf68c0fb48025c5dab21faa81 by Andrea Iacovitti.
Committed on 17/06/2012 at 19:04.
Pushed by aiacovitti into branch 'KDE/4.8'.

Do not call setStyle() in RenderWidget::updateFromElement() for all type
of widgets. It is needed for select elements when switching internal
widget type.

An evident side effect is the unwanted scroll of textarea's content on
focus out, caused in turn by setAlignment() in RenderTextArea::setStyle().
Related: bug 302052

M  +5    -1    khtml/rendering/render_form.cpp
M  +0    -4    khtml/rendering/render_replaced.cpp

http://commits.kde.org/kdelibs/95ccc85ae312453bf68c0fb48025c5dab21faa81