Skip to content

[BI-2977] - Genotype Results UI/UX improvements (w/o delete) - #496

Merged
nickpalladino merged 4 commits into
developfrom
feature/BI-2977
Oct 2, 2026
Merged

nickpalladino merged 4 commits into
developfrom
feature/BI-2977

Conversation

@HMS17

@HMS17 HMS17 commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

Description

Story: BI-2977 - Genotype Results: UI/UX Improvements (w/o delete)

Renamed "Genotyping" to "Genotypes" on the left navigation sidebar and on the germplasm results page
Removed "Download" text from download links in table, making them icon-only
Added tooltip with "Download" to download link
Added CSS to style tooltips
Added aria-label for screen reader accessibility of download link

Dependencies

bi-api: develop

Testing

  • Check left navigation sidebar shows "Genotypes" not "Genotyping"
  • Navigate to Genotypes
  • Check page title is "Genotypes"
  • Check that for each row only the download icon is visible when there is no hover/focus on it
  • Make sure tooltip appears on both hover and focus on download icon for each row (ie tabbing through page)
  • Make sure tooltip matches figma wireframe (especially color, text formatting, location relative to link, lack of shading)

Checklist:

  • I have performed a self-review of my own code
  • I have tested my code and ensured it meets the acceptance criteria of the story
  • I have commented my code, particularly in hard-to-understand areas
  • I have either updated the source of truth or arranged for update with product owner if needed https://breedinginsight.atlassian.net/wiki/spaces/BI/pages/1559953409/Source+of+Truth
  • I have run SiteImprove on pages impacted by changes

@HMS17
HMS17 marked this pull request as ready for review September 30, 2026 15:29
@HMS17
HMS17 requested review from a team and nickpalladino and removed request for a team September 30, 2026 15:29

@nickpalladino nickpalladino left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good

@nickpalladino
nickpalladino merged commit c917572 into develop Oct 2, 2026
1 check passed
@nickpalladino
nickpalladino deleted the feature/BI-2977 branch October 2, 2026 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants