-
Notifications
You must be signed in to change notification settings - Fork 21
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
Standard theme font has changed from sans-serif to serif #25
Comments
Hi @sesh! Yes, this was an intentional change. What do you think about it? |
It's quite abrupt. When I jumped online to share a post on Saturday morning my immediate thought was that some CSS had failed to load. The "Times New Roman" in your face look wasn't what I was expecting. The old theme appealed to me in a bunch of ways. Perhaps a change like this is something that could be an option, or a different theme folks can choose (though, I realise that goes against some of your core goals). I'm sure I'll eventually come around to it but my initial reaction was that something was broken – perhaps not what you were aiming for. |
Hey @sesh,
|
this is a little unrelated, but will there be opportunities to add other CSS and theming customization? i appreciate the optional email notifications but would love an option to disable the RSS option and customize link colors. if not, no problem - thanks for the work put into this project thus far, it's fantastic |
Hey @yffr! We had a similar discussion at #3. Essentially, there are no plans to enable users to add custom CSS because that would mean that they would have to spend time deciding what style they want their blog to have. This is in contrast to our philosophy, which is very narrow. However, one can add a few allowed inline CSS styles (a limited set here), for example, when one makes a table and want to make everything fit in one line. |
Closing this because it was an intentional change, and additional themes have been added. Feel free to re-open it if you wanted to keep it around for some other reason. |
A recent change has changed the look and feel for the standard theme.
Checking in to see if this was intentional or not?
A quick bisect indicates this issue was introduced in this commit:
[d85bd95] move style into asset templates
Current rendering for me:
Previous design:
The text was updated successfully, but these errors were encountered: