Change "notify" interface

openlab-at-city-tech / webworkqa

WeBWorK integration for WordPress and BuddyPress

GNU General Public License v2.0

4 stars 2 forks source link

Change "notify" interface #155

Closed boonebgorges closed 5 years ago

boonebgorges commented 5 years ago

Remove bell icon and replace with text link: ‘Follow’ with tooltip saying “Notify me of future replies to this question.”

boonebgorges commented 5 years ago

What do we show for users who have clicked the button? The text "unfollow" with a corresponding tooltip?

jennaspevack commented 5 years ago

How about this? For tooltip text: "Notify me of future replies" and "Stop notifications" follow-plus following-check unfollow-minus

boonebgorges commented 5 years ago

Thanks, @jennaspevack ! When is the "Following" text to be used? Only as a success message when initially clicking 'Follow'? (Subsequent pageloads then show 'Unfollow'?)

jennaspevack commented 5 years ago

Oops. Sorry that wasn't clear. I was thinking that unfollow would be a hover on the Following state.

Base state: FOLLOW / Tool Tip: "Notify me of future replies" On-Click: Change to FOLLOWING FOLLOWING State Hover: UNFOLLOW / ToolTip: "Stop notifications"

If this too complicated, then maybe lose the "Stop notifications" tooltip.

boonebgorges commented 5 years ago

Oh, gotcha! That makes more sense, and shouldn't be a problem to implement.

boonebgorges commented 5 years ago

I think I've captured it. Here's a gif (apologies, wanted to show hover!!): Screenshot_2019-05-01_13-39-07 Peek 2019-05-01 14-03

I made a judgment call about colors, based on what I saw in your mockup:

When subscribed, color is blue. Hover is black.
When unsubscribed, color is black. Hover is blue.

bree-z commented 5 years ago

This looks good, thanks! UI looks good, and subscribing and unsubscribing works as expected.