Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 350784 - Texts cut off when Browser zoomed
Summary: Texts cut off when Browser zoomed
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: 1.5   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 1.5 M1   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard: sr141
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-30 04:48 EDT by Ralf Sternberg CLA
Modified: 2011-08-25 08:54 EDT (History)
1 user (show)

See Also:

Attachments
Screenshot (101.21 KB, image/png)
2011-06-30 04:51 EDT, Ralf Sternberg CLA 		no flags Details
fix for 1.4.1 (1.49 KB, patch)
2011-07-22 06:10 EDT, Tim Buschtoens CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ralf Sternberg CLA 2011-06-30 04:48:56 EDT 
Reproducible at least with Chromium on Linux.
* Go to http://rap.eclipsesource.com/rapdemo/examples
* Zoom browser using Ctrl +
* Reload
-> Note that the texts in some navigation buttons is cut off
Comment 1 Ralf Sternberg CLA 2011-06-30 04:51:41 EDT 
Created attachment 198881 [details]
Screenshot

Attached screenshot that shows the issue
Comment 2 Tim Buschtoens CLA 2011-06-30 05:04:15 EDT 
Possibly the same issue or similar to Bug 340841.
Might also be connected to Bug 292189.
Comment 3 Ralf Sternberg CLA 2011-06-30 05:07:18 EDT 
The problem was reproducible even without browser zoom when we had a theming addition that changed the bold font to normal on hover. Strangle, it only appeared when pressing the TreeViewer button in the demo and hovering over the navigation buttons after this.
Comment 4 Tim Buschtoens CLA 2011-06-30 05:10:45 EDT 
There might be an explanation for that second scenario: TreeRow uses qx.ui.basic.Label._getMeasureNode(), just as MultiCellWidget/Button does. Perhaps something is done to that node that breaks textsize measuring.
Comment 5 Tim Buschtoens CLA 2011-07-05 12:38:06 EDT 
Fixed in CVS HEAD.

As suspected, these were two issues. One is the problem with textsize measurement not working anymore after a Tree appears. That was fixed by resetting the measurement Node properly. The other issue is about zooming an application. Thats actually the same bug as Bug 340841, and could be fixed in the same manner. 

What remains is the issue of zooming on runtime, i.e. after the application is already loaded. For that scenario i opend Bug 351218 - Improve support for browser-zooming on runtime.
Comment 6 Tim Buschtoens CLA 2011-07-22 06:10:48 EDT 
Created attachment 200167 [details]
fix for 1.4.1
Comment 7 Tim Buschtoens CLA 2011-08-22 09:39:43 EDT 
Commited patch(attachment 200167 [details]) to CVS v14_Maintenance branch.
Comment 8 Tim Buschtoens CLA 2011-08-25 08:54:12 EDT 
Commited patch to v14_Tree_Table_Merge branch