When the site is operatioing in offline mode, a message appears for logged-in admins informaing of this fact. However, it's unhelpful in as much is it does not provide any information about how to out the site back online.

For 6.x we could make this entire message a link (to avoid any t() string change). For 7.x, add the link as a separate piece.

CommentFileSizeAuthor
#2 go-online-320146-7x-2.patch820 bytespwolanin
#1 go-online-320146-6x-1.patch813 bytespwolanin

Comments

pwolanin’s picture

pwolanin commented
StatusFileSize
new go-online-320146-6x-1.patch813 bytes

here's the 6.x patch first.

pwolanin’s picture

pwolanin commented
Status: Active » Needs review
StatusFileSize
new go-online-320146-7x-2.patch820 bytes

patch for 7.x with string change.

Anonymous’s picture

Anonymous (not verified) commented

I don't want to give any anonymous user a clicking point to this. Leaving as CNR for others comments.

pwolanin’s picture

pwolanin commented

The only users who see this message are those with the 'administer site configuration' permission

pwolanin’s picture

pwolanin commented

here are a couple D7 screenshots:

Only local images are allowed.

Only local images are allowed.

yoroy’s picture

yoroy commented
Status: Needs review » Reviewed & tested by the community

Good idea. Offer just the right help/task in the context of the status message. Very helpful usability tweak.

keith.smith’s picture

keith.smith commented

Yes, I agree. This is very nice.

Anonymous’s picture

Anonymous (not verified) commented

Ok, I agree that it is a good link. Rather than adding more text can we not just wrap the link in the "Operating in offline mode." text and use a link description text of "Go online"?

yoroy’s picture

yoroy commented

How is that any more clearer than the current proposal? You're initially hiding the actual task if you only put it in the title attribute of the link.

Bojhan’s picture

Bojhan commented

Looks good to me, I hope it gets in.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Good suggestions. Committed to CVS HEAD. Thanks.

pwolanin’s picture

pwolanin commented
Version: 7.x-dev » 6.x-dev
Status: Fixed » Reviewed & tested by the community

6.x patch is in #1

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Reviewed & tested by the community » Fixed

Ok, committed this little tweak to Drupal 6 in the hopes that it will make upgrading from Drupal 6 point versions easier.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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