Ticket #6250 (assigned defect)

Opened 4 months ago

Last modified 3 months ago

Timestamp editing is confusing

Reported by: markjaquith Assigned to: markjaquith (accepted)
Priority: normal Milestone: 2.6
Component: Administration Version: 2.5
Severity: normal Keywords: has-patch
Cc:

Description

  1. The "Edit Timestamp" checkbox is redundant. The clicking of the "Edit" link implies that you want to edit the timestamp.
  2. The "Edit" link functions as a way to hide the date-editing fields -- the text should change to reflect this when the panel is open.
  3. Canceling and hiding the timestamp editing fields should cancel the timestamp edit.
  4. The time interface is confusing with its "@ FIELD FIELD" interface and because 24-hour time is left for the user to figure out

http://img98.imageshack.us/img98/4338/timestampoy0.png

Attachments

better-date-editing.diff (8.6 kB) - added by markjaquith on 03/16/08 22:12:28.
7596.diff (0.6 kB) - added by stevish on 04/03/08 17:48:58.
Fixes the mm typo so that changing minutes triggers edit_time

Change History

03/16/08 22:12:28 changed by markjaquith

  • attachment better-date-editing.diff added.

03/16/08 22:17:46 changed by markjaquith

  • keywords set to has-patch.
  • owner changed from anonymous to markjaquith.
  • status changed from new to assigned.
  • milestone changed from 2.6 to 2.5.

better-date-editing.diff does the following:

  1. No more "Edit timestamp" checkbox. Hidden form fields track whether or not the date/time has been changed
  2. "Edit" link turns to "Cancel" after being clicked
  3. Clicking "Cancel" link resets the date fields (synced from the hidden ones)
  4. Re-worked the text in front of the date to be more accurate. Future-dated drafts are given "Publish on: DATE" ... future-published posts are given: "Scheduled for: DATE" ... past-published posts are given: "Published on: DATE," and drafts that will be published immediately when the Publish button is pressed just say: "Publish immediately"
  5. Added the "by Username" to the "Last edit" portion, per HC mockup and at Ryan's suggestion.
  6. inserted manual line break before @ HOURS : MINUTES, for better readability
  7. inserted a comma between the day of the month and the year for better readability

03/16/08 23:05:17 changed by ryan

(In [7338]) Better date editing from MarkJaquith?. see #6250

03/16/08 23:07:11 changed by ryan

Changed the "Last edit..." text to look more like the mockups. I'm not sure how I feel about "Publish immediately". I think a timestamp needs to be there to better indicate the purpose of the edit link.

03/16/08 23:44:51 changed by ryan

Need the same for pages.

03/17/08 02:18:23 changed by fitztrev

After r7338, Firebug spotted this error when changing the month of the timestamp: edit_date is not defined

03/17/08 03:09:51 changed by markjaquith

(In [7339]) Remove reference to edit_date which no longer exists. Props fitzrev. see #6250

03/17/08 03:26:53 changed by markjaquith

(In [7341]) Bring Edit Page screen up to parity with Edit Post screen WRT timestamps. see #6250

03/17/08 03:34:05 changed by markjaquith

(In [7342]) Sync "Last edited" on Edit Page screen to that on Edit Post screen. see #6250

03/17/08 03:38:57 changed by markjaquith

The problem with having a timestamp is that after one minute, the timestamp is out of date, and it implies that hitting "Publish" will publish the post in the past. Clearly the point of having the current timestamp up there is to indicate that it will be published "now." What better way to represent an ever-changing "now" than with a word like "immediately" ?

I just think that if someone is publishing without scheduling, they don't care what the time was when they started editing... that's just cruft that makes them look at the clock to make sure it represents the current time, minus how long they spent editing.

04/03/08 17:15:52 changed by stevish

Found a typo in the diff file (but I don't have shell access therefore I don't think I can add patches myself... So one of you other spiffy people will have to do it).

In comments.php, the foreach() that was added to see if anything has been changed doesn't check if 'mn' was changed. Instead, it checks 'mm' twice. The second 'mm' simply needs to be changed to 'mn'.

This typo is causing the timestamp not to be edited if you only edit the minute.

04/03/08 17:48:58 changed by stevish

  • attachment 7596.diff added.

Fixes the mm typo so that changing minutes triggers edit_time

04/03/08 17:51:30 changed by stevish

Alright, Now that it's uploaded, I think I went about this wrong... But regardless, I just uploaded a patch to fix the typo. If I did it very very wrong, and you'd like to yell at me for it (I wouldn't mind), direct your comments to stevish@gmail.com so that this article is not bogged down.