Bug 51544 - [Qt] Zoom checkboxes and radio buttons according to zoom factor (Mac OS X && Maemo 5)
Summary: [Qt] Zoom checkboxes and radio buttons according to zoom factor (Mac OS X && ...
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P3 Normal
Assignee: Nobody
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks:
 
Reported: 2010-12-23 10:47 PST by Martin Storsjö
Modified: 2014-02-03 03:17 PST (History)
2 users (show)

See Also:


Attachments
Sample application, demonstrating the issue (570 bytes, application/x-gzip)
2010-12-23 10:47 PST, Martin Storsjö
no flags Details
Screenshot of the sample application on Qt 4.6.3 on Maemo5 (31.29 KB, image/png)
2010-12-23 10:49 PST, Martin Storsjö
no flags Details
Screenshot of the sample application on Qt 4.6.3 on OS X (42.74 KB, image/png)
2010-12-23 10:49 PST, Martin Storsjö
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Storsjö 2010-12-23 10:47:21 PST
Created attachment 77347 [details]
Sample application, demonstrating the issue

A simple form, displayed in QtWebKit with a zoom factor applied, doesn't zoom all UI components as much. On Mac OS X, checkboxes, radio buttons and the submit button aren't zoomed at all. On Maemo 5, checkboxes and radio buttons aren't zoomed.

Especially on Maemo 5, being able to specify a zoom factor is very important in order to get a usable UI.

The attached screenshots show what the attached example application looks like on Mac OS X and Maemo5, with Qt 4.6.3. Qt 4.7 on Maemo5 looks marginally different - the radio buttons and checkboxes still are unaffected by the zoom.

This bug was originally reported at http://bugreports.qt.nokia.com/browse/QTWEBKIT-231.
Comment 1 Martin Storsjö 2010-12-23 10:49:10 PST
Created attachment 77348 [details]
Screenshot of the sample application on Qt 4.6.3 on Maemo5
Comment 2 Martin Storsjö 2010-12-23 10:49:47 PST
Created attachment 77350 [details]
Screenshot of the sample application on Qt 4.6.3 on OS X
Comment 3 Sriram 2011-01-26 09:26:45 PST
sriram.seetharam@nokia.com is looking at this error. 
No access rights to change the status to INSTUDY
Comment 4 Benjamin Poulain 2011-01-30 05:34:14 PST
My guess is the bug is invalid here because it is the style that cannot draw bigger control element. In which case that should be reported to Qt and Maemo to fix that in the platform styles.

Sriram, do you look into that? Did you get the rights on bugzilla?
Comment 5 Sriram 2011-01-31 06:38:24 PST
Hi Benjamin,
You are right, this looks like an issue with styles not being able to scale the checkbox and radio buttons. 

As an experiment, I added an <img> image to the html. Every time I changed the zoom factor, I was able to see the image scale accordingly but not the checkbox or radio buttons.

Will check with Henry Haverinen about how to get this resolved on platform side.

Still no rights on webkit bugzilla.
Comment 6 Benjamin Poulain 2011-01-31 08:02:40 PST
(In reply to comment #5)
> You are right, this looks like an issue with styles not being able to scale the checkbox and radio buttons. 
> 
> As an experiment, I added an <img> image to the html. Every time I changed the zoom factor, I was able to see the image scale accordingly but not the checkbox or radio buttons.
> 
> Will check with Henry Haverinen about how to get this resolved on platform side.

Please create a reduction using Qt only. Then create two bug reports on http://bugreports.qt.nokia.com/ , one for Mac, one for Maemo 5.

Post the links to those two reports here and I will close this bug.
Comment 7 Sriram 2011-01-31 10:29:45 PST
(In reply to comment #6)
> (In reply to comment #5)
> > You are right, this looks like an issue with styles not being able to scale the checkbox and radio buttons. 
> > 
> > As an experiment, I added an <img> image to the html. Every time I changed the zoom factor, I was able to see the image scale accordingly but not the checkbox or radio buttons.
> > 
> > Will check with Henry Haverinen about how to get this resolved on platform side.
> 
> Please create a reduction using Qt only. Then create two bug reports on http://bugreports.qt.nokia.com/ , one for Mac, one for Maemo 5.
> 
> Post the links to those two reports here and I will close this bug.


This bug was originally in qt but was closed there and moved to webkit. (http://bugreports.qt.nokia.com/browse/QTWEBKIT-231). Are you suggesting we create a new error in qt.
Could you please clarify
Comment 8 Benjamin Poulain 2011-01-31 10:47:28 PST
(In reply to comment #7)
> > Please create a reduction using Qt only. Then create two bug reports on http://bugreports.qt.nokia.com/ , one for Mac, one for Maemo 5.
> > 
> > Post the links to those two reports here and I will close this bug.
> 
> 
> This bug was originally in qt but was closed there and moved to webkit. (http://bugreports.qt.nokia.com/browse/QTWEBKIT-231). Are you suggesting we create a new error in qt.
> Could you please clarify

The bug was closed in Qt because the lowest component it could be narrowed to is WebKit.

We think the problem is in QStyle. So make a reduction using only QWidget and QStyle, and you can report it to Qt. Then the lowest component we can narrow the bug to is Qt...

With the reduction, the Qt devs can work directly to solve the bug. Fabien will probably fix Mac and Harald will fix Maemo. Without reduction, they have to go through a big pile of code before they can start working...
Comment 9 Sriram 2011-02-09 08:52:33 PST
Not actively working on this bug anymore. Returning bug back to bug list.
Comment 10 Martin Storsjö 2011-02-09 10:06:46 PST
Was a reduced Qt bug ever reported? If someone knows off-hand what QtWebKit does in order to scale the widgets, I could try to create that reduced bug report, so that this bug report doesn't stall for another 5 months (as it initially did in the Qt Jira database).
Comment 11 Jocelyn Turcotte 2014-02-03 03:17:05 PST
=== Bulk closing of Qt bugs ===

If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary.

If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines.