Skip to content

Added popup#43

Open
haram082 wants to merge 2 commits intomainfrom
info-popup
Open

Added popup#43
haram082 wants to merge 2 commits intomainfrom
info-popup

Conversation

@haram082
Copy link
Collaborator

image

  • Created a learn more button that shows a popup for more info about p-recs
  • couldn't default to show popup at first render due to new render every time search function is used
  • Any feedback on the button or modal UI appreciated

@haram082 haram082 requested a review from evelynhasama April 19, 2024 21:05
Copy link
Collaborator

@evelynhasama evelynhasama left a comment

Choose a reason for hiding this comment

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

looks good! i think we still need to have this open on first visit but this is good for now

Copy link
Collaborator

@evelynhasama evelynhasama left a comment

Choose a reason for hiding this comment

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

sweet looks good!

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.

2 participants

Comments