Bug 392013 - Orion banner is too wide
Summary: Orion banner is too wide
Status: RESOLVED FIXED
Alias: None
Product: Orion (Archived)
Classification: ECD
Component: Client (show other bugs)
Version: 1.0   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 2.0 M1   Edit
Assignee: John Arthorne CLA
QA Contact:
URL:
Whiteboard:
Keywords: helpwanted
Depends on:
Blocks:
 
Reported: 2012-10-15 18:17 EDT by Mark Macdonald CLA
Modified: 2012-11-30 15:05 EST (History)
3 users (show)

See Also:


Attachments
screenshot showing how wide it is (128.45 KB, image/png)
2012-10-15 18:17 EDT, Mark Macdonald CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Macdonald CLA 2012-10-15 18:17:02 EDT
Created attachment 222367 [details]
screenshot showing how wide it is

While taking some screenshots, I noticed that the Orion banner expects the browser to be approx 900px wide or wider. If you make it any smaller, the banner truncates. I actually had to doctor some of the screenshots to make them fit inside 800px.

This is wasteful since there's a lot of empty horizontal space available. The empty space should collapse first.
Comment 1 Mark Macdonald CLA 2012-10-15 18:18:13 EDT
Looks like ide.css is setting 'min-width: 900px' on the <body>, not sure why. Removing that allows the banner to collapse more nicely...
Comment 2 John Arthorne CLA 2012-10-15 22:17:35 EDT
It used to do this in the not too distant past. For screen shots I wouldn't worry about showing everything, just crop down to the parts you are talking about.
Comment 3 Susan McCourt CLA 2012-10-16 00:39:24 EDT
I definitely remember that the banner used to scale down indefinitely, because we had even discussed different strategies for wrapping the data.   I don't recall us ever deciding on a fixed/min page width, and 900px is weird because I would expect to support an iPad in portrait mode (768) with minimally a media query, or at least to choose this if we need a minimum.

Out of curiosity, I checked the history and the change is pinned to this commit, so presumably Anton and/or McQ will recall...

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=e19cd156ec7d113ca0e90498c575df4d52a586bc
Comment 4 John Arthorne CLA 2012-11-30 15:03:46 EST
Taking this in preparation for m1 screenshots next week.
Comment 5 John Arthorne CLA 2012-11-30 15:05:16 EST
I have changed it to 700px. When I tried removing it entirely, strange things started to happen when the search bar ran up against the page buttons. This compromise at least support (old) iPad portrait mode and will be a bit nicer for screen shots:

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=d3577620b14cd2bc86f3d7a4d9fcfe7d096e01e7