github-vscode-theme icon indicating copy to clipboard operation
github-vscode-theme copied to clipboard

GitHub Default and Dimmed themes don't have different color for components/elements

Open max-programming opened this issue 3 years ago • 6 comments

I just noticed that the normal GitHub Theme is working fine but the GitHub Dimmed and GitHub Default don't have a different colour for components. And the components are of the same colour as the HTML elements. (React, Vue, Svelte)

GitHub Dark

image

GitHub Dimmed

image

max-programming avatar Apr 15 '21 08:04 max-programming

I will look into this. this might be due to the new themes following GitHub.com where this is no difference between HTML element tags and component tags.

dominicegginton avatar Apr 15 '21 09:04 dominicegginton

Here a test on dotcom:

const Untitled = () => {
  return (
    <div>
      <Hello />
    </div>
  )
}

and yeah, it seems that there is no difference between HTML and component tags and both have the pl-ent class:

Screen Shot 2021-04-16 at 16 48 43

simurai avatar Apr 16 '21 07:04 simurai

I have opened #147 to get feedback on if this theme should strictly stay consistent to the GitHub.com syntax highlighting or implement extra syntax highlighting features such as this. Ultimately it's up the the GitHub primer team.

dominicegginton avatar Apr 16 '21 08:04 dominicegginton

I personally think it's better to have a different colour for components as it also makes this theme competitive to others

max-programming avatar Apr 16 '21 12:04 max-programming

@dominicegginton Thanks for opening up a discussion. I'll also bring it up in our sync to discuss.

simurai avatar Apr 21 '21 08:04 simurai

Yes please :( Bothers me way too much when reading code, components should have a different color.

Bosphoramus avatar Aug 09 '21 11:08 Bosphoramus