Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 369683 - Problem with styling on new plug-ins, repo and commit pages
Summary: Problem with styling on new plug-ins, repo and commit pages
Status: RESOLVED FIXED
Alias: None
Product: Orion
Classification: ECD
Component: Client (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 0.4 RC1   Edit
Assignee: Anton McConville CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 369585 (view as bug list)
Depends on:
Blocks:
 
Reported: 2012-01-25 10:49 EST by Szymon Brandys CLA
Modified: 2012-02-07 06:58 EST (History)
1 user (show)

See Also:


Attachments
The problem on plug-ins page (25.56 KB, image/png)
2012-01-25 10:50 EST, Szymon Brandys CLA
no flags Details
The problem on git repo page (44.43 KB, image/png)
2012-01-25 10:51 EST, Szymon Brandys CLA
no flags Details
Problem with longer titles and descriptions (38.96 KB, image/png)
2012-01-26 06:13 EST, Szymon Brandys CLA
no flags Details
js/css files (14.79 KB, application/x-zip-compressed)
2012-01-26 22:50 EST, Anton McConville CLA
no flags Details
screenshot of commit page with latest code (41.58 KB, image/png)
2012-01-27 12:39 EST, Susan McCourt CLA
no flags Details
designer mockup of repo page (97.07 KB, image/png)
2012-01-28 16:12 EST, Susan McCourt CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Szymon Brandys CLA 2012-01-25 10:49:14 EST
I'm attaching a screenshot showing the problem. When the plug-in title is very long or we just make the browser window smaller, the page does not behave well. The same problem on repo page where the problem is more visible.
Comment 1 Szymon Brandys CLA 2012-01-25 10:50:05 EST
Created attachment 210057 [details]
The problem on plug-ins page
Comment 2 Szymon Brandys CLA 2012-01-25 10:51:27 EST
Created attachment 210058 [details]
The problem on git repo page
Comment 3 Susan McCourt CLA 2012-01-25 13:48:25 EST
*** Bug 369585 has been marked as a duplicate of this bug. ***
Comment 4 Anton McConville CLA 2012-01-25 22:56:23 EST
Please push this fix ...

https://github.com/antonmc/orion.client/commit/3156bb30e9f514ee3d80c433af71479bc1f91871

Anton
Comment 5 Szymon Brandys CLA 2012-01-26 06:13:42 EST
Created attachment 210105 [details]
Problem with longer titles and descriptions

It looks better, but there is still problem for longer titles and descriptions. I think we should modify our title, description and action area classes too. I'm not sure what the good practice is, but I think that the content should be centered instead of left-aligned.
Comment 6 Anton McConville CLA 2012-01-26 09:58:17 EST
Ah - ok, I we now need to make some more width settings on the description area.

I'm not so sure we should center everything, it wouldn't always solve the problem with large amounts of empty space between the blocks.

I'd personally be happier tightening up the content - making some size decisions about it and cultivating some predictability in this layout. Some fields should wrap and grow vertically rather than grow without bounds horizontally.

I can see that I'd need to tweak this some more - and I can also see that in the case of a really wide screen, we'd be left with some blank space at one side - however, the alternative would be to have distributed wide gaps between the blocks - in other words this page has empty space however we look at it - I favor the former approach.

We also need to make good calls on the width so that it works well on a tablet and phone - though maybe we should tailor a UI for those.
Comment 7 Susan McCourt CLA 2012-01-26 22:00:36 EST
(In reply to comment #6)
> 
> We also need to make good calls on the width so that it works well on a tablet
> and phone - though maybe we should tailor a UI for those.

fyi...we have some tablet-ish media queries in ide.css so we could set standard width type css there.
Comment 8 Anton McConville CLA 2012-01-26 22:48:47 EST
I made some changes, but for some reason, was unable to push them to github from the status page.

I've attached some files.

So - if you look at the page - there are some lines with short descriptions on the left, and long lines of buttons on the right - and some with long descriptions on the left and short lines of buttons on the right. 

I'm not able to just make the description contents wrap. I think I'd need to work with the css, or maybe rethink some parts.

So I needed to set some width restrictions on the descriptions as well as the page. And so this becomes a balancing act. I think I have a good balance - but that is testing only on my screen. You'll need to try on yours and may need to play with the numbers a bit.

I would say that, although it works now, I'm uneasy with mixing these two kinds of rows on the same page. This also resonates with my personal taste of seeing the content ( and page ) simplified to use a left navigation to jump to the different sections.

Sorry I wasn't able to push this ... not sure what the problem is. The change is small in gitRepositoryExplorer.js - search for tag-description in there.


tag-description is a new css class - and so that is also in the settings.css file.

I needed to make some minor changes to settings.js too - because it has slightly different width requirements on the same css.
Comment 9 Anton McConville CLA 2012-01-26 22:50:03 EST
Created attachment 210166 [details]
js/css files
Comment 10 Susan McCourt CLA 2012-01-27 12:39:04 EST
Created attachment 210206 [details]
screenshot of commit page with latest code

The latest changes to "stretch" are causing the commit details page to have tiny code boxes.  I think we need to stop using stretch on that page.
Comment 11 Susan McCourt CLA 2012-01-28 16:12:20 EST
Created attachment 210234 [details]
designer mockup of repo page

Here's a designer mockup of the repo page, which I quite like.
To note:

- for sections where everything underneath is the same model icon, use the model icon in the header.  Helps declutter.
- use the gray section heading bars and max out the width at a certain point, with the bars stopping where the content is.
- we talked about centering but concluded that this doesn't look that great unless you have a "plated" fixed width page where even the banner stops.  We aren't using this style, so the approach is to simply max out and then leave white space on right.
- note the use of a single pixel border on the bottom of each item rather than shaded bars.  With the "bar" look of the section header, using bars for content was getting busy.
- fonts are smaller
Comment 12 Susan McCourt CLA 2012-01-31 12:18:24 EST
(In reply to comment #10)
> Created attachment 210206 [details]
> screenshot of commit page with latest code
> 
> The latest changes to "stretch" are causing the commit details page to have
> tiny code boxes.  I think we need to stop using stretch on that page.

this has been fixed in bug 370175.
Comment 13 Susan McCourt CLA 2012-02-02 12:52:48 EST
Heading on the repo pages are looking good.
I think (?) we've fixed the scroll bar/layout issues for different pages?
 I think the only thing left here for the repo/commit page is to get the fonts smaller.  Then we can close this?

Anton, if you think this is not part of the shared css then let Szymon know.
Comment 14 Szymon Brandys CLA 2012-02-07 06:58:30 EST
I fixed font size on repo pages with http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=107e7a0e423788ac34a82337414e9d0d5126f250. Fonts are not part of the shared css now and are unified with git-log page fonts.