#28350 closed defect (bug) (fixed)
"View Post" buttons for drafts not consistent
| Reported by: | mordauk | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | low | Milestone: | 4.0 |
| Component: | Editor | Version: | 4.0 |
| Severity: | normal | Keywords: | commit dev-reviewed |
| Cc: | Focuses: | ui, administration |
Description
When a post is saved as a draft, the main publish box shows a "Preview" button. The URL for the preview button is http://site.com/?p=149&preview=true.
Next to the post's permalink, however, is a "View" button. The URL for this button is http://site.com/?p=149 and is missing the &preview=true parameter.
Also in the toolbar is a "View" button that's identical to the "View" button next to the permalink. It also has a URL of http://site.com/?p=149.
Since the post is a draft, it seems logical that all of the buttons should all be "Preview" or "Preview Post" and should all have the &preview=true parameter.
Having the buttons next to the permalink and the toolbar show "View Post" is just confusing when the post is a draft. I've caught myself several times thinking "oh, did I publish this by accident?" when seeing the "View Post" buttons.
Attachments (5)
Change History (24)
#3
in reply to: ↑ 2
@
12 years ago
Replying to SergeyBiryukov:
The docblock should just be
/** This filter is documented in wp-admin/includes/meta-boxes.php */.
Aha! I was wondering what the proper method was for that, thanks! Updated patch attached.
#4
@
12 years ago
28350-3.patch updates the tool bar to show Preview instead of View if the current item is a draft and the current user can read it.
#6
@
12 years ago
28350.4.patch uses a consistent target for all three links to avoid opening multiple tabs with the same preview.
#11
@
12 years ago
- Resolution fixed
- Status closed → reopened
The standard "Preview Changes" button actually submits a form to ensure you are previewing the latest changes. This new "Preview" button does not do this, which is why it was always just "View Post".
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
The docblock should just be
/** This filter is documented in wp-admin/includes/meta-boxes.php */.