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

Reduced button sizes from Bootstrap's oversized defaults for improved aesthetics #1456

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

afridsyed326
Copy link

@afridsyed326 afridsyed326 commented Mar 12, 2024

This change is Reviewable


Screenshot from 2024-03-12 15-04-32
Screenshot from 2024-03-12 15-04-00

@@ -48,3 +48,12 @@
border-radius: 4px; /* future proofing */
-khtml-border-radius: 4px; /* for old Konqueror browsers */
}

.btn {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overwriting all .btn styles with !important is not the best idea.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

agreed! It would be better create another class and not overwrite bootstrap's classes

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, make this, so I can merge it

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jchabrowski Thank you for the suggestion. I Will make it . @shakaran

Copy link
Contributor

@xTudoS xTudoS Mar 25, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@afridsyed326 You could try just reducing the number of columns that this buttons are using to one instead of two.

image

@xTudoS xTudoS mentioned this pull request Mar 12, 2024
@shakaran
Copy link
Collaborator

shakaran commented Apr 1, 2024

Does this still a issue or it will be updated or should I close it?

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.

None yet

4 participants