Bug 1014586

Summary: "Add To Dashboard" button size is too small on some displays
Product: [JBoss] JBoss Operations Network Reporter: Libor Zoubek <lzoubek>
Component: UIAssignee: Mike Thompson <mithomps>
Status: CLOSED CURRENTRELEASE QA Contact: Mike Foley <mfoley>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: JON 3.2CC: lzoubek, mithomps, theute
Target Milestone: ER04   
Target Release: JON 3.2.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-01-02 20:43:13 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
Metrics Screen snapshot
none
invalid button size none

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