Skip to content

Conversation

@andre8244
Copy link
Collaborator

@andre8244 andre8244 commented Oct 20, 2025

Add NeStepper component and Storybook stories

Other changes:

  • NePaginator: fix disabled state of Next button
  • NeDropdown: make alignToRight prop optional
  • NeTableRow: add transition to hover color

@andre8244 andre8244 marked this pull request as ready for review October 21, 2025 10:55
@andre8244 andre8244 requested a review from Copilot October 21, 2025 10:55
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR introduces a new NeStepper component that displays progress through a multi-step workflow using a progress bar and step indicators. The PR also includes several minor improvements to existing components.

Key changes:

  • Added NeStepper component with visual step progression tracking
  • Fixed NePaginator's Next button disabled state logic
  • Made NeDropdown's alignToRight prop optional to improve API ergonomics

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/components/NeStepper.vue New component implementation with progress bar and step labels
stories/NeStepper.stories.ts Storybook stories for the new NeStepper component
src/main.ts Export statement for the new NeStepper component
src/components/NeTableRow.vue Added transition effect to hover state
src/components/NePaginator.vue Fixed Next button disabled condition to handle edge cases
src/components/NeDropdown.vue Made alignToRight prop optional

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@andre8244 andre8244 requested a review from Tbaile October 21, 2025 10:58
@andre8244 andre8244 changed the title feat: add NeStepper component feat: add NeStepper and minor fixes Oct 21, 2025
@andre8244 andre8244 merged commit c31d655 into main Oct 21, 2025
4 checks passed
@Tbaile Tbaile mentioned this pull request Oct 21, 2025
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.

3 participants