Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Gábor Hojtsy’s picture

The reason it is on the right is that it is less important then the actual forms and other content displayed IMHO. I agree that the Druplicon looking right looks bad at the same time :)

Bojhan’s picture

Moving the whole block ? What about just moving the direction of Druplicons face? Still find it somewhat scary to have an alien looking over me installing :P

timmillwood’s picture

Assigned: Unassigned » timmillwood
Status: Active » Needs review
FileSize
835 bytes

Here is a patch that moves the sidebar_first region to the left instead of the right in maintenance.

catch’s picture

Druplicon's face always faces right, I'm not keen on flipping the image. Can we have a screenshot of #3?

Dries’s picture

Flipping the image is not recommended because it could dilute the brand. We always want Druplicon to look in the same direction. I think Tim's patch in #3 is the way to go, however, I haven't taken it for a spin yet.

timmillwood’s picture

Screenshot of #3.

timmillwood’s picture

Another screenshot of #3 to give similar context to Dries' screenshot.

chx’s picture

We had it this way in six. So we want to change it back? I kinda support it -- but then again maybe not because of #1. What about just making an animated Druplicon turning around :D

catch’s picture

While I think #1 is valid, the counter arguments are that the left column is the same as left navigation on a lot of websites - which is rarely more important than the main content, but very common, also this means the main content is put further towards the center of the page, which isn't a bad thing at all.

timmillwood’s picture

Maybe we put Druplicon on the left and the task list on the right?

Damien Tournoud’s picture

I suggest we remove the Druplicon from here. It simply doesn't fit.

Bojhan’s picture

I should have added a a smiley to my comment, obviously we can't move a logo around (failed at being funny)

Anyway, seems rather strange to move a whole sidebar over the direction of a face (I agree if this is a highly visual decision) but it seems rather easy. The reason its on the right as clearly explained in #1 is because it should not hold the same visual importance as other stuff we normally put on the right, especially if we continue this pattern into Drupal core (we need to be aware of possible mulitstep forms in core - that would have to be exposed in Seven on the right).

So lets get the wordmark in, I think that was the original goal anyway.

@chx That seems like the best option :D

catch’s picture

We can't get the wordmark in the CVS checkout, because that would involve it being GPL-ed. However if it's in the tarball (which is on the cards afaik, but probably needs an infrastructure issue opened), then does this even matter?

Damien Tournoud’s picture

+1 for putting the wordmark in there. Is there any reason not to release it in GPL? After all, you cannot reliably protect a wordmark anyway.

catch’s picture

There's a whole series of posts about the wordmark in #547068: D7UX: use Seven for installation / updates

timmillwood’s picture

Assigned: timmillwood » Unassigned

I guess the advantage of putting the wordmark in is that new users to Drupal will see a similarity between Drupal.org and the product they are installing. Where as Druplicon will be somewhat unknown to new Drupal users.

Although it will be sad not to see Druplicon.

yoroy’s picture

chx’s picture

Are we going to ban jeans and t-shirts from drupalcons and mandate black tie?

Gábor Hojtsy’s picture

@chx: the jeans story is discussed in #605710: Decide on if and if so, how to implement the Drupal wordmark in core as pointed out by @yoroy.

rmontero’s picture

FileSize
12.83 KB

The Druplicon is such a cool logo that it looks great either way.

One thought though, why not use the one that's looking straight at you?

kika’s picture

1. Let's drop the idea flipping Druplicon icon. This will not happen (brand dilute).

2. It miiight be considered putting completed steps block to the left. It's in the left in D6. When that decision was made -- was it part of Mark's design?

I am personally indifferent for 2. It does it's work in both cases.

Gábor Hojtsy’s picture

@kika: Mark's latest design is posted at #605710: Decide on if and if so, how to implement the Drupal wordmark in core (the evolution which you can track at #547068: D7UX: use Seven for installation / updates - originally without the wordmark / Druplicon).

Dries’s picture

Committed this patch for now, but keeping the status at 'needs review' so we can continue the discussion.

David_Rothstein’s picture

The idea here is good, but the implementation that went in seems like it needs work - why are we putting the task list front and center like that?

The attached patch moves it to the left where it is more out of the way. Note: I have only tested it on Firefox, but seems pretty straightforward.

Before patch:
druplicon-before.png

After patch:
druplicon-after.png

Dries’s picture

That looks better indeed. Now the testbot is back, let's wait for test results.

jbrown’s picture

How could a CSS-only patch cause a test to fail?

eigentor’s picture

When I check out drupal, I do not see the outline around the main region, so it looks very disoriented.

The left-aligning looks good on the screenshot, but on a large screen this makes up for a very weird use of real-estate:

Am not a big fan at all of having the list on the left, it felt much more natural on the right side.
Sure the druplicon facing away (with horror of all these amateurs trying to install his beloved system :P )
is not good, for me the information-presentation would be more important.

But I see... Ack ack classical bikeshed-thread, and only because we got our hot new wordmark protected...

So my vote for keeping it centered and getting me to see the outline.
edit -- The outline is visible on some pages, but not on all, especially the first one --

eigentor’s picture

FileSize
9.97 KB

Image with border for left-aligned version so one can see the limits of the screen.

eigentor’s picture

FileSize
10.3 KB

once more, sorry, one cannot edit attachments...

David_Rothstein’s picture

@eigentor, either you didn't clear your CSS cache when you made those screenshots, or something is wrong with the patch on certain browsers.... but your screenshots are missing the key improvement, which is that the headline (e.g., "Select an installation profile") is supposed to align with the main page content, not the task list. That is the key feature shown in #24.

See the first attached screenshot for a particularly bad example (from current HEAD) of what happens when it is not aligned. We really need to fix this :)

I guess you are right that centering this on the page is better than left-aligning it. Ideally what we want is to center the content but have the task list jut out to the left of it, but I'm not sure how simple that is. Worse case, we can center the whole thing (as in D6 Garland - see second attached screenshot), but to my eye that looks somewhat lopsided because the space on the left and right side is not equal.

aspilicious’s picture

Can't we close this? I think everybody loves the current situation...

yoroy’s picture

Status: Needs review » Needs work

I'm certainly not happy with the current situation. Druplicon on the right was better, since in general people focus on the left side of the screen, which previously showed them the task at hand but now shows them the static part of the info. Unnecessary focus on the branding instead of the interaction. Also: it's too big and not well-aligned, should align with the text in the list, not the list markers.

eigentor’s picture

I'm with yoroy on this.
If we have druplicon on the left hand, let's at least align it correctly or be consequent to have a border around the content everywhere and not just somewhere.

yoroy’s picture

We don't put outlines around the main interaction of a page. It's a guideline we should probably communicate better :)

aspilicious’s picture

FileSize
32.36 KB

Something like this? See screenshot (I know we can't flip icon, just a mockup)
I don't know... druplicon seems so "alone" then.

About the border, I think we agreed *NOT* using a border in seven theme when it's not necessary, border is used to group regions in seven theme, on some installer pages we only have one set of data so it's not necessary to put an extra border around that set.
Personally I dislike the border very much.
But I learned that mostly others have different opinions.

yoroy’s picture

Nah, there's no reason to make this a 3 column layout.
My preference would be to revert to the initial situation: druplicon and list on the right, then make druplicon smaller and better aligned.
This is the installer, people haven't even succeeded in getting Drupal up and running, so we should get out of the way as much as possible. Right now, druplicon and the list are (more) in the way than in the previous situation.

But we could start with the alignment and sizing first.

eigentor’s picture

Ah, this entire thing brings the discussion up again, why the druplicon is facing away from the content and it is not good him showing such lack of interest. Reversing the Druplicon is even worse and won't get in. Idea for that, yoroy?
I think I can live without outlines :P

yoroy’s picture

That's why I'm not pushing too hard to revert this, though I think with a smaller size, this problem will become smaller as well.
I'd like to see a patch that improves alignment with a smaller druplicon.

David_Rothstein’s picture

Title: Druplicon should look inward on the installer » Fix issues with Druplicon looking inward on the installer
Category: task » bug

We definitely still need to fix the alignment problems that were introduced here - see #31 for an explanation. I think that screenshot qualifies as a bug that would go into the "major" category, if one existed....

The patch in #24 does that and also has the effect of putting the user's focus back on the main page content, which is where it belongs. A smaller Druplicon might not hurt either :) Otherwise, seems like we just need to fix the issue @eigentor brought up (entire page content should be more centered so it looks OK on big monitors) and we are in good shape.

sun’s picture

Version: 7.x-dev » 8.x-dev
Status: Needs work » Needs review
FileSize
19.42 KB
2.45 KB

I'd suggest to resolve this issue this way:

install-sidebar.png

Bojhan’s picture

I think the actual fix here, is to use the Drupal wordmark.

chx’s picture

We discusses that previously, it's a legal no-go, the wordmark is not GPL and Drupal is.

Bojhan’s picture

I'd suggest to just fix the alignment and make it smaller then, as suggested in #39. I feel little for the druplicon in the header (to me it looks totally wierd aesthetically) and moving the wizard (although its possibly sun's huge screen but the steps now feel disconnected).

jhedstrom’s picture

Issue summary: View changes
Status: Needs review » Needs work
Issue tags: +Needs reroll

Patch needs a reroll, if this is still being pursued.

cweagans’s picture

Issue summary: View changes
LewisNyman’s picture

Status: Needs work » Closed (fixed)

This is fixed right now.