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

Trim vim system commands #69

Merged
merged 1 commit into from
Sep 30, 2024
Merged

Conversation

phantomwhale
Copy link
Contributor

When vim executes a system command it gets the full string response, including the trailing newline character

This newline needs to be trimmed off the end of the return value before doing a string comparison with the value of the existing colorscheme

This prevents the focus handler resetting the colorscheme every time it's called, but rather just when an actual change has occurred

When vim executes a system command it gets the full string response, including the trailing newline character

This newline needs to be trimmed off the end of the return value before doing a string comparison with the value of the existing colorscheme

This prevents the focus handler resetting the colorscheme every time it's called, but rather just when an actual change has occurred
@phantomwhale phantomwhale requested a review from a team as a code owner September 30, 2024 01:48
@phantomwhale phantomwhale requested review from JamyGolden and removed request for a team September 30, 2024 01:48
Copy link
Member

@JamyGolden JamyGolden left a comment

Choose a reason for hiding this comment

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

Thanks!

@JamyGolden JamyGolden merged commit 1f9b545 into tinted-theming:main Sep 30, 2024
5 checks passed
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