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.

Sign up for GitHub

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

Jump to bottom

Result cards #21

Merged
merged 12 commits into from
Sep 20, 2023
Merged

Result cards #21

merged 12 commits into from
Sep 20, 2023

Conversation

EmilyClare4
Copy link
Collaborator

Basic cards for displaying search results

  • cards display basic info for each result
  • warning label added to image of any toxic mushrooms
  • link for further details (possibly displayed as a pop up window?) - logic will be implemented later
  • link for lookalikes shows if there are lookalikes associated - logic will be implemented later

@EmilyClare4 EmilyClare4 added enhancement New feature or request Merge Request labels Sep 20, 2023
@EmilyClare4 EmilyClare4 self-assigned this Sep 20, 2023
@EmilyClare4 EmilyClare4 linked an issue Sep 20, 2023 that may be closed by this pull request
UI Result Cards #14
Closed
RosieHall
RosieHall approved these changes Sep 20, 2023
View reviewed changes
Copy link
Collaborator

@RosieHall RosieHall 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 to me

@EmilyClare4 EmilyClare4 merged commit f725f35 into master Sep 20, 2023
0 of 3 checks passed
@EmilyClare4 EmilyClare4 deleted the result-cards branch September 20, 2023 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RosieHall RosieHall RosieHall approved these changes

@joel-yap joel-yap Awaiting requested review from joel-yap

@AdamN2424 AdamN2424 Awaiting requested review from AdamN2424

@kloudsoft91 kloudsoft91 Awaiting requested review from kloudsoft91

Assignees

@EmilyClare4 EmilyClare4

Labels
enhancement New feature or request Merge Request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

UI Result Cards
2 participants
@EmilyClare4 @RosieHall