insanum / obsidian_nord

A nord theme for Obsidian
MIT License
228 stars 62 forks source link

The table is not very clear, can you add a color scheme for the table? #26

Closed lin0kin closed 2 years ago

lin0kin commented 2 years ago

like ITS theme , it's adding customer color scheme for table

thx

insanum commented 2 years ago

Well to me tables look ok. Could you provide a pull request with changes to tables that you'd like to see? Then we can decide if the changes are worthwhile and enhance the theme overall.

lin0kin commented 2 years ago

Well to me tables look ok. Could you provide a pull request with changes to tables that you'd like to see? Then we can decide if the changes are worthwhile and enhance the theme overall.

Sorry to reply to you only now

This is the look of a css I'm using

Snipaste_2021-12-18_21-10-35

This is a table style from another theme

Snipaste_2021-12-18_21-09-57
vii33 commented 2 years ago

Hi, I created a pull request for this: https://github.com/insanum/obsidian_nord/pull/36

I went for a bit more subtle colors, though. Reason being that tables are normally embedded inside text areas and therefore shoud not stand out too much (also the headlines are already quite colorful in this theme).

Original | New theme-dark

lin0kin commented 2 years ago

Hi, I created a pull request for this: #36

I went for a bit more subtle colors, though. Reason being that tables are normally embedded inside text areas and therefore shoud not stand out too much (also the headlines are already quite colorful in this theme).

Original | New theme-dark

this looks awesome , bro