Revisions and Commits
Status | Assigned | Task | ||
---|---|---|---|---|
Resolved | joshuaspence | T7603 Remove the deprecated "Repositories" application | ||
Resolved | joshuaspence | T7604 Remove "Arcanist Projects" | ||
Resolved | epriestley | T8320 Bring Owners out of the dark ages | ||
Resolved | None | T8004 Highlight files owned by self/project when viewing revision | ||
Resolved | epriestley | T2183 unify "table of contents" views between Differential and Diffusion |
Event Timeline
Mock from @nickz:
My plan here is:
- Complete T2183 first, which unifies this element between Diffusion and Differential. Diffusion already has some code which highlights files.
- Highlight files in a package you are an owner of, in both applications.
- Probably add a smaller icon with a tooltip for the package name, instead of putting the entire package name on the line.
@nickz, does that sound reasonable? The other use cases I've seen (this task, T1226), the primary use case is quickly identifying files the viewer owns. Is that the case in your use case, as well, or do you have a more complicated use case where identifying the specific package is important? There's some additional discussion in T1226.
@epriestley, thanks for consolidating my request into this task! Some comments: a person could be an owner of multiple packages, it would be better to indicate all files with their package names instead of only indicate the files you are an owner of; We actually don't need to put the package name on each line, we can group the files by package and put the package name to each group. One benefit of indicating all files with package names is everybody see the same, there will be less confusion (in most cases we can assume everybody knows if s/he is an owner of a package).