-
Notifications
You must be signed in to change notification settings - Fork 2
Stepper #1580
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
oscarcarlstrom
wants to merge
88
commits into
main
Choose a base branch
from
stepper
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Stepper #1580
Conversation
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
oscarcarlstrom
commented
Dec 12, 2025
Contributor
Author
|
Er mange changesets her, men denne komponenten er avhenig av alle disse endringene. |
oscarcarlstrom
commented
Dec 15, 2025
oscarcarlstrom
commented
Dec 15, 2025
oscarcarlstrom
commented
Dec 15, 2025
oscarcarlstrom
commented
Dec 15, 2025
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.
Stepper
New
<Stepper>component in beta. It can be used to render a series of steps that the user has to go through.The current step is set on the
<Stepper>using thecurrentStepprop.It is possible for the user to navigate between the steps using the
<Link>component. A step can be either pending or completed. This is defined by theisCompletedprop on each step. The progress on each step is displayed by rendering a<ProgressBar>component in each<Step>. The value of the progress bar is given as a number representing percentage of completion. So why both a<ProgressBar>andisCompeted? Beacause: when used with a form, a step can be 100% filled out, but not submitted yet. It may also contain validation errors even though all fields are filled out.On smaller screens where all the steps can not fit horizontally we use horizontal scrolling with clickable chevron icons that acts as scroll indicators/scroll click areas. These are not accessible through keyboard navigation, but this is intentional: keyboard users can just use the arrow keys to scroll instead. Screen reader users will not have any need for them either.
Screen.Recording.2025-12-12.at.13.28.14.mov
TBD
grunnmuren-tailwindpackage? If so, we should probably start working on doing this for all components. We should also consider addinggrunnmuren-tailwindas a peer dependency togrunnmuren-react(in reality it already is, because the react components are using classes and the theme set up bygrunnmuren-tailwind)