Repository navigation
663 jasmine to vitest - #664
Merged
Merged
Conversation
…nitial state of reveal card
BREAKING CHANGE: wrap the enhanced image in a dialog component
removed because of security issues and to much maintainability costs. Also mdx is not a standard, it is nice but it had to much dependencies, so we want to keep it as lightweight as possible. sorry :/
Member
Author
|
I removed storybooks for now because it had security issues and it had too many dependencies. Also it was incomplete, but it was a nice overview. |
Also moved char counter to textfields and waves to ripple. Fixed tests of scrollspy and prepared docked and modal display plugin for removal.
Member
Author
|
We should release this as 2.4.X, because there were many changes, and it will break some things. I removed many old stuff, to keep it maintainable and clean. Also the test are waaaaaay faster now, so we can move a lot faster from now on 💪 also typescript slows development down, due to packing and transpiling... we should transition back to to good ol JS! |
Also remove compiled examples and prepare gitignore.
Remove old test references and complicated texts.
Also update breadcrumb element
joestr
approved these changes
Sep 23, 2026
Also removed the plugin files. This has to be done the other way around. The datepicker and timepicker can be placed inside a modal or popup. Otherwise the code becomes even bigger, they already are way too long! Over 1000 lines of code. This needs to be refactored! Also use Temporal API for calendar and time.
Member
Author
|
The Datepicker and Timepicker is a mess! I think we should start in small steps and make a seperate MiniCalendar which holds calender business logic which we can build upon from TemporalJS. |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Proposed changes
Screenshots (if appropriate) or codepen:
Types of changes
Checklist: