Skip to content

Complete flavor card, compare polish, open-by-default panels - #185

Merged
rvnminers-A-and-N merged 1 commit into
mainfrom
fix/ui-polish-card-compare
Jul 18, 2026
Merged

rvnminers-A-and-N merged 1 commit into
mainfrom
fix/ui-polish-card-compare

Conversation

@rvnminers-A-and-N

Copy link
Copy Markdown
Collaborator

Round of polish from live review.

  • Flavor card (/api/card): shows all six taste heads (was 4) and a new AROMA MODEL section — every confident odor head with its probability, honest top signals (none cleared 0.5) label when nothing fires. Card height is computed from content so nothing truncates; structure panel centers to match.
  • Compare molecules: open by default; each box shows the flask loader while its card renders, then swaps to the image (and a clean error state on a bad molecule).
  • Formulation Studio: confirmed open by default.
  • Removed the stray light-theme separator line between Library and Flavor-map.

Verified live with Playwright (open-by-default, loading→image swap, no JS errors) and by eyeballing rendered cards (vanillin: 6 taste bars + honest 'top signals' aroma; limonene: 6 taste bars + citrus 100% confident).

- Flavor card now shows ALL SIX taste heads (sweet/bitter/umami/sour/salty/
  tasteless) and a dedicated AROMA MODEL section (every confident odor head with
  its probability; honest 'top signals' label when none clear 0.5). Card height
  is computed from content so nothing is truncated; structure panel centers to
  match.
- Compare molecules: open by default; each side-by-side box shows the flask
  loader while its card renders, then swaps to the image.
- Formulation Studio confirmed open by default.
- Removed the stray light-theme separator line between Library and Flavor map.

Signed-off-by: Austin L. <86896075+rvnminers-A-and-N@users.noreply.github.com>
@rvnminers-A-and-N
rvnminers-A-and-N merged commit 6f02658 into main Jul 18, 2026
3 checks passed
@rvnminers-A-and-N
rvnminers-A-and-N deleted the fix/ui-polish-card-compare branch July 18, 2026 20:31
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