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.

Sign up for GitHub

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

Jump to bottom

Replace path range instead of word range for some modes #78

Merged
merged 4 commits into from
Jan 29, 2023
Merged

Replace path range instead of word range for some modes #78

merged 4 commits into from
Jan 29, 2023

Conversation

kLabz
Copy link
Contributor

@kLabz kLabz commented Jan 25, 2023 •
edited
Loading

#77 was breaking new B|<String, String>(); use case:
Recording 2023-01-25 at 11 06 23

With this version:
Recording 2023-01-25 at 11 08 19
Recording 2023-01-25 at 11 09 19

Update: I noticed some other cases had the original issue:
Recording 2023-01-25 at 11 27 17

So I added TypeHint, TypeRelation, and also (but not recorded) Implements and Extends to modes using path replace range:
Recording 2023-01-25 at 11 24 36

@kLabz kLabz changed the title Complete path instead of word for new Replace path range instead of word range for some modes Jan 25, 2023
@Gama11 Gama11 merged commit d55c50b into vshaxe:master Jan 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@kLabz @Gama11