Page MenuHomePhabricator

New Pages Feed: Hide 'Review' button for non-editors
Closed, ResolvedPublic

Description

Don't show the blue 'Review' button on the New Pages Feed list if you are not an editor.

It's not relevant, and unfairly raises their expectations, since they cannot review that page with the curation bar anyway.


Version: unspecified
Severity: normal

Details

Reference
bz38055

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 12:48 AM
bzimport added a project: PageTriage.
bzimport set Reference to bz38055.
bzimport added a subscriber: Unknown Object (MLST).

To clarify a bit: The 'Review' button should only be shown to users with the patrol right, but we do want to show the 'Review' button on all articles in the queue regardless of whether they have already been reviewed or not.

I'd prefer it if the review button were *disabled* rather than hidden, with a hover stating why it's disabled.

I generally don't like interfaces that add and remove affordances. Grey it out, fine, but don't remove it entirely.

Thanks, Brandon. Your proposal works for me. If you haven't already, can you send a grayed-out version of the button to Kaldari, in case he doesn't have it?