-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
🐛 Fix highlight alignment in emojified gin-log #117
Conversation
WalkthroughThe recent update introduces a conditional feature in the Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChat with CodeRabbit Bot (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files selected for processing (1)
- denops/gin/command/buffer/edit.ts (1 hunks)
Additional comments: 2
denops/gin/command/buffer/edit.ts (2)
100-100: The conditional application of the
emojify
function to thecontent
array is a logical way to handle the emojification based on theoptions.emojify
flag. This should ensure that the text is only emojified when the option is enabled, which aligns with the PR objectives.105-105: The replacement of the
trimmed
array with the emojifiedcontent
is crucial for the fix. However, it's important to verify that this change does not affect other parts of the code wheretrimmed
might be used in its original, unemojified form.Verification successful
The verification process has confirmed that the
trimmed
variable is used immediately after its definition and conditional modification within theexec
function. There is no evidence from the provided context thattrimmed
is used elsewhere in a way that would conflict with the change to applyemojify
conditionally. The change appears to be safely contained within the local scope of theexec
function.Scripts Executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Verify that the 'trimmed' variable is not used elsewhere in a way that requires it to be unemojified. rg --type ts "trimmed" --context 5 "denops/gin/command/buffer/edit.ts"Length of output: 424
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I'll merge it once the CI passed (let me know if I forget) |
@lambdalisue Could you check the CI results? |
This fixes highlight alignment in emojified gin-log buffer.
Before fix: Misalignment of highlights in emojified text.
After fix: Correct alignment of highlights in emojified text.
Summary by CodeRabbit