Skip to content

Added highlighting for additional equal signs and package names - #29

Closed
jeremy-allen wants to merge 2 commits into
r-lib:mainfrom
jeremy-allen:twilight
Closed

jeremy-allen wants to merge 2 commits into
r-lib:mainfrom
jeremy-allen:twilight

Conversation

@jeremy-allen

Copy link
Copy Markdown

Hi Gabor. I started working on this for Tidy Dev Day. I am happy to take any guidance or feedback you offer.

Changes:
Added highlighting for package names and additional equal sign operator tokens. I could not get the package name tokens to highlight in light purple as the Twilight theme does in the editor, so I set package names to take on the same color as operators. This addresses issue #15.

Note:
This solution was suggested by Claude Code ✨. I tested many versions in the RStudio R console and terminal, and the highlights work in both, though the terminal still uses its own color for operators (green) versus what the Twilight theme can produce in the console (light purple). I am not sure if this meets user expectations.

…or tokens. I could not get the package name tokens to highlight in light purple as the Twilight theme does in the editor, so I set package names to take on the same color as operators. This addresses issue r-lib#15.
@gaborcsardi

Copy link
Copy Markdown
Member

Thank you and sorry for the long wait! Was about to merge it, but just noticed that a newer PR has a slightly broader fix, AFAICT.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants