Skip to content

Add overlay feature to application#173

Merged
btmonier merged 20 commits intomainfrom
vis-overlay
Mar 23, 2026
Merged

Add overlay feature to application#173
btmonier merged 20 commits intomainfrom
vis-overlay

Conversation

@btmonier
Copy link
Member

  • Adds overlay feature to GUI:
    • Allows for the import of NumPy "path" files for both "predicted" and "observed" path types
  • Add new features to image export:
    • Modify title
    • Modify legend types (presence, absence) for paths and cell values
    • Modify image dimensions and scale
    • Modify preexisting range parameters for bigger or smaller views
  • General reorganization of UI layout to conserve screen space

@btmonier btmonier requested review from aberthel, smm477 and zrm22 March 23, 2026 14:39
@btmonier btmonier self-assigned this Mar 23, 2026
@btmonier btmonier added the enhancement New feature or request label Mar 23, 2026
{matrixData && !isLoading && !error && (
<>
{topControlsVisible && (
{topControlsVisible && (<>
Copy link
Contributor

Choose a reason for hiding this comment

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

Just because I don't fully understand this. Why have the <>? Wouldn't that just be an empty HTML element?

Copy link
Member Author

Choose a reason for hiding this comment

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

It's a commonly used React shorthand for Fragments. Instead of writing something like <React.Fragment>...</React.Fragment>, you can use <>...</>. In other words, this doesn't return an empty HTML element but is a grouping device which allows me to avoid pointless divs while still satisfying the "one parent" rule in JSX or in this app's case, TSX (https://docs.thcl.dev/react/jsx-one-parent).

More info here: https://react.dev/reference/react/Fragment

@btmonier btmonier merged commit 9b10777 into main Mar 23, 2026
@btmonier btmonier deleted the vis-overlay branch March 23, 2026 23:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants