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

CHE-2922: Fix Ctrl + S hotKey for Emacs. Add simple UI for incremental search. #3260

Merged
merged 1 commit into from
Jan 16, 2017

Conversation

AndrienkoAleksandr
Copy link
Contributor

What does this PR do?

Fix Ctrl + S hotKey for Emacs. Add simple UI for incremental search. This UI used for straight incremental find "Ctrl + S" and for revers incremental find "Ctrl + R" on the EMACS keybinding.

What issues does this PR fix or reference?

#2992

Previous behavior

Ctrl + S hotkey combination doesn't work. And reverse incremental find (Ctrl + R) has no UI for displaying user typing.

New behavior

Ctrl + S and Strl + R works by EMACS keybinding and user typing displays on the simple special UI.
@vparfonov @azatsarynnyy @slemeur @tolusha review please.
incrementalfind
incrementalfind2


/**
* Create StatusMessageReporterOverlay for delegation status message
* reporting to class class {@link StatusMessageReporter}.
Copy link
Member

Choose a reason for hiding this comment

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

class class

@codenvy-ci
Copy link

@codenvy-ci
Copy link

@codenvy-ci
Copy link

…l search.

Signed-off-by: Aleksandr Andrienko <aandrienko@codenvy.com>
@codenvy-ci
Copy link

Build # 1647 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/1647/ to view the results.

JPinkney pushed a commit to JPinkney/che that referenced this pull request Aug 17, 2017
…l search. (eclipse-che#3260)

Signed-off-by: Aleksandr Andrienko <aandrienko@codenvy.com>
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.

4 participants