Bug 142722 - setting size value of object changes font size
Summary: setting size value of object changes font size
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: khtml ecma (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-03-09 11:09 UTC by Rob Kaper
Modified: 2012-06-17 17:24 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
Testcase (755 bytes, text/html)
2007-03-09 11:11 UTC, Rob Kaper 		Details
new testcase (912 bytes, text/html)
2012-05-03 13:17 UTC, Andrea Iacovitti 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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