Problem/Motivation

One thing that is bugging me every time I install Drupal 8 (thanks for the shiny new design! loving it):
the progress bar goes much too far to the right:

Installer progress bar before

This is probably just an oversight. The right padding should at least match the padding the page has on the left. I noticed that not only the progress bar, but everything is too far to the right.

Proposed resolution

Just match the right padding to the left padding.
I gave body.install-page #page a right-padding of 3.5 em and made the entire container a bit narrower (740px instead of 770) so it remains about the same width.
As the progress bar still does not look right with this, I gave it a further 2em margin to the right.

Installer progress bar after
So that's my initial patch. Patch uploaded in first comment, as without posting the issue first, there is no issue number.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

eigentor’s picture

Status: Active » Needs review
Issue tags: +CSS, +installer
FileSize
611 bytes

Status: Needs review » Needs work

The last submitted patch, drupalcore-fixinstallercss-2094445.patch, failed testing.

LewisNyman’s picture

Issue tags: +CSS novice
LewisNyman’s picture

Title: Fix installer right padding - make it look balanced » Fix installer padding
FileSize
598.1 KB

I also noticed this vertical spacing issue when the page takes up the height of the viewport. There should be some blue spacing before and after the white container. It would be good to fix it here.

Screen Shot 2013-09-21 at 15.37.51.png

Outi’s picture

Status: Needs work » Needs review
FileSize
1002 bytes

The patch should set the margins and paddings in a symmetric way and add a margin on the top and the bottom of the page.

pameeela’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
396.25 KB

Looks good.

Screenshot:

installer-padding.png

LewisNyman’s picture

Status: Reviewed & tested by the community » Needs work
FileSize
1.22 MB

Ah, just noticed that the container is no longer centered...

Screen Shot 2013-09-22 at 15.43.36.png

LewisNyman’s picture

Title: Fix installer padding » Fix installer right padding - make it look balanced

After some discussion and head bashing, I think it's best to deal with the vertical padding separately and simply fix the right padding issue in this patch.

Outi’s picture

I removed the lines I added or changed on the last patch while trying to add some space between the top and the bottom of the white container and the navigator window.

Outi’s picture

Status: Needs work » Needs review
LewisNyman’s picture

Status: Needs review » Needs work
+++ b/core/themes/seven/install-page.css
@@ -50,7 +51,6 @@ body.install-page {
     padding: 1em 0 1em 2em;

@@ -62,6 +62,7 @@ body.install-page {
     padding-left: 50px; /* LTR */

We also need to change the RTL styling now that we've changed the LTR

Outi’s picture

Status: Needs work » Needs review
FileSize
869 bytes
Outi’s picture

LewisNyman’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
1013.72 KB
1.01 MB
1.01 MB

Looks good! Here are some before/after screenshots:

2094445-before.png

2094445-after.png

eigentor’s picture

O.K. this fixes the vertical margin issue, but leaves out the stuff why I originally created the issue...
-- edit --
Or maybe not.
I'll have a look at the patch first.

eigentor’s picture

O.K. my bad, fixes the right padding issue.
Can't see any change on vertical margin, though.
http://screencast.com/t/CAQ8JwNlFTBh

Did this get lost, or would it be another issue?
Keeping things incremental would be in favor of creating another issue for it.

LewisNyman’s picture

Yeah let's deal with it in another issue, I think the best action is to re-open #2032895: Follow-up use better technique for vertical centering

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.