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

[Feat] Favorite Posts from Preview Panel #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Nikey646
Copy link

This makes the preview panel slightly more useful, especially since the heart icon has a hover effect indicating that it should be usable, despite the fact that all it does is send you to the top of the page (Well, at least in firefox)

The way I did this is a wee bit hacky, but it works (Well, at least in firefox... Don't see why it wouldn't work in Chrome)... It however doesn't take into account if the user is logged in already or not, but I imagine danbooru will complain bitterly (or stop working) when the user isn't.

I would have done the same for the Up / Down vote buttons, however the API doesn't provide that information, even the main posts page waits for you try to up / down vote before offering the chance to remove your vote.

This makes the preview panel slightly more useful, especially since the heart icon has a hover effect indicating that it should be usable, despite the fact that all it does is send you to the top of the page (Well, at least in firefox)

The way I did this is a wee bit hacky, but it works (Well, at least in firefox... Don't see why it wouldn't work in Chrome)... It however doesn't take into account if the user is logged in already or not, but I imagine danbooru will complain bitterly (or stop working) when the user isn't.

I would have done the same for the Up / Down vote buttons, however the API doesn't provide that information, even the main posts page waits for you try to up / down vote before offering the chance to remove your vote.
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.

1 participant