Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#6054 closed defect (bug) (fixed)

"Saved at [time]" status box shouldn't shift page

Reported by: lloydbudd Owned by: mdawaffe
Milestone: 2.5 Priority: normal
Severity: normal Version: 2.5
Component: General Keywords: has-patch
Focuses: Cc:


"Saved at [time]" status box shouldn't shift page

ENV: WP trunk r7117

I don't know if this is only because it isn't styled yet.

"Saved at [time]" status box shouldn't shift parts of the page down. This is distracting for a person. It should appear in an existing space for it.

Attachments (1)

6054.diff (4.5 KB) - added by mdawaffe 13 years ago.

Download all attachments as: .zip

Change History (7)

#1 @mdawaffe
13 years ago

  • Status changed from new to assigned

But is it distracting for a robot? I think not.

Maybe iammattthomas can advise.

#2 @mdawaffe
13 years ago

How about we put it in the sidebar?

It was under the post box before because that's where the save buttons were. Now the save buttons are in the sidebar, so autosave notices should go there too.

Attached make the message more subtle and puts it by the save buttons. It shifts the layout of the sidebar a bit still. Good? Bad? The sidebar is also shifted a bit when the preview link is added; this has been the case for a while.

The main content layout is still shifted a bit by the permalink line. This also has been the case for a while.

13 years ago

#3 @lloydbudd
13 years ago

  • Keywords has-patch added

mdawaffe, I think this addresses the issue, primarily by being more subtle, but it is also good to have it in the vicinity to have the state near the Save button, b/c that matches expectations and is common in other applications.

#4 @ryan
13 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [7145]) Autosave notice styling from mdawaffe. fixes #6054

#5 @mdawaffe
13 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

#6 @mdawaffe
13 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

oops: wrong ticket

Note: See TracTickets for help on using tickets.