Yes, please! And while we’re on it, there may be related opportunities to make the review system feel more consistent and approachable.
We have two basic types of review items:
- “Post flagged as _______” and
- “Queued post Needs Approval”
And the resolutions Approved and Rejected have opposite meanings for each class:
- Issue confirmed with a flagged post? → “Approved.”
- Issue confirmed with a queued post? → “Rejected.”
The colored resolution pills are clearly labeled, which is nice:
,
…etc. But these are located fairly far from the “Timeline & notes” area. When scrolling items and scanning notes, my brain wants to think in the context of posts and what’s done with them, but instead I have to think about what Approved or Rejected means in context of the review item type:
| note | meaning - if flag item | meaning - if queued post item |
|---|---|---|
| “Approved by” | Flag approved (problem confirmed)  |
Post approved (no problem confirmed)  |
| “Rejected by” | Flag rejected (no problem confirmed) |
Post rejected (problem confirmed) |
And a little thing, in the filter drop-down:
- Approved and Rejected just find Flag approved and Flag rejected items.
- only All reviewed or Everything will find Post approved and Post rejected items.
None of this makes the system unworkable – it’s just a bit of mental overhead that, IMHO, contributes to the review queue system being a little confusing and intimidating at first.
Maybe this should be a separate post: “Inconsistent meaning of resolutions in the review queue” – but the matters seem pretty closely tied.