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

Adding reaction button (toggle on/off) to post container #45

Closed
3 tasks
yukiiii04 opened this issue Feb 27, 2025 · 0 comments · May be fixed by #50
Closed
3 tasks

Adding reaction button (toggle on/off) to post container #45

yukiiii04 opened this issue Feb 27, 2025 · 0 comments · May be fixed by #50
Assignees
Milestone

Comments

@yukiiii04
Copy link
Contributor

yukiiii04 commented Feb 27, 2025

Description

This sub issue is for getting the actual reaction button to show up. The user would click on this reaction button to toggle a reaction. For this project, we wanted to implement separate buttons for different reactions, but for the sake of time and development, we changed it so that we only have 1 overall button for this purpose.

Acceptance Criteria

This part feature will be accepted after verifying that:

  • button with fa-smile icon shows up on post container
  • when hovered, background color of button should match the other buttons (light grey)
  • when clicked, background color changes again as a signal of confirmed interaction

Implementation Notes

To implement this, @yukiiii04 and @lynzhlang had to go through all relevant files for Upvote/Downvote, which we identified to be the most similar to the feature we want to create.

For this issue specifically (frontend changes & general event function calling), @yukiiii04 looked at upvote/downvote functions and changed similar ones. Primarily by adding a component in the tpl files and creating related components.

Dependencies

No dependencies.

Resources

Branch: reactions-sprint2-yuki
All relevant files doc: https://docs.google.com/document/d/19IliLN_3vqqb-hlR2MCBgwjhhXwsxUy8r6mByMTBIhs/edit?tab=t.0

Testing

This change should be (and can be tested) through navigating to posts after they are posted and seeing if the acceptance criteria are met.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants