Image-geometry ROI editor: red/green default ROI colors; verify ROI is named without color (#1605 audit X2)

Aperta
  • Qualcosa non va?
#2,133 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
64/100
Tipo di issue
Funzionalità
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
java
Ambito
accessibility

Direzione di ricerca

Start from vcell-client/.../geometry/gui/OverlayEditorPanelJAI.java: the CONTRAST_COLORS array at line 95 and the ROI combo box swatch/name rendering at line 803. Use tools/debug-bridge to run the client and check whether hover or selection on the canvas names the ROI under the cursor; if not, add that name to the hover readout, mirroring the subvolume fix from #1605. Done means the first 6 default colors satisfy ΔE′ ≥ 10 under protan/deutan/tritan simulation and color-audit.md row X2 can be marked verified.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Accessibility Requirement Java-based User Interface

Follow-up from the #1605 color audit, row X2 (docs/accessibility/color-audit.md).

Finding. The image-based geometry ROI editor gives each ROI a color from OverlayEditorPanelJAI.CONTRAST_COLORS (vcell-client/.../geometry/gui/OverlayEditorPanelJAI.java:95). The sequence starts red 0xFF0000, green 0x00FF00, blue…, so the first two ROIs are the red/green pairing. The ROI combo box shows a swatch and the ROI name (:803). The audit did not verify whether the canvas names the ROI under the cursor. If it doesn't, telling painted regions apart depends on color.

Tasks.

  1. Check in the running client (tools/debug-bridge) whether hover or selection names the ROI.
  2. If it doesn't, add the ROI name to the hover readout, as the geometry viewer fix under #1605 does for subvolumes.
  3. Reorder CONTRAST_COLORS so red and green are not adjacent defaults.

Acceptance.

  • In a deuteranopia-simulated screenshot, every ROI region can be matched to its name.
  • The first 6 default ROI colors measure ΔE′ ≥ 10 under protan, deutan and tritan simulation.

Part of #1603.

Lingua principale
Java
Stelle
140
Fork
35
Merge medio
22h 6m
PR unite (30g)
14

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di virtualcell/vcell

Tutte le issue di virtualcell/vcell

Issue simili

Altre issue su Java

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.