WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#29025 closed enhancement (fixed)

Link to front-end post from dashboard activity stream

Reported by: UaMV Owned by: SergeyBiryukov
Milestone: 4.2 Priority: normal
Severity: minor Version: 4.0
Component: Administration Keywords: ux-feedback has-patch
Focuses: ui, administration Cc:
PR Number:

Description

The dashboard activity stream currently links to the post edit screen when a user has capabilities to edit the post. However, non-editors only see a list of recently published posts ... with no way of easily viewing the actual post. It may benefit UX to add a link to the post, such as that which is available in the comment stream.

Attachments (4)

29025.inaccessible-posts.png (80.7 KB) - added by UaMV 5 years ago.
View of activity stream for non-editors
29025.patch (1.3 KB) - added by UaMV 5 years ago.
Appends permalink to post title
29025.2.patch (1.3 KB) - added by DrewAPicture 5 years ago.
alternate approach
29025.3.patch (1.3 KB) - added by michalzuber 5 years ago.
Moved translator comment to context

Download all attachments as: .zip

Change History (12)

@UaMV
5 years ago

View of activity stream for non-editors

#1 @UaMV
5 years ago

  • Focuses accessibility added
  • Keywords ux-feedback added
  • Severity changed from normal to minor

@UaMV
5 years ago

Appends permalink to post title

#2 @UaMV
5 years ago

  • Keywords has-patch added

@DrewAPicture
5 years ago

alternate approach

#3 follow-up: @DrewAPicture
5 years ago

Hi UaMV, welcome to Trac!

I could see maybe modifying the output for users lacking the ability to edit any/all of the posts listed here, though I'm not sure we really need a view link. I also hesitate to use a hash for a link label -- we used to do that in the Comments screen and it was confusing for people.

I've outlined an alternate approach in 29025.2.patch that would reduce the need for two separate strings, simply swapping the edit link for the permalink for users lacking the edit_post capability for the given post. This would at least give non-editors the ability to directly visit the posts on the front-end as proposed.

#4 in reply to: ↑ 3 ; follow-up: @UaMV
5 years ago

Replying to DrewAPicture:

I've outlined an alternate approach in 29025.2.patch that would reduce the need for two separate strings, simply swapping the edit link for the permalink for users lacking the edit_post capability for the given post. This would at least give non-editors the ability to directly visit the posts on the front-end as proposed.

Nice. I like this alternate approach. Yes, I think the prime objective is to get non-editors to the front end. The hash is obtrusive.

Thanks for the welcome. Any advice or direction you can offer as I step in is more than welcome.

#5 in reply to: ↑ 4 @DrewAPicture
5 years ago

  • Milestone changed from Awaiting Review to Future Release

Replying to UaMV:

Thanks for the welcome. Any advice or direction you can offer as I step in is more than welcome.

You're here and you've uploaded a patch, so you're already well on your way :-)

We're kind of nearing the end of a development cycle just now (4.0), so it's unfortunately too late for any new enhancements. I'm marking the ticket Future Release, so let's circle back around once 4.1 gets rolling.

@michalzuber
5 years ago

Moved translator comment to context

#6 @joedolson
5 years ago

  • Focuses accessibility removed

#7 @SergeyBiryukov
5 years ago

  • Milestone changed from Future Release to 4.2

Moved translator comment to context

The translator comment is appropriate here, no need to move it to context. _x() is meant to distinguish the same string used in different contexts.

29025.2.patch looks good to me.

#8 @SergeyBiryukov
5 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 31075:

In dashboard activity widget, display a front-end link if the user cannot edit the post.

props UaMV, DrewAPicture.
fixes #29025.

Note: See TracTickets for help on using tickets.