-
Notifications
You must be signed in to change notification settings - Fork 70
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(core/progress-bar): add new web component 'ix-progress-bar' #1248
base: main
Are you sure you want to change the base?
Conversation
feat(core): add progress bar component
|
@rouven-s Thank for your PR. We have to address some points before we can proceed here.
If this points are fulfilled we can further proceed with the code review. |
@rouven-s Lint fails on your component: |
@rouven-s please read the first sentence of the linked visual regression test guide: https://github.com/siemens/ix/blob/main/CONTRIBUTING.md#visual-regression-testing. Its not allowed to add windows based snapshots these snapshots cannot be checked via github workflow. |
@danielleroux I added a11y and regression tests. Following the instructions for the screenshot creation, running Running Do you have a hint on what I'm doing wrong? |
@rouven-s I have looked a little bit more into detail and also tried setting up an docker development enviroment but it just seems that windows making strange symlinks and pnpm seems not really capable of dealing with multiple filesystems and volumes (when mounting folders). (https://pnpm.io/faq#:~:text=Does%20pnpm%20work%20across%20multiple,address%20a%20location%20in%20another.) For further development i would suggest to you:
I generally would you suggest to use the WSL2 approach, but i also wrote a small Dockerfile, so it is also possible to use: |
@rouven-s PR review + adaptions will be part of our next sprint |
Hi, i'm interested by this component; is this still planned ? |
Hello @BastLast, yes, this component is planned for the next quarter of 2025 and will also be aligned with the new form components. |
💡 What is the current behavior?
🆕 What is the new behavior?
Added a new core web component "ix-progress-bar".
🏁 Checklist
A pull request can only be merged if all of these conditions are met (where applicable):
pnpm run docs
)pnpm test
)pnpm lint
)pnpm build
, changes pushed)👨💻 Help & support