-
Notifications
You must be signed in to change notification settings - Fork 35
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
Implement styling for split Button across all themes #6228
Conversation
Netlify Draft Deployment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Vor dem eigentlichen Code Review würde ich hier gerne die eigentliche Style-Änderungen abklären: Zunächst mal sehen einzelne Themes "kaputt" aus, das sollte auf jeden Fall behoben werden. Default z.B. sieht schon besser aus, hier würde ich bei einem Split-Button aber ein leicht anderes Styling erwarten: Der Split Button sollte als ein Element, mit gemeinsamen umgebenden Rahmen erkennbar sein, mit einem Trennelement zwischen den beiden Elementen. Ein gutes Beispiel sind hier die Buttons von GitHub: Beim Default-Theme könnte das ungefähr so aussehen: @deleonio Gib hier gerne noch Input, wenn du genauere Vorstellungen hast. |
☑️ Definition of Done checklist