Skip to content

User feedback: panel layout, HEX colour input, annotation fixes, identity tooltip#16

Open
happykhan wants to merge 4 commits into
masterfrom
user-feedback-fixes
Open

User feedback: panel layout, HEX colour input, annotation fixes, identity tooltip#16
happykhan wants to merge 4 commits into
masterfrom
user-feedback-fixes

Conversation

@happykhan
Copy link
Copy Markdown
Owner

Summary

Fixes four tractable issues from user feedback testing. Each fix is a separate commit.

Issue Change Status
#8 - Settings and output panels cannot be viewed simultaneously Left settings column scrolls independently; right output column uses sticky positioning on lg+ screens so both remain visible Fixed
#9 - Add HEX colour code input alongside colour picker Added a text input next to each ring colour picker that accepts/displays hex values, synced bidirectionally with the colour swatch Fixed
#11 - Statistics panel "average identity" has no explanation Added an info icon with hover tooltip explaining the alignment-length-weighted mean calculation Fixed
#12 - Annotation spreadsheet: cannot delete rows, edit cells, or paste from Excel Switched Handsontable from controlled to uncontrolled mode, eliminating the React re-render overwrite conflict. Context menu now includes row insert/delete. Added "Reset All" button Fixed
#10 - File read permission error Needs deeper root cause investigation Out of scope
#13 - Linear visualisation Feature request, out of scope for this PR Out of scope

Test plan

  • Build passes (npm run build)
  • All 132 unit tests pass (npm test)
  • On a standard laptop (1366x768 or 1440x900), verify settings panel scrolls while output/plot stays visible
  • Add a ring, verify hex input appears next to colour picker; type a hex value and confirm swatch updates; pick a colour and confirm hex input updates
  • In Statistics panel, hover the "?" icon next to Avg Identity and verify tooltip appears
  • Open annotation editor, edit a cell inline, verify the edit persists; right-click to delete a row via context menu; paste multi-row data from a spreadsheet; click "Reset All" to clear

Generated with Claude Code

happykhan and others added 4 commits May 27, 2026 16:19
On standard laptops the settings and output panels stacked vertically,
requiring users to scroll past all settings to see results. The left
settings column now scrolls independently while the right output column
stays visible via sticky positioning on lg+ screens.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Users can now type or paste a hex colour value (e.g. #3498db) directly
instead of relying solely on the browser colour picker. The text input
syncs bidirectionally with the colour swatch and validates on blur.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The Avg Identity value in the Statistics panel now has a small info icon
that shows a tooltip on hover explaining the calculation: alignment-
length-weighted mean of BLAST percent identity across filtered hits.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Switches Handsontable from controlled to uncontrolled mode to fix the
classic React-rerender-overwrites-cell-edits conflict. Table data is
now held in a ref and only pushed to Handsontable via loadData() when
external operations occur (file import, add, delete). In-cell edits,
context menu row operations, and paste from Excel now work reliably.
Also adds a "Reset All" button to clear all annotations.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying brigx with  Cloudflare Pages  Cloudflare Pages

Latest commit: d072897
Status: ✅  Deploy successful!
Preview URL: https://b21e6f7f.brigx.pages.dev
Branch Preview URL: https://user-feedback-fixes.brigx.pages.dev

View logs

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.

1 participant