Page MenuHomePhabricator
Create Task
Maniphest T10084

Flag notes should be visible in object properties
Open, LowPublic
Actions

Assigned To
None
Authored By
scfc
Jan 5 2016, 10:19 AM
Tags
Referenced Files
None
Subscribers
chad
DheerendraRathor
epriestley
ftdysa
hach-que
johnny-bit
joshuaspence
scfc
Tokens
"Like" token, awarded by cspeckmim."Like" token, awarded by johnny-bit."Like" token, awarded by hach-que."Like" token, awarded by Luke081515.2."Like" token, awarded by DheerendraRathor.

Description

When I click "Flag for later" in a Maniphest task and add a note, this note is only visible at https://secure.phabricator.com/flag/, but not in the Maniphest task. It is also not shown when you click "Remove [Blue] Flag", but just silently deleted. As I typically use the note to remember the reason why I flagged this or that task and what I need to do with it, this makes the process more cumbersome than it needs to.

The note should be visible in the Maniphest UI, either around/"over" (as a tooltip) "Remove [Blue] Flag" on the top or perhaps directly above the comment input box.

Related Objects

Event Timeline

scfc created this task.Jan 5 2016, 10:19 AM
DheerendraRathor awarded a token.Jan 5 2016, 2:28 PM
Luke081515.2 awarded a token.Jan 5 2016, 2:35 PM
hach-que awarded a token.Jan 6 2016, 12:52 AM
hach-que added a subscriber: hach-que.

This would make flags actually useful. I think the last time I used them was years ago, and I've just been adding comments to tasks whenever I need to add personal reminders.

epriestley mentioned this in T10085: Phabricator should ask if it should remove a flag when the flag's owner changes a task.Jan 6 2016, 2:50 AM
epriestley renamed this task from Flag notes should be visible in Maniphest UI to Flag notes should be visible in object properties.Jan 6 2016, 2:54 AM
epriestley triaged this task as Low priority.
epriestley added subscribers: chad, epriestley.Jan 6 2016, 3:09 AM

@chad, not sure if you have design thoughts here, but I think this is broadly reasonable.

My naive approach would be to do one of these in the property list:

Flag: This task contains spicy drama!

I'd probably only show that if the flag had a note? I don't think this adds too much clutter because it's deeply opt-in (you have to add a flag, and you have to leave a note on the flag).

Alternatively we could do tooltip stuff, although that's kinda junk on mobile.

It's also maybe worth thinking about turning "Remove Flag" into some sort of flow that lets you review/edit the note, but I don't have any great ideas on that and I sort of like that it's pretty lightweight.

chad added a comment.Jan 6 2016, 3:12 AM

good enough for me

epriestley moved this task from Backlog to vNext on the Flags board.Jan 6 2016, 3:18 AM
DheerendraRathor added a subscriber: DheerendraRathor.Jan 6 2016, 5:17 AM
johnny-bit awarded a token.Jan 6 2016, 7:21 AM
johnny-bit added a subscriber: johnny-bit.
joshuaspence added a subscriber: joshuaspence.Jan 6 2016, 8:12 AM
ftdysa added a subscriber: ftdysa.Jan 6 2016, 2:14 PM
cspeckmim awarded a token.Jan 8 2016, 2:20 AM
Phabricator · Built by Phacility