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

Clean up CSS from Inkscape cruft #143

Merged
merged 1 commit into from
Jul 15, 2021
Merged

Conversation

waldyrious
Copy link
Contributor

The changes in #140 brought a lot of inert/redundant CSS from Inkscape's default export.
This PR cleans them up, and fixes the formatting of the remaining rules to be consistent with the rest of the file.
These changes should produce no visible differences to the rendered page.

The changes in c773e89 brought a lot of inert/redundant CSS from Inkscape's default export.
This commit cleans them up, and fixes the formatting of the remaining rules to be consistent with the rest of the file.
These changes should produce no visible differences to the rendered page.
@waldyrious
Copy link
Contributor Author

I would also like to propose removing this line:

line-height: 1;

since it makes the lines be a little too cramped, and the browser default gives a little more breathing room.

With line-height: 1 Browser default
Screen Shot 2021-06-16 at 11 10 51 Screen Shot 2021-06-16 at 11 10 43

@Genora51 Genora51 merged commit adf0886 into 8values:master Jul 15, 2021
@waldyrious waldyrious deleted the patch-1 branch July 15, 2021 21:28
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