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

Likes Suggestion #727

Merged
merged 4 commits into from Jul 21, 2013
Merged

Likes Suggestion #727

merged 4 commits into from Jul 21, 2013

Conversation

Spuds
Copy link
Contributor

@Spuds Spuds commented Jul 20, 2013

This changes the likes button from a dropdown selection to a "descriptive" icon ... yeah that's not very descriptive but it was my best short explanation.

  1. remove the pulldown to get like or unlike links
  2. remove the heart 💔
  3. user a new star icon in three ways ...
  • plain star if it has likes and you can't like it so ⭐ 4 Likes
  • a ⭐with a plus if you can like it (followed by the number if there are likes), just click it to like
  • a ⭐ with a minus if you liked it so you can remove it, just click it to remove your previous like

…remove and view style

Signed-off-by: Spuds <spuds@spudsdesign.com>
Signed-off-by: Spuds <spuds@spudsdesign.com>
Signed-off-by: Spuds <spuds@spudsdesign.com>
…ve a distinct button for add/remove/view

Signed-off-by: Spuds <spuds@spudsdesign.com>
@norv
Copy link
Contributor

norv commented Jul 21, 2013

Thank you! This is much more like they should behave, to be user-friendly and fit the quick 'click to like this and move on' expectation users of a forum (and other content sharing apps) have.

@norv
Copy link
Contributor

norv commented Jul 21, 2013

On a side note, are the former icons still needed? Those that were used for adding and removing a like.

@Spuds
Copy link
Contributor Author

Spuds commented Jul 21, 2013

The former ones should not be needed, two were in the sprite and I changed those to the star and one was a standalone that was moved to the sprite (where it should have been) and the standalone removed.

norv added a commit that referenced this pull request Jul 21, 2013
@norv norv merged commit 3132cb3 into elkarte:master Jul 21, 2013
@Spuds Spuds deleted the lksg branch July 21, 2013 13:38
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.

None yet

2 participants