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 and improve some actions #6287

Merged
merged 6 commits into from
Sep 15, 2017
Merged

Fix and improve some actions #6287

merged 6 commits into from
Sep 15, 2017

Conversation

AndrienkoAleksandr
Copy link
Contributor

What does this PR do?

Fix scroll to line for OpenFileAction.java. Fix broken "Link with editor" button and RevealResourceAction.java . Improve RevealResourceAction.java : added property "path" to reveal node. Add method "addSelectionHandler" to ProjectExplorerTree.

What issues does this PR fix or reference?

#6285
#6286
codenvy/redhat#30

Changelog

  • Fix scroll to line for OpenFileAction.java.
  • Fix broken "Link with editor" button and RevealResourceAction.java .
  • Improve RevealResourceAction.java : add to this action property "path" to reveal node.
  • Apply to ProjectExplorerTree method addSelectionHandler.

Release Notes

  • Fix scroll to line for OpenFileAction.java.
  • Fix broken "Link with editor" button and RevealResourceAction.java .
  • Improve RevealResourceAction.java : add to this action property "path" to reveal node.
  • Apply to ProjectExplorerTree method addSelectionHandler.

Docs PR

Don't need: little enhancements and bug fixes.

Signed-off-by: Aleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Aleksandr Andriienko <[email protected]>
new EditorAgent.OpenEditorCallback() {
@Override
public void onEditorOpened(EditorPartPresenter editor) {
new Timer() {
Copy link
Contributor

Choose a reason for hiding this comment

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

Move

 new Timer() {
...

inside scrollToLine method

Signed-off-by: Aleksandr Andriienko <[email protected]>
Signed-off-by: Aleksandr Andriienko <[email protected]>
@codenvy-ci
Copy link

@tolusha tolusha merged commit 20d5966 into master Sep 15, 2017
@tolusha tolusha deleted the FixSomeActions branch September 15, 2017 15:44
@AndrienkoAleksandr AndrienkoAleksandr added this to the 5.18.0 milestone Sep 16, 2017
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.

5 participants