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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[POC] Remove declaration of CSS Custom Properties #121

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

Conversation

edalgrin
Copy link
Owner

@edalgrin edalgrin commented Feb 25, 2021

Theory: removing the declaration of custom properties we will reduce the weight of the main.css file

before: 233 KB
after: 234 KB

clearly it's not working 馃槃

@liferay-continuous-integration
Copy link
Collaborator

To conserve resources, the PR Tester does not automatically run for every pull.

If your code changes were already tested in another pull, reference that pull in this pull so the test results can be analyzed.

If your pull was never tested, comment "ci:test" to run the PR Tester for this pull.

@edalgrin edalgrin changed the title remove declared css custom properties [POC] Remove declaration of CSS Custom Properties Mar 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants