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

Tweak UI for identifications #417

Merged
merged 2 commits into from
Jun 12, 2024
Merged

Tweak UI for identifications #417

merged 2 commits into from
Jun 12, 2024

Conversation

annavik
Copy link
Member

@annavik annavik commented Jun 11, 2024

Now when identifications may include both algorithm details and comment I thought could be helpful to tweak the card layout slightly, to distinguish the taxon from other details. Also I disabled the button "✓ Agreed" to avoid confusions. If already agreed, pressing it would not have any effect.

Let me know what you think, I just did these tweaks quickly after testing submitting a comment! :)

Before:

Skärmavbild 2024-06-11 kl  11 42 46

After:

Skärmavbild 2024-06-11 kl  11 42 28

@annavik annavik requested a review from mihow June 11, 2024 10:03
Copy link

netlify bot commented Jun 11, 2024

Deploy Preview for ami-web ready!

Name Link
🔨 Latest commit ac83220
🔍 Latest deploy log https://app.netlify.com/sites/ami-web/deploys/666950b706ad490008fcc629
😎 Deploy Preview https://deploy-preview-417--ami-web.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 70
Accessibility: 95
Best Practices: 92
SEO: 92
PWA: 70
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Jun 11, 2024

Deploy Preview for ami-storybook ready!

Name Link
🔨 Latest commit ac83220
🔍 Latest deploy log https://app.netlify.com/sites/ami-storybook/deploys/666950b7136e430008a9eb3b
😎 Deploy Preview https://deploy-preview-417--ami-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mihow
Copy link
Collaborator

mihow commented Jun 11, 2024

This is great thank you @annavik! I stuffed in the ML model name in the easiest place I could. I like the Agree button change.

I did notice an empty tooltip shows if the model description is empty
image

image

At some point we will show more information about each prediction:

  • A link to the job that was used to process the image (so you can review errors or see the collection it was apart of, and what other processing steps were run).
  • A link to the model page
  • The accuracy of this species in this model
  • The number of images the model was trained on for this species

@annavik
Copy link
Member Author

annavik commented Jun 12, 2024

This is great thank you @annavik! I stuffed in the ML model name in the easiest place I could. I like the Agree button change.

I did notice an empty tooltip shows if the model description is empty image

image

At some point we will show more information about each prediction:

  • A link to the job that was used to process the image (so you can review errors or see the collection it was apart of, and what other processing steps were run).
  • A link to the model page
  • The accuracy of this species in this model
  • The number of images the model was trained on for this species

Oops, tooltip thing fixed!

Good to know! I guess we could introduce some expand feature on the card, to fit more details... Or some other solution!

@annavik annavik merged commit c99d8af into main Jun 12, 2024
5 of 6 checks passed
@annavik annavik deleted the improvement/identifications branch June 12, 2024 07:43
@annavik annavik self-assigned this Jun 17, 2024
@annavik annavik added this to the Internal Beta V2 Release milestone Jun 17, 2024
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