Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Commit

Permalink
Show the find editor on cmd-e
Browse files Browse the repository at this point in the history
  • Loading branch information
@benogle
benogle committed Dec 9, 2013
1 parent 021a683 commit 2d813e9
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions lib/find.coffee
View file
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,11 @@ module.exports =
@projectFindView.attach()
@projectFindView.findInCurrentlySelectedDirectory($(e.target))

atom.workspaceView.command 'find-and-replace:use-selection-as-find-pattern', =>
return if @projectFindView.isOnDom() or @findView.isOnDom()
@projectFindView.detach()
@findView.showFind()

atom.workspaceView.command 'find-and-replace:show', =>
@projectFindView.detach()
@findView.showFind()
Expand Down

3 comments on commit 2d813e9

@probablycorey
Copy link

Choose a reason for hiding this comment

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

I like how cmd-e doesn't pop-up the find and replace view. Do other editors show the find and replace view when cmd-e is pressed?

@benogle
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Can we try it out for a bit with this behavior? If you hate it, we can take it out. My concern is that people dont know it worked. There is sort of an expectation to show the find markers when you hit cmd+e

@probablycorey
Copy link

Choose a reason for hiding this comment

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

👍

Please sign in to comment.