Skip to content

Wrkship fix 2 - #263

Open
johnarban wants to merge 25 commits into
cosmicds:devfrom
johnarban:wrkship-fix-2
Open

Wrkship fix 2#263
johnarban wants to merge 25 commits into
cosmicds:devfrom
johnarban:wrkship-fix-2

Conversation

@johnarban

@johnarban johnarban commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

This PR implements fixes from the workshop

Fix #256 Fix #260

@patudom
patudom marked this pull request as ready for review August 6, 2026 20:20
@patudom

patudom commented Aug 6, 2026

Copy link
Copy Markdown
Contributor
  • Move tour signpost icon to left of button; remove icon from "just start" button.
  • Check with Erika about desired behavior for outlined regions. Does this seem reasonable?
  • Opacity slider only impacts region fill.
  • Outline only goes away if user unchecks box to not display regions.
  • Question for Erika: Should outlines stay black if fill opacity is 0, which makes it so that multiple regions on the map would not be distinguishable from each other; or should they switch to their region color, which may make some colors very hard to see?

@johnarban
johnarban changed the base branch from main to dev August 10, 2026 14:07
-  make sure the drawn layer changes by updating the paint properties
- to make it easier to connect region and layer, give the layer/source the same id as the region
- have to place the button inside a div since tooltip does not activate on a disabled button
… it is easier to edit and maintain. prep for using a caret expanding panel for both
…d allow scrolling. the problem is there is no way to know you can scroll
- fix broken dataset check with was always truthy
- hook up the sample errors which were doing nothing
- enable retrying the dataset (and fix possible issues with _boundaries cache)
- demphasize cancel selection
- clear -> uncheck all
- disable graph datasets until something is checked
…so allow graphing and hide duplicate trash bin
@johnarban johnarban linked an issue Aug 11, 2026 that may be closed by this pull request
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.

Change abbreviated mol / cm^2 to molecule / cm^2 everywhere

2 participants