Uses modern methods in Slowvote, adds appmenu, consistent create into Badges
Details
- Reviewers
epriestley - Maniphest Tasks
- T9690: Remove calls to methods deprecated by newPage()
- Commits
- Restricted Diffusion Commit
rP065df01f656e: Modernize Slowvote, fix Badges mobile menu
View Poll list, new poll, edit poll, vote in poll.
Diff Detail
- Repository
- rP Phabricator
- Branch
- newpage-slowvote
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 9112 Build 10739: Run Core Tests Build 10738: arc lint + arc unit
Event Timeline
I think this has come up before, but I don't remember how we resolved it: I think we should put "Create X" only on the list view, not the detail view.
We aren't consistent about this right now. Applications like Differential, Diffusion, People, Files and Projects do not have this action. Other applications, like Maniphest, Phame, and Slowvote, do have this action.
I'd like to make it consistent. We could make it consistent by adding the action everywhere or by removing it everywhere; of the two, I think adding it everywhere is worse.
Although it's fine for most applications, it's misleading in some applications (particularly, applications where there's some sort of object hierarchy). For example, I think having this action would be undesirable in applications like Differential (unclear/confusing that "New Diff" is not related to the current revision) and currently find it perplexing in Phame (I'm "in" a blog, but "New Post" doesn't add a new post to the blog I'm in).
In other applications, object creation is very rare and this button just feels like clutter on every page (Diffusion is the big one here, but probably Legalpad too). Badges, in this diff, is also in this vein for me -- I suspect I will never use this button, because if I wanted to create a new badge I'd never navigate to an existing badge first.
Phame is kind of a special case of messiness (per T9742).
In Maniphest and other applications, this action is also available from the "Quick Create" menu.
I do think there are two sort of special cases:
- In Phriction, this action is present but already contextual (it prefills with the current path) and I use it fairly regularly, and the list view isn't a primary UI.
- This might also be common enough in Maniphest to justify either special casing it or finding some other approach. I do use that one sometimes, I think.
Phame is a bad example for sure, I'm working on a new BlogView now and it will remove a lot of junk. Other apps I don't find it as cut and dry. Mostly, if we always have it on every page, on every app, it because something users don't think about, it's just there instinctually. I also think in cases like Badges, people when they first set up the application, create many. Moving to the list view only makes it a bit harder to remember and process as a consistent action, but I think it's possible. Maybe we could do something like:
- Add a "Create X" method higher up
- It automatically becomes a button in search views (not a crumb action, reinforces it appears on lists)
- It automatically is in the "Quick Create" anywhere in that app.
My Phame plan is basically, on the BlogView page, there is a header with "Write" and "Manage" as action buttons. We could explore something similar in Phriction pretty easily and see how it feels.