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

fix(api/#163): Part 1: Add an API to get pending operator information #218

Merged
merged 3 commits into from
Aug 26, 2020

Conversation

bryphe
Copy link
Member

@bryphe bryphe commented Aug 25, 2020

This adds an API to get the current pending-operator information (what operator is pending, any associated register, and count). This is part of a fix for #163 - there are still some items that would be useful that this doesn't provide (ie, typed characters prior to an operator becoming active).

bryphe added a commit to onivim/oni2 that referenced this pull request Aug 26, 2020
Some prep work to address #2357 - this plumbs through some additional context for modes, so that we can display it in the status bar and elsewhere. This updates the `Mode` variant to be able to handle contextual state (and updates all the downstream dependencies).

With this change - we'll be able to integrate it with onivim/libvim#218 so we can add some operator pending info for #2357
@bryphe bryphe merged commit 853bb49 into master Aug 26, 2020
@bryphe bryphe deleted the fix/api/163/1-pending-op-state branch August 26, 2020 19:46
bryphe added a commit to onivim/oni2 that referenced this pull request Aug 26, 2020
__Issue:__ As described in #2357 , we weren't showing any context around a pending operator in the status bar - this is really a step backward.

__Fix:__ This picks up onivim/libvim#218 so that we get some context around the current pending operator, and we show it in now in the statusbar:

![2020-08-26 12 56 23](https://user-images.githubusercontent.com/13532591/91350541-b02e6880-e79b-11ea-8015-2670a7bf690f.gif)

This fixes the operator-pending part of #2357 - we're missing showing the in-progress count of a command, though.
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.

1 participant