The changes introduced in #60 should be reverted. The official Solarized Colors (which people expect to install when selecting a theme titled "Solarized Dark") can be found in the altercation/solarized repo.
I can understand the motives of some people to use a different, more readable color for Ansi 8 Color. There has been much talk about this, to the point that some have even reported it as a bug to altercation/solarized. That's why my recommendation would not be to just revert #60. Instead, we should have a theme called "Solarized Dark" track the same .itermcolors as can be found in the original repo, and we should introduce a new theme with a name like "Solarized Dark - Patched ANSI 8". This way people could choose, and not be surprised if they're expecting the defaults.
I'd be more than happy to open a PR for this if you'd support it.
The changes introduced in #60 should be reverted. The official Solarized Colors (which people expect to install when selecting a theme titled "Solarized Dark") can be found in the altercation/solarized repo.
I can understand the motives of some people to use a different, more readable color for
Ansi 8 Color. There has been much talk about this, to the point that some have even reported it as a bug to altercation/solarized. That's why my recommendation would not be to just revert #60. Instead, we should have a theme called "Solarized Dark" track the same.itermcolorsas can be found in the original repo, and we should introduce a new theme with a name like "Solarized Dark - Patched ANSI 8". This way people could choose, and not be surprised if they're expecting the defaults.I'd be more than happy to open a PR for this if you'd support it.