Repository navigation
feat(detail): add a short intro under each detail page section heading - #1041
Merged
Merged
Conversation
Each section on the pathway detail page now explains what it is for, directly under its heading (#1029). Pathway Description explains itself and gets none. Following the revised wording in the issue, the At a glance panels are renamed "Timeseries Preview" and "Assumptions & Trends Preview", and the full panel on the Assumptions & Dependencies tab becomes "Assumptions & Trends", so the two are no longer both "Overview". Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Expected version change and release notes🚨 WARNING: This PR is not expected to trigger a new versionTo trigger a version bump, use at least one conventional commit message in this branch. See: https://www.conventionalcommits.org/en/v1.0.0/ |
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://proud-glacier-0f640931e-1041.westus2.2.azurestaticapps.net |
AlexAxthelm
approved these changes
Oct 28, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #1029
What changes
Each section on the pathway detail page now has a one- or two-sentence intro under its heading, so readers know what it is for. Pathway Description explains itself and gets none.
The copy follows the revised wording in the issue comment. That renames three headings:
Benchmark Plots, Supplemental Information, Dependencies, Data Availability and Coverage keep their names.
How
SectionIntro: one grey paragraph used for all eight intros, so they look the same. The heading above it drops tomb-1so the intro reads as part of the heading.KeyFeaturesandPlotGriddraw their own headings, so each takes an optionaldescriptionshown under it.PlotGridkeeps it in the empty and non-Power states too.Tests
PathwayDetailPage.tabs.test.tsx: walks all four tabs and checks every section heading is followed by its intro, and Pathway Description is not.KeyFeatures.test.tsx,PlotGrid.test.tsx: the description renders directly under the heading (forPlotGrid, with and without data).Checked in the browser on NGFS NZ2050 REMIND across all four tabs.
Open points
🤖 Generated with Claude Code