Skip to content

Conversation

@tuurep
Copy link

@tuurep tuurep commented Nov 3, 2025

Resolves #20

@tuurep
Copy link
Author

tuurep commented Nov 4, 2025

Found out this breaks visual mode in weird ways, and sometimes shows this error message:

Error detected while processing function wordmotion#object:
line   16:
E471: Argument required: normal!
Press ENTER or type command to continue

execute 'normal!' visualmode()

@tuurep tuurep force-pushed the fix-noisy-dot-repeats branch from 710c722 to 1c8dc3d Compare November 4, 2025 14:33
@tuurep
Copy link
Author

tuurep commented Nov 4, 2025

Easy immediate fix would be to use a <cmd> mapping only if it's not visual mode, force pushed that change above.

@tuurep tuurep force-pushed the fix-noisy-dot-repeats branch from 1c8dc3d to 44b6f43 Compare November 4, 2025 15:43
@tuurep
Copy link
Author

tuurep commented Nov 4, 2025

Style change: converted space-indents to tabs to be consistent with the rest

@tuurep tuurep force-pushed the fix-noisy-dot-repeats branch from 44b6f43 to e88977e Compare November 5, 2025 18:55
@tuurep
Copy link
Author

tuurep commented Nov 5, 2025

Obsessive style change: since you never capitalize your comments, I wanted to follow suit

Now there's nothing I want to add here

I suppose it would be possible to make the <cmd> mappings work with visual mode too but it would be a bigger refactor

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.

Dot repeat of ciw emits 'AOrInnerWordMotion' output if 'set noshowmode' is set.

1 participant