On the new seven admin theme we have a link at the top with the text "Back to the live site".

This is wrong and misleading - Drupal does not have a front/back end and we should not pretend that it does.

CommentFileSizeAuthor
#12 truth-546694-12.patch617 bytespwolanin
#1 truth-546694-1.patch618 bytespwolanin

Comments

pwolanin’s picture

Priority: Normal » Critical
Status: Active » Needs review
StatusFileSize
new618 bytes

Here's some text that's at least true.

dries’s picture

Priority: Critical » Normal

Not critical, but still a good patch.

markboulton’s picture

I actually disagree. It should say 'live site', or 'site' because if you're seeing that link, then you have JS disabled (this was the intention of the 'non-overlay' version). If you don't have an overlay, then there is nothing to close to return to the page. For example, if you are editing in place (not your homepage), you edit a block, how do you go back? What do you go back to?

We know that technically, Drupal does not have a back/front end. We're not trying to impose one either. However, one of the findings of our early research was that some users, especially new users, find the mental model of not having a separate admin 'area' very confusing. This lead to one of our guiding UX principles: As a user, I should know where I am.

kaakuu’s picture

+1 for Back to your front page

It has been hell lot of confusing when our group started testing Drupal dev 7 version.
I have already posted some minor bugs from those.

'Back to the live site' was creating confusion with "site offline" AS IF the admin was in offline while he has been setting various admin configs and has to click "back to live site" to change from offline to live.

Though this will not be a problem when the decision to toggle this off and use Admin_menu instead has been taken, it is good to see that this confusion is being removed in favor of "Back to your front page"

pwolanin’s picture

@markboulton the current text is false, which is not helpful for the users getting a correct mental model of how Drupal works.

dries’s picture

I agree that "live" is inaccurate. We could change "Back to the live site" to "Back to the site" (i.e. drop the "live").

pwolanin’s picture

@Dries - the admin page is part of the site right? We should indicate actually what the link does - it goes to the front page. Anything else is (imho) damaging to helping users build their mental model of Drupal.

kaakuu’s picture

@pwolanin +1

andypost’s picture

+1 for front page

David_Rothstein’s picture

We also discussed this at length in #484860: Initial D7UX admin theme but it did not make it into the final patch. So it's good to see this followup :)

Given that this link is visible to all users, I think we should avoid using the word "your" here. Maybe "Back to the front page"?

Note also that there are a number of other issues with this link (for example, it shows on the front page of the site, where clicking on it takes you nowhere new). See http://drupal.org/node/484860#comment-1841424 and followup comments for more discussion. We can certainly do a quick fix here and not deal with all the issues at once, but it might be worth keeping in mind that other, more well-tested implementations (breadcrumbs, site logo, etc) could potentially be cleaner than this one-off text link and would automatically solve some of the problems with it.

pwolanin’s picture

Issue tags: +Quick fix

I think a quick fix is a good idea :)

pwolanin’s picture

StatusFileSize
new617 bytes
David_Rothstein’s picture

Status: Needs review » Reviewed & tested by the community

Seems like there's relatively wide agreement...

webchick’s picture

This makes sense to me, but I'd rather defer to Dries on this patch.

I agree with what Mark Boulton says in #3: the only reason this link is confusing so many people right now is because overlays aren't in yet. But the easy solution to that is for overlays patch to change the link title to "back to site" when that feature is switched on. Seven is a really nice theme in and of itself, so we probably shouldn't require the overlays for it to make sense.

naheemsays’s picture

Just to add that that link also makes the seven theme unsuitable for use as anything but an admin theme.

Why not have that link as a breadcrumb that all other themes also use and then somewhere have the first link in the breadcrumb (typically going to Home - the same place as this is linking) be modified to the new text?

That way, this theme is not being forced into a straight jacket.

zzolo’s picture

As a seasoned Drupal user, I have to say I was confused by that link because of the word "Back", which to me means that the "live site" is where I just was, but this is not always the case. If you think about how browsers work, there is always a "Back" button that goes to the last page you were at and is there to navigate through the pages you have visited, not through the structure of the website.

I think just "front page" would better. But I think an icon in the top left middle bar for a link to the front page would be even more ideal. Anyway, just my two cents.

mikey_p’s picture

If the goal is inline editing then all inline editing links should have ?destination= set. Then the user saves the block/settings/whatever page and goes back to whatever page they were on. Back to live site may imply some form of logic behind the link, when it is a simple link to the home/front page of the site.

xmacinfo’s picture

«Back to the front page» makes much more sense and is semantically correct. I'm eager to see this commited. ;-)

As for the breadcrumb, I think we will need to open another issue as it is very hard to see were we are in D7, new and old users alike.

The root for the new breadcrumb should be "dashboard". For example:

dashboard > config > development > maintenance

With the new breadcrumb, we could keep the «Back to the front page» link displayed below the breadcrumb.

dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed this patch. I don't fully agree though, but there seems to be strong support for it.

naheemsays’s picture

The root for the new breadcrumb should be "dashboard". For example:

dashboard > config > development > maintenance

I disagree - I want to atleast have the option to use seven as a regular site theme.

Bojhan’s picture

Common, after all the research and all the discussion over frontpage, we are still siding for it? Ugh

mcrittenden’s picture

Issue tags: -Quick fix

Further discusson on this going on in #548806: No breadcrumbs in Seven

Status: Fixed » Closed (fixed)

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