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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
FileSize
go-online-320146-6x-1.patch813 bytes

here's the 6.x patch first.

pwolanin’s picture

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

patch for 7.x with string change.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous 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 CreditAttribution: pwolanin commented

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

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented

here are a couple D7 screenshots:

yoroy’s picture

yoroy CreditAttribution: 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 CreditAttribution: keith.smith commented

Yes, I agree. This is very nice.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous 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 CreditAttribution: 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 CreditAttribution: Bojhan commented

Looks good to me, I hope it gets in.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Good suggestions. Committed to CVS HEAD. Thanks.

pwolanin’s picture

pwolanin CreditAttribution: 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
CreditAttribution: Gábor Hojtsy 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) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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