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

💡 Hinweis: Ausrichtung von Kol-Button mit Icon #5413

Closed
cbraehmig opened this issue Oct 6, 2023 · 5 comments · Fixed by #5490
Closed

💡 Hinweis: Ausrichtung von Kol-Button mit Icon #5413

cbraehmig opened this issue Oct 6, 2023 · 5 comments · Fixed by #5490
Assignees
Labels
bug Something isn't working cop Community of Practices v1 v2

Comments

@cbraehmig
Copy link
Contributor

Beschreibung:

Werden mehrere Buttons in einer Reihe abgebildet, und mindestens eines der Buttons beinhaltet ein Icon (Standard oder Linksplatziert), dann werden die restlichen Buttons aus derselben Reihe um einige Pixel nach unten gedrückt (siehe zweite Reihe des nachfolgenden Bildes):

image

Reproduktionsbeispiel

https://stackblitz.com/edit/vitejs-vite-4zkq1w?file=src%2FApp.tsx

@deleonio deleonio added the bug Something isn't working label Oct 20, 2023
@deleonio
Copy link
Contributor

Gehen wir nach @cbraehmig.

@deleonio deleonio added the DESY label Oct 26, 2023
@deleonio deleonio moved this to 📄 Ready in KoliBri Board Oct 26, 2023
@deleonio deleonio added in v1 and v2 cop Community of Practices labels Oct 26, 2023
@sdvg sdvg self-assigned this Oct 27, 2023
@sdvg sdvg moved this from 📄 Ready to 🏗 In progress in KoliBri Board Oct 27, 2023
@sdvg
Copy link
Member

sdvg commented Oct 27, 2023

image

Die Baseline wird vom ersten Element im Button definiert, in diesem Fall vom Icon.

@sdvg
Copy link
Member

sdvg commented Oct 27, 2023

Ich würde erwägen, einfach mit dem Problem zu leben und bei inline-block Elemente bei Bedarf vertical-align händisch zu setzen.

Ich lege trotzdem einmal einen PR mit einem etwas hacky Workaround an. Vielleicht hat ja auch jemand eine bessere Lösung parat.
(Bisher nur gegen develop, 1.7 würde ich nach dem entsprechenden Feedback anlegen.)

@sdvg sdvg linked a pull request Oct 27, 2023 that will close this issue
@sdvg sdvg moved this from 🏗 In progress to 🔍Review in KoliBri Board Oct 27, 2023
@deleonio
Copy link
Contributor

@sdvg - wenn nichts gegen Deine hacky Lösung spricht - funktioniert diese aber recht gut.

Spricht sonst noch etwas dagegen - Du würdest ja erwägen, es von außen handle zu müssen.

@sdvg
Copy link
Member

sdvg commented Oct 30, 2023

@deleonio Der Gedanke war, dass die Lösung nicht unbedingt notwendig ist. Buttons auf der Baseline platzieren zu müssen scheinen mir ein seltener Fall zu sein und man kann das vertical-align immer selber definieren wo notwendig. Oder eben mit Flex oder Grid arbeiten.

Aber der Workaround scheint mir tatsächlich gut zu funktionieren und ich sehe keine Probleme damit, ihn so umzusetzen.

@sdvg sdvg moved this from 🔍Review to 🏗 In progress in KoliBri Board Oct 30, 2023
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in KoliBri Board Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cop Community of Practices v1 v2
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants