Skip to content
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

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

Open
max-programming opened this issue Apr 15, 2021 · 6 comments
Labels
type: syntax Syntax highlighting

Comments

@max-programming
Copy link

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

@dominicegginton
Copy link
Contributor

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.

@simurai
Copy link
Contributor

simurai commented Apr 16, 2021

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

@dominicegginton
Copy link
Contributor

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.

@max-programming
Copy link
Author

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

@simurai
Copy link
Contributor

simurai commented Apr 21, 2021

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

@Bosphoramus
Copy link

Bosphoramus commented Aug 9, 2021

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

@simurai simurai added the type: syntax Syntax highlighting label Jan 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: syntax Syntax highlighting
Projects
None yet
Development

No branches or pull requests

4 participants