Open Bug 1604841 Opened 5 years ago Updated 5 years ago

Conditionally disable "New Changes" link with reason

Categories

(Conduit :: Phabricator, enhancement, P3)

Product:
Conduit
Component:
Phabricator
Type:
enhancement

Tracking

(Not tracked)

People

(Reporter: u587052, Unassigned)

References

Details

(Keywords: conduit-triaged, conduit-upstream)

As mentioned in this comment, the "New Changes" link from 1596023 might be confusing.

We should consider cases where the "New Changes" link wouldn't be helpful, and disable the button accordingly with a reason. Some such cases could be:

  • If the user hasn't performed any actions on the differential, then there might not be a reason to see "New Changes"
  • If the user is not logged in, then seeing "New Changes" won't provide any value
  • ... Maybe more?
Depends on: 1596023
Mentor: pzalewa
Keywords: conduit-triaged, conduit-upstream
Priority: P2 → P3
You need to log in before you can comment on or make changes to this bug.