Skip to content
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

feat(Stepper): new component #549

Closed
wants to merge 77 commits into from
Closed

feat(Stepper): new component #549

wants to merge 77 commits into from

Conversation

eduayme
Copy link
Contributor

@eduayme eduayme commented Aug 18, 2023

Resolves #180

Inspirations:

Work done for the first version of this new component (based on the tabs component):

  • Prop default-index to be able to set any step as the default active.
  • Props completed-icon, active-icon and pending-icon to set which icon to display on each step.
  • Prop display-divider to be able to hide the horizontal line.
  • Emit change to be able to listen to changes by using the @change event.

Preview:
screenshot

In the future some ideas would be:

  • Add vertical-navigation prop to display steps on top of each other.
  • Add vertical-layout prop to display the avatar on top of the label.
  • Add read-only prop to disable any click on the component.
  • Add size prop to set the size of all elements.
  • Add color and variant props to change the visual style of the step.
  • Add custom icons as dividers.
  • Add description property to step item.

PD:
To develop this Stepper I have used the UButton and UAvatar to simplify the amount of code. I saw in other components the use of internal components are used so I though it was not a problem. Feedback is more than welcome!! :)

benjamincanac and others added 30 commits August 1, 2023 15:05
Class priority issues in some cases when ring already defined on dark mode for example (input).
@benjamincanac
Copy link
Member

Still sorry for the delay, this will still not be part of the next 2.10.0 release, but it will be part of the 2.11 for sure!

@benjamincanac
Copy link
Member

Unfortunately I won't have time to include this in tomorrow's release against what I said above, sorry about that. I'll also have to consider #976. Thanks for your patience!

@benjamincanac
Copy link
Member

Closing this in favor of #976.

@uricholiveira
Copy link

Any news?

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.

[Navigation] Steps