Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clarify in tooltip what flag button does #373

Merged
merged 1 commit into from
May 15, 2017
Merged

Conversation

seanh
Copy link
Contributor

@seanh seanh commented Apr 26, 2017

The flag button on an annotation has a tooltip that says simply "Flag"
(or "Annotation has been flagged" if it's already flagged) but there's
no indication to the user what flagging is or does (even when they click
the button, all that happens is the flag turns red):

screenshot from 2017-04-26 16-21-46

screenshot from 2017-04-26 16-22-06

(or, err, blue?? not sure what happened in this screenshot)

"Flagging" can certainly be misinterpreted. I think some apps use it as
a way to keep track of "special" or "favorite" items (usually called
"starring" in most modern apps), or have a variety of different "flags"
that you can apply to an email or whatever it is (important, work,
personal, etc, this is usually called "tagging" or "labelling" in most
modern apps).

This commit changes the tooltips to clarify that what the button does is
report the annotation to the moderators. I've used the verb "report"
rather than "flag" as I think "Report this annotation to the moderators"
sounds better than "Flag this annotation to the moderators". Either way
I think "to the moderators" is important to clarify that this button
sends a message to the moderators and isn't, for example, just flagging
for personal use like starring / tagging / labeling.

peek 2017-04-26 16-34

The flag button on an annotation has a tooltip that says simply "Flag"
(or "Annotation has been flagged" if it's already flagged) but there's
no indication to the user what flagging is or does (even when they click
the button, all that happens is the flag turns red).

"Flagging" can certainly be misinterpreted. I think some apps use it as
a way to keep track of "special" or "favorite" items (usually called
"starring" in most modern apps), or have a variety of different "flags"
that you can apply to an email or whatever it is (important, work,
personal, etc, this is usually called "tagging" or "labelling" in most
modern apps).

This commit changes the tooltips to clarify that what the button does is
report the annotation to the moderators. I've used the verb "report"
rather than "flag" as I think "Report this annotation to the moderators"
sounds better than "Flag this annotation to the moderators".  Either way
I think "to the moderators" is important to clarify that this button
sends a message to the moderators and isn't, for example, just flagging
for personal use like starring / tagging / labeling.
@seanh seanh force-pushed the clarify-flag-tooltip branch from 069683f to e64fa19 Compare April 26, 2017 15:42
@dhwthompson
Copy link

I like the use of “Report” rather than “Flag”: I agree it’s less prone to confusion, and it seems to be more widely used elsewhere:

Reporting on Twitter

Reporting on Facebook

I wonder, since we’re already saying “report” instead of “flag”, do we need the “to the moderators” part, or can we leave it implicit that the report will go to (or has gone to) someone relevant?

@seanh
Copy link
Contributor Author

seanh commented Apr 26, 2017

I'm not sure. I think the presence of both "report" and "moderators" makes it doubly clear that what's going on here is reporting inappropriate content (as opposed to, say, reporting an annotation that has a bug). But maybe it's not necessary.

@robertknight robertknight self-requested a review May 4, 2017 21:42
@seanh
Copy link
Contributor Author

seanh commented May 12, 2017

Ping

@robertknight
Copy link
Member

LGTM and good call 👍

@robertknight robertknight merged commit 2031f06 into master May 15, 2017
@robertknight robertknight deleted the clarify-flag-tooltip branch May 15, 2017 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants