Codecov Report

Attention: Patch coverage is 80.95238% with 4 lines in your changes missing coverage. Please review.

Project coverage is 90.91%. Comparing base (28bc599) to head (56d8423).
Report is 553 commits behind head on dev.

@@            Coverage Diff             @@
##              dev    #9989      +/-   ##
==========================================
+ Coverage   89.82%   90.91%   +1.08%     
==========================================
  Files         412      407       -5     
  Lines       11878    12610     +732     
  Branches     2364     2458      +94     
==========================================
+ Hits        10670    11464     +794     
+ Misses        681      587      -94     
- Partials      527      559      +32     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Even better explanation of the navigation control event

• Skip button isn't shown on the first step
• Maybe "Reset" should be "First" or something similar to avoid the implication that it resets all the data in the steps
• Reset button is always enabled if Step 1 has an error
• In "Controlling navigation", I would think the next steps would be disabled until the prerequisite is complete, instead of showing a modal telling you since it already has validation errors

• Skip button isn't shown on the first step

That is expected. The first step is not skippable in the docs. I changed that the SkipButton is only shown if the step is skippable. I think that is better UI. For most steppers the button will never be shown because they have no skippable steps and I think that is good.

• Maybe "Reset" should be "First" or something similar to avoid the implication that it resets all the data in the steps

The reset button is actually only for the docs so that users can revert to the initial state. In real life you'd not need it.

• Reset button is always enabled if Step 1 has an error

This is expected.

• In "Controlling navigation", I would think the next steps would be disabled until the prerequisite is complete, instead of showing a modal telling you since it already has validation errors

Yeah, good point. But we currently can't do that with the existing design. We'll need to add more API to the event to distinguish between the stepper just checking if the action would be allowed and an actual attempt by the user.