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

NcDialog: Padding not applied correctly #4977

Closed
Chartman123 opened this issue Dec 19, 2023 · 5 comments
Closed

NcDialog: Padding not applied correctly #4977

Chartman123 opened this issue Dec 19, 2023 · 5 comments

Comments

@Chartman123
Copy link
Contributor

padding-block: 4px 8px; // 4px to align with close button, 8px block-end to allow the actions a margin of 4px for the focus visible outline
padding-inline: 12px 8px; // Same as with padding-block, we need the actions to have a margin of 4px for the button outline

The padding specified in the above rules is not applied correctly. To solve this problem !important should be added to those two lines.

@susnux
Copy link
Contributor

susnux commented Dec 20, 2023

!important should be avoided where possible to allow users of the library to override the styles if needed (e.g. the filepicker in this case)

@ShGKme
Copy link
Contributor

ShGKme commented Dec 20, 2023

@Chartman123
Copy link
Contributor Author

Chartman123 commented Dec 20, 2023

Is this already released? If not, yes this should probably fix it...

@ShGKme
Copy link
Contributor

ShGKme commented Dec 22, 2023

Is this already released? If not, yes this should probably fix it...

Not yet.

@susnux
Copy link
Contributor

susnux commented Feb 10, 2024

Is fixed with the latest release at least it looks correct, reopen if I am wrong.

@susnux susnux closed this as completed Feb 10, 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

No branches or pull requests

3 participants