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

Remove default button margin #1487

Merged
merged 2 commits into from
Sep 5, 2024
Merged

Remove default button margin #1487

merged 2 commits into from
Sep 5, 2024

Conversation

tschumpr
Copy link
Contributor

@tschumpr tschumpr commented Sep 4, 2024

Wenn standardmässig marginLeft: 5px gesetzt wird, kann das sehr mühsam werden, wenn der Button zusammen mit anderen Elementen platziert werden muss. Daher nur da definieren, wos auch gebraucht wird.

Zudem zum Überprüfen, ob Lithologie wirklich gelöscht werden soll, auf Prompt gewechselt. Wenn zukünftig irgendwo in der Applikation ein solcher Check gebraucht wird, immer den Prompt verwenden, auch wenn sonst noch der Legacy Code im Einsatz ist.

@tschumpr tschumpr changed the title Remove default margin Remove default button margin Sep 4, 2024
Copy link

sonarqubecloud bot commented Sep 5, 2024

@tschumpr tschumpr merged commit 8aee1b1 into main Sep 5, 2024
13 checks passed
@tschumpr tschumpr deleted the remove-default-margin branch September 5, 2024 07:53
@tschumpr tschumpr mentioned this pull request Sep 5, 2024
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.

2 participants