The phrase is either redundant or contradictory
Description
Description
Details
Details
- Reference
- fl129
| Status | Subtype | Assigned | Task | |
|---|---|---|---|---|
| · · · | ||||
| Resolved | None | T61 Descriptive labels instead of Phabricator app names | ||
| Invalid | • chasemp | T34 Configure Phabricator for our needs | ||
| Resolved | Qgil | T70 "Edit Differential Revisions" is jargon if you don't know that Differential is the name of Phab's Code Review tool | ||
| · · · |
Event Timeline
Comment Actions
aklapper wrote on 2014-04-18 08:47:06 (UTC)
Where to find this phrase exactly? Steps to reproduce welcome, as with every ticket...
Comment Actions
aklapper wrote on 2014-04-18 09:31:30 (UTC)
It is not "Edit differential versions", it is "Edit Differential Versions" instead.
"Differential" is the name of the Code Review tool, see http://fab.wmflabs.org/
This is the general question how much of Phabricator's eigen names for its tools we want to keep in the UI - same complaint pattern would be about "Edit Maniphest Tasks" if you don't know that Maniphest is for Tasks and Bugs.
Comment Actions
Nemo_bis wrote on 2014-04-18 09:39:46 (UTC)
True, I normalised the case to what it should be. Proper names are made worse by the fact that even common names are uppercase, so there is no way to identify what's supposed to be a proper name.
same complaint pattern would be about "Edit Maniphest Tasks" if you don't know that Maniphest is for Tasks and Bugs.
Yes, I wanted to file that one too but I refrained because I'm so sober. :P
Comment Actions
qgil wrote on 2014-04-21 03:27:24 (UTC)
translation.override can be edited through the Admin panel, and this is useful to change strings like this one. Now it says "Edit Code Revisions".
I have changed this one for demonstration/testing purposes. In the official Phabricator instance we will need to discuss what default strings we want to change.