Skip to content

Open the data view from an Extended view button - #151

Merged
aivanchenk merged 4 commits into
mainfrom
feat/open-extended-view
Oct 17, 2026
Merged

aivanchenk merged 4 commits into
mainfrom
feat/open-extended-view

Conversation

@aivanchenk

@aivanchenk aivanchenk commented Oct 14, 2026 •

Copy link
Copy Markdown
Collaborator

Small changes

  • Opening: a floating "Data" button sits beside the top of the chart panel.
  • Leaving: a "Map" back button, Escape, and the browser back button all return to the map at the same view, with the chart panel open again. Browser forward reopens the view without a reload.
  • Local/UTC moves from Settings into the extended view's plot controls. The time-basis chip is gone; the fingerprint caption and tooltips still name the basis.
  • Removes the "No point picked yet" empty state, since the view only opens with a point picked, and deletes MainViewSwitch.
  • On phones, the back button sits on the left above the title so the floating theme, guide and GitHub buttons no longer cover it.
  • The tour now says UTC is switched in the extended view.
image image

Replaces the Map/Data switch. A floating "Extended view" button sits
beside the chart panel (above the sheet on phones). The data view has a
"Map" back button, and Escape or the browser back button also return to
the map, reopening the chart panel.

Local/UTC moves from Settings into the data view's plot controls, and the
empty "No point picked yet" state is gone since the view needs a point.
@github-actions

github-actions Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

🚀 Vercel preview deployed

Preview URL: https://earth-prints-2r4ndxqas-anastasiia-s-projects10.vercel.app

@aivanchenk
aivanchenk requested a review from lazarusA October 14, 2026 17:19
@lazarusA

lazarusA commented Oct 14, 2026 •

Copy link
Copy Markdown
Member

if we select more than one year, are the statistics across all of them, or only the last?

Screenshot 2026-10-14 at 19 39 34

should we show them per year, and across all ?

@lazarusA

Copy link
Copy Markdown
Member

also, this might be of interest:

@aivanchenk

aivanchenk commented Oct 17, 2026 •

Copy link
Copy Markdown
Collaborator Author

They're across all selected years. The four cards are built from the daily means of every day in the selection (2019 to 2021 in your screenshot, so 1,096 days): the mean, the lowest and highest day with its date, and how many days have data. The heading "Daily means over the selected years" is meant to say that, but it's easy to miss.

Showing them per year as well makes sense. Suggestion: when more than one year is selected, add a small table under the cards with one row per year and a last row for all years together, using the same four columns. With a single year selected nothing changes. Would that work?

Thanks for the ACE link. Its data is public Zarr (6-hourly, with an ensemble dimension), so I logged it on #132 as a test store for reading any zarr store.

@lazarusA

Copy link
Copy Markdown
Member

Would that work?

yes

Adds a table under the stat cards with one row per selected year and an
all-years row, using the same mean, lowest day, highest day and day count.
@aivanchenk

Copy link
Copy Markdown
Collaborator Author

Would that work?

yes

@lazarusA done

@aivanchenk
aivanchenk merged commit fdd36f1 into main Oct 17, 2026
2 checks passed
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.

2 participants