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

PrismJS-highlighted TOML table is wrong, due to Tailwind #320

Closed
applejag opened this issue Jul 23, 2022 · 0 comments · Fixed by #396
Closed

PrismJS-highlighted TOML table is wrong, due to Tailwind #320

applejag opened this issue Jul 23, 2022 · 0 comments · Fixed by #396
Labels
bug Something isn't working Design UI design

Comments

@applejag
Copy link
Contributor

Describe the bug
Tailwind styling messes up PrismJS-highlighted TOML code

To Reproduce

  1. Enable PrismJS

  2. Write TOML snippet in note, e.g:

    ```toml
    [root]
    foo = "bar"
    ```
  3. See invalid formatting:

    image

Expected behavior

Should be rendered as:

image

Additional context

The PrismJS generated HTML looks like this:

image

With the culprit being the table class that Tailwind also defines.

PrismJS defers to overriding the Tailwind styling, as that's what messing this up: PrismJS/prism#2443 (comment)

But how do I change the Tailwind class in Emanote?

@applejag applejag added the bug Something isn't working label Jul 23, 2022
@applejag applejag mentioned this issue Jul 25, 2022
@srid srid added the Design UI design label Aug 1, 2022
kukimik added a commit to kukimik/emanote that referenced this issue Dec 8, 2022
kukimik added a commit to kukimik/emanote that referenced this issue Dec 9, 2022
@srid srid closed this as completed in #396 Dec 24, 2022
srid added a commit that referenced this issue Dec 24, 2022
shivaraj-bh pushed a commit to shivaraj-bh/emanote that referenced this issue Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Design UI design
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants