Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

fix(highlighting): Update JSX/TSX highlighting to more properly label tag delimiters#63138

Merged
mmanela merged 3 commits into
mainfrom
mmanela/improve-jsx-highlighting
Jun 7, 2024
Merged

fix(highlighting): Update JSX/TSX highlighting to more properly label tag delimiters#63138
mmanela merged 3 commits into
mainfrom
mmanela/improve-jsx-highlighting

Conversation

@mmanela

@mmanela mmanela commented Jun 6, 2024

Copy link
Copy Markdown
Contributor

Update JSX/TSX highlighting to identify tag delimiters properly.

Test plan

  • Unit tests pass
  • Inspect snap file

Changelog

  • Update highlighting of JSX/TSX Tags to correctly label delimiters

@cla-bot cla-bot Bot added the cla-signed label Jun 6, 2024
@mmanela mmanela changed the title Update JSX highlighting to more properly label tags fix(graph): Update JSX highlighting to more properly label tags Jun 6, 2024
@mmanela mmanela changed the title fix(graph): Update JSX highlighting to more properly label tags fix(search): Update JSX highlighting to more properly label tags Jun 6, 2024
@mmanela mmanela force-pushed the mmanela/improve-jsx-highlighting branch from d24ffd3 to 7904db3 Compare June 6, 2024 20:04
Base automatically changed from mmanela/GRAPH-205 to main June 6, 2024 20:16
@mmanela mmanela force-pushed the mmanela/improve-jsx-highlighting branch from 7904db3 to 442c64f Compare June 6, 2024 20:27
@mmanela mmanela force-pushed the mmanela/improve-jsx-highlighting branch from 442c64f to 597d44b Compare June 6, 2024 20:32
@mmanela mmanela requested a review from varungandhi-src June 6, 2024 20:39

@varungandhi-src varungandhi-src left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you planning to fix the CSS in this PR or a different PR? It still says TagDelimited instead of TagDelimiter on main.

(Also, I noticed you're using the search area for highlighting PRs, but it's technically incorrect, because this highlighting does not actually affect highlighting for search results, which go through a different code path.)

@mmanela

mmanela commented Jun 7, 2024

Copy link
Copy Markdown
Contributor Author

Thanks for calling that out, I can fix in this PR @varungandhi-src

@mmanela mmanela changed the title fix(search): Update JSX highlighting to more properly label tags fix(highlighting): Update JSX highlighting to more properly label tags Jun 7, 2024
@mmanela mmanela changed the title fix(highlighting): Update JSX highlighting to more properly label tags fix(highlighting): Update JSX/TSX highlighting to more properly label tags Jun 7, 2024
@mmanela mmanela changed the title fix(highlighting): Update JSX/TSX highlighting to more properly label tags fix(highlighting): Update JSX/TSX highlighting to more properly label tag delimiters Jun 7, 2024
@mmanela mmanela merged commit c9d391c into main Jun 7, 2024
@mmanela mmanela deleted the mmanela/improve-jsx-highlighting branch June 7, 2024 12:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants