Skip to content

Improves "Inline Blame" step of the walkthrough (#4860)#4861

Merged
sergeibbb merged 3 commits intomainfrom
feature/4860-improve-blame-walkthrough-step-PLG-121
Jan 12, 2026
Merged

Improves "Inline Blame" step of the walkthrough (#4860)#4861
sergeibbb merged 3 commits intomainfrom
feature/4860-improve-blame-walkthrough-step-PLG-121

Conversation

@sergeibbb
Copy link
Member

@sergeibbb sergeibbb commented Jan 9, 2026 •
edited
Loading

Fixes #4860 (PLG-121)

Description

  • Completes gitBlame step of the Walkthrough by toggling "line-blame" rather than by "file-blame" (yes, it keeps completing the step on toggling).
  • Adds completion on viewing the section in the Walkthrough.
  • Changes CTA to opening the settings and updates the button label accordingly.
image

It leaves one problem with completion that is discussed in this thread.

Checklist

  • I have followed the guidelines in the Contributing document
  • My changes follow the coding style of this project
  • My changes build without any errors or warnings
  • My changes have been formatted and linted
  • My changes include any required corresponding changes to the documentation (including CHANGELOG.md and README.md)
  • My changes have been rebased and squashed to the minimal number (typically 1) of relevant commits
  • My changes have a descriptive commit message with a short title, including a Fixes $XXX - or Closes #XXX - prefix to auto-close the issue that your PR addresses

sergeibbb added a commit that referenced this pull request Jan 9, 2026
@sergeibbb
Completes gitBlame step of the Walkthrough by toggling "line-blame"…
4c53838 
… rather than by "file-blame"

PLG-121 #4860 #4861
@sergeibbb sergeibbb force-pushed the feature/4860-improve-blame-walkthrough-step-PLG-121 branch from 24b81e5 to 4c53838 Compare January 9, 2026 22:00
sergeibbb added a commit that referenced this pull request Jan 9, 2026
@sergeibbb
Completes gitBlame step of the Walkthrough by toggling "line-blame"
f49e6f4 
rather than by "file-blame"
PLG-121 #4860 #4861
@sergeibbb sergeibbb force-pushed the feature/4860-improve-blame-walkthrough-step-PLG-121 branch from 4c53838 to 9866617 Compare January 9, 2026 22:31
sergeibbb added a commit that referenced this pull request Jan 9, 2026
@sergeibbb
Completes "InlineBlame" step of walkthrough simply by being visited b…
312406f 
…y user

PLG-121 #4860 #4861
sergeibbb added a commit that referenced this pull request Jan 9, 2026
@sergeibbb
Changes CTA of "Inline Blame" section of the Walkthrough to open sett…
9866617 
…ings

The button in the "Inline Blame" section of the Walkthrough opens to the
"Current Line Blame" section in the GitLens settings and labeled
accordingly.
PLG-121 #4860 #4861
@sergeibbb sergeibbb marked this pull request as ready for review January 9, 2026 22:35
@sergeibbb sergeibbb added this to the 17.9 milestone Jan 9, 2026
@sergeibbb sergeibbb requested review from d13 and ramin-t January 9, 2026 22:36
@augmentcode
Copy link

augmentcode bot commented Jan 9, 2026

🤖 Augment PR Summary

Summary: Updates the Welcome walkthrough “Inline Blame” step to target line-level blame and adds a direct “Configure Inline Blame” CTA that opens the Settings webview at the inline-blame section.

Why: Improves walkthrough completion reliability (tracking toggleLineBlame usage and step selection) and better guides users to the relevant configuration.

🤖 Was this summary useful? React with 👍 or 👎

augmentcode[bot]
augmentcode bot reviewed Jan 9, 2026
View reviewed changes
Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

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

Review completed. 1 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

d13
d13 approved these changes Jan 12, 2026
View reviewed changes
Copy link
Member

@d13 d13 left a comment

Choose a reason for hiding this comment

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

As discussed, we'll use both line and file blame commands. Then, good to go.

sergeibbb added a commit that referenced this pull request Jan 12, 2026
@sergeibbb
Completes gitBlame step of the Walkthrough by toggling "line-blame"
01368a7 
rather than by "file-blame"
PLG-121 #4860 #4861
sergeibbb added a commit that referenced this pull request Jan 12, 2026
@sergeibbb
Completes "InlineBlame" step of walkthrough simply by being visited b…
d556a3b 
…y user

PLG-121 #4860 #4861
sergeibbb added a commit that referenced this pull request Jan 12, 2026
@sergeibbb
Changes CTA of "Inline Blame" section of the Walkthrough to open sett…
89ffa9e 
…ings

The button in the "Inline Blame" section of the Walkthrough opens to the
"Current Line Blame" section in the GitLens settings and labeled
accordingly.
PLG-121 #4860 #4861
@sergeibbb sergeibbb force-pushed the feature/4860-improve-blame-walkthrough-step-PLG-121 branch from 9866617 to 89ffa9e Compare January 12, 2026 16:11
@sergeibbb
Completes gitBlame step of the Walkthrough by toggling "line-blame"
72b813a 
rather than by "file-blame"
PLG-121 #4860 #4861
@sergeibbb
Completes "InlineBlame" step of walkthrough simply by being visited b…
14c615a 
…y user

PLG-121 #4860 #4861
@sergeibbb
Changes CTA of "Inline Blame" section of the Walkthrough to open sett…
85f8012 
…ings

The button in the "Inline Blame" section of the Walkthrough opens to the
"Current Line Blame" section in the GitLens settings and labeled
accordingly.
PLG-121 #4860 #4861
@sergeibbb sergeibbb force-pushed the feature/4860-improve-blame-walkthrough-step-PLG-121 branch from 89ffa9e to 85f8012 Compare January 12, 2026 16:13
@sergeibbb sergeibbb merged commit 8417c95 into main Jan 12, 2026
4 checks passed
sergeibbb added a commit that referenced this pull request Jan 12, 2026
@sergeibbb
Completes gitBlame step of the Walkthrough by toggling "line-blame"
d7e55d8 
rather than by "file-blame"
PLG-121 #4860 #4861
sergeibbb added a commit that referenced this pull request Jan 12, 2026
@sergeibbb
Completes "InlineBlame" step of walkthrough simply by being visited b…
d5d98ca 
…y user

PLG-121 #4860 #4861
@sergeibbb sergeibbb deleted the feature/4860-improve-blame-walkthrough-step-PLG-121 branch January 12, 2026 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@d13 d13 d13 approved these changes

@ramin-t ramin-t Awaiting requested review from ramin-t

+1 more reviewer

@augmentcode augmentcode[bot] augmentcode[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

17.9

Development

Successfully merging this pull request may close these issues.

Improve "Inline Blame" step of the walkthrough

2 participants

@sergeibbb @d13