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

feat(normal)!: remove the gs, g? normal command #28489

Closed
wants to merge 2 commits into from

Conversation

lewis6991
Copy link
Member

@lewis6991 lewis6991 commented Apr 24, 2024

Problem:

gs and g? are not useful normal commands.

Solution:

Remove them.

@lewis6991 lewis6991 changed the title feat(mappings)!: Remove gs feat(keymap)!: Remove the default gs keymap Apr 24, 2024
@zeertzjq

This comment was marked as resolved.

@lewis6991

This comment was marked as resolved.

@clason

This comment was marked as resolved.

Problem:

gs is a stupid normal command.

Solution:

Remove it!
@lewis6991 lewis6991 changed the title feat(keymap)!: Remove the default gs keymap feat(normal)!: Remove the gs normal command Apr 24, 2024
@zeertzjq zeertzjq changed the title feat(normal)!: Remove the gs normal command feat(normal)!: remove the gs normal command Apr 24, 2024
@lewis6991 lewis6991 changed the title feat(normal)!: remove the gs normal command feat(normal)!: remove the gs, g? normal command Apr 25, 2024
runtime/doc/vim_diff.txt Outdated Show resolved Hide resolved
runtime/doc/vim_diff.txt Outdated Show resolved Hide resolved
@zeertzjq zeertzjq added normal-mode text objects, normal commands and removed mappings key bindings labels Apr 25, 2024
:[N]sl[eep] [N][m] Do nothing for [N] seconds, or [N] milliseconds if [m]
was given. "gs" always uses seconds.
was given.
Default is one second. >
:sleep "sleep for one second
:5sleep "sleep for five seconds
:sleep 100m "sleep for 100 milliseconds
10gs "sleep for ten seconds
Copy link
Contributor

Choose a reason for hiding this comment

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

Line 525, which refers to gs, should go as well.

@lewis6991 lewis6991 closed this May 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change normal-mode text objects, normal commands
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants