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

Fix/card clicks #603

Merged
merged 3 commits into from
Oct 31, 2023
Merged

Conversation

Adamik10
Copy link
Contributor

@Adamik10 Adamik10 commented Oct 16, 2023

Link to issue

https://reload.atlassian.net/browse/DDFLSBP-168

Description

Makes reservation material & stackable material card clickable. Before, the user would have to click on the card title, or an arrow that shows up on hover.

Screenshot of the result

n/a

Checklist

  • My complies with our coding guidelines.
  • My code passes our static analysis suite. If not then I have added a comment explaining why this change should be exempt from the code standards and process.
  • My code passes our continuous integration process. If not then I have added a comment explaining why this change should be exempt from the code standards and process.

Additional comments or questions

n/a

Copy link
Contributor

@spaceo spaceo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@Adamik10 Adamik10 changed the title Make reservation material & stackable material card clickable Fix/card clicks Oct 19, 2023
Before, the user would have to click on the card title, or an arrow that
shows up on hover.
So the problem was that "stackable material" and "reservation material"
are cards that need to be clickable, but they also have elements inside
that need to be clickable, sometimes with different effects.
We now make sure that the children elements handle the click events and
don't trigger the parent card onClick event handler.
@Adamik10 Adamik10 merged commit c649789 into danskernesdigitalebibliotek:develop Oct 31, 2023
9 of 10 checks passed
@Adamik10 Adamik10 deleted the fix/card-clicks branch October 31, 2023 10:32
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.

5 participants