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

(#713) Change default icon font size #96

Closed
wants to merge 379 commits into from

Conversation

justinsunho
Copy link

liferay#713

Not sure if it was better to do this or to change height and width to 16px in .btn-monospaced.btn .lexicon-icon

@jonmak08

Carlos Lancha and others added 30 commits February 26, 2018 23:47
Site: Grid add more detailed docs about various containers
Rename hideLabel params to showLabel for consistency
carloslancha and others added 28 commits March 15, 2018 10:28
…onsMenus

Fixes the quick action style in ClayManagementToolbar | Fixes liferay#738
Fix tag type JSDOC when use multiple types | Fixes liferay#736
…eation-menu

Improve Creation Menu items logic | Fixes liferay#749
@pat270
Copy link
Owner

pat270 commented Apr 7, 2018

New Pull: liferay#779

Hey Justin,

This is the right way to go about it. Generally it's variables first, then add css.

I just have some suggestions for the commit message. If you use any of these keywords https://help.github.com/articles/closing-issues-using-keywords/ along with the issue number github will automatically close the issue when it gets merged into master. I also use Breaking, New, Update, Site at the beginning of my commit messages to denote what kind of change it is.

Thanks!

@pat270 pat270 closed this Apr 7, 2018
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.

10 participants