search

Bluemoondragon07 / chime-theme

A lightweight theme for Obsidian designed to be simple yet highly customizable.
MIT License
60 stars 4 forks source link

Remove trailing whitespace from theme file #19

Closed FireIsGood closed 10 months ago

FireIsGood commented 10 months ago

There were many spots with trailing whitespace.

I have specifically not modified spacing or removed empty lines at the ends of rules as I am unsure whether you intend to refactor the tab sizes, empty lines, and other inconsistencies later. You can confirm this by checking that the additions and deletions are the same.

You may want to consider an opinionated linter extension like prettier for a CSS file of this size, as manual fixes past these whitespaces would be very difficult. Looking through the code, I don't see a specific personal style being used for everything (e.g. spaces after comments and between different rules) so an opinionated linter may be good compared to something more customization.

Further linting—changing spaces to be 4 may change lines like lines 2607 to 2631 when I am unsure of the intended tab sizes or ideas about spacing between rules—may not match with your style, so I have done the least opinionated linting.

Bluemoondragon07 commented 10 months ago

Thank you. I will take a look at prettier; 15-year-old me was a messy coder, haha.