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

Visual Indication For Favourites #70

Closed
BrainBlasted opened this issue Jun 1, 2018 · 8 comments
Closed

Visual Indication For Favourites #70

BrainBlasted opened this issue Jun 1, 2018 · 8 comments
Labels
➕Enhancement New feature or request ❗❗ Priority: High Important 👀 UI An issue with the interface / frontend 💼 WIP This is currently being worked on

Comments

@BrainBlasted
Copy link
Contributor

Currently when you favourite an image, the heart stays the same color. Since a common pattern is to change the color of already favourited entities in other platforms, I was confused and did a brief like-unlike loop. Perhaps the color could be changed to black when something is already favourited?

@kpcyrd kpcyrd added ➕Enhancement New feature or request ❗❗ Priority: High Important 👀 UI An issue with the interface / frontend labels Jun 2, 2018
@trwnh
Copy link
Member

trwnh commented Jun 4, 2018

Seems to be live, albeit with a bug where already-liked posts are unfilled and stuck in reverse of what they should be.

@trwnh
Copy link
Member

trwnh commented Jun 4, 2018

Also not indicating on individual posts - only on timeline.

@dansup
Copy link
Member

dansup commented Jun 4, 2018

Fixed in #186 #187 #188

@dansup dansup closed this as completed Jun 4, 2018
@BrainBlasted
Copy link
Contributor Author

@dansup can you reopen this issue? Currently the favourites button no longer changes colors.

@BrainBlasted
Copy link
Contributor Author

Aplogies, must have been something on my end.

@BrainBlasted
Copy link
Contributor Author

Ah, figured it out. The button is not the proper color if you reload with the image already favourited. Hitting the favourite button again may unfavourite it.

@dansup dansup reopened this Jun 13, 2018
@dansup dansup added the 💼 WIP This is currently being worked on label Jun 13, 2018
@trwnh
Copy link
Member

trwnh commented Jun 13, 2018

@BrainBlasted The current solution to that right now is to run ls.clear() in your browser's console, because the cause of the issue is the hydration of likes. Anything you liked before the indicator logic was implemented seems to be reversed as you described.

This will be less and less of an issue for new users, but I have to think that hydrating likes shouldn't cause any issues, and the indicators should be updated at the same time, but idk how all that works.

@dansup
Copy link
Member

dansup commented Nov 22, 2018

Fixed in latest commits!

@dansup dansup closed this as completed Nov 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
➕Enhancement New feature or request ❗❗ Priority: High Important 👀 UI An issue with the interface / frontend 💼 WIP This is currently being worked on
Projects
None yet
Development

No branches or pull requests

4 participants