Bug 1014586 - "Add To Dashboard" button size is too small on some displays
Summary: "Add To Dashboard" button size is too small on some displays
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Operations Network
Classification: JBoss
Component: UI
Version: JON 3.2
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ER04
: JON 3.2.0
Assignee: Mike Thompson
QA Contact: Mike Foley
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-10-02 11:19 UTC by Libor Zoubek
Modified: 2015-11-02 00:43 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-01-02 20:43:13 UTC
Type: Bug

Attachments (Terms of Use)
Metrics Screen snapshot (686.33 KB, image/jpeg)
2013-10-02 14:44 UTC, Mike Thompson 		no flags Details
invalid button size (192.66 KB, image/jpeg)
2013-10-02 15:11 UTC, Libor Zoubek 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Description Libor Zoubek 2013-10-02 11:19:46 UTC 
Description of problem:"Add To Dashboard" button size is too small, text does not fit in


Version-Release number of selected component (if applicable):
JON 3.2.ER2, Firefox 24, Chromium 29

How reproducible: always
Comment 1 Mike Thompson 2013-10-02 14:43:31 UTC 
What resolution are you testing? I have attached a 1280x1024 screenshot.
Please attach screenshot as well. Also, a pixel ruler would help as well to quantify what you are seeing on the screen.
Comment 2 Mike Thompson 2013-10-02 14:44:30 UTC 
Created attachment 806526 [details]
Metrics Screen snapshot
Comment 3 Libor Zoubek 2013-10-02 15:11:11 UTC 
Created attachment 806531 [details]
invalid button size

I am on 1920x1080. I've just doublechecked with Filip (Fedora 16, same resolution) and it's ok on his environment. So it must be something on my side. I wonder why it is the only button, everything else within jon looks correctly.
Comment 4 Mike Thompson 2013-10-02 15:20:01 UTC 
Maybe its a fonts/zoom issue. In FF, if you View --> Zoom --> Reset.
Does the problem still exist?
Comment 5 Mike Thompson 2013-10-02 16:56:00 UTC 
So it looks like smartGWT is not sizing the button correctly to fit the width because it is using a default width (not specified). I'm making the button width fixed at 120 px (it was automatically sized at 101px) as to give it room to show all the characters in the label.

The only side effect is that possible i18n would be greater than the 120px and would truncate the text (much like it is doing right now).
Comment 6 Mike Thompson 2013-10-03 17:17:32 UTC 
Committed to master with commit id: 6da584e
Note You need to log in before you can comment on or make changes to this bug.