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

Conversation

@zigomir
Copy link
Contributor

@zigomir zigomir commented Mar 28, 2015

When working, I love having minimal number of opened tabs. Thus, I close and open files all the time. This is kinda ingrained habit for me. I'm really used to flow of JetBrains products, because their fuzzy search remembers a query string of your previous search.

Lack of this option was always a deal breaker for me. But since Atom is so hackable, it was just few lines of code to make it work. What's your opinion on it?

fuzzy

Here's a gif where I show pain of typing same query string two times. And then, after enabling this option where you can see the preserved query string so I can open file instantly. From this example it might seem I'm not aware of undo close tab shortcut. This is not the case.

Better way to describe problem that this feature solves is that on our projects we have lots of files with similar names (models / views). When you open a model file you later on want to open view file too and you don't want to type same file name again.

@zigomir
Copy link
Contributor Author

zigomir commented Apr 5, 2015

Any feedback? Should this be an additional package?

@kevinsawicki
Copy link
Contributor

Any feedback? Should this be an additional package?

I like this and think it makes sense to include this as an option in this package like you have it now.

Would you mind adding a spec for this new option?

@kevinsawicki kevinsawicki self-assigned this Apr 6, 2015
@zigomir
Copy link
Contributor Author

zigomir commented Apr 7, 2015

@kevinsawicki thanks for feedback. I added specs.

@kevinsawicki
Copy link
Contributor

Sorry, looks like this no longer cleanly merges, would you mind rebasing/merging master into this branch?

# Conflicts:
#	lib/main.coffee
@zigomir
Copy link
Contributor Author

zigomir commented Apr 14, 2015

@kevinsawicki done

# Conflicts:
#	lib/main.coffee
@zigomir
Copy link
Contributor Author

zigomir commented Apr 30, 2015

@kevinsawicki did you change your mind about this feature?

@kevinsawicki
Copy link
Contributor

did you change your mind about this feature?

Nope, just lost track of this pull request, will re-review and merge shortly, thanks for the bump 👍

Copy link
Contributor

Choose a reason for hiding this comment

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

calling @getFilterQuery() returns the filter text without a trailing line number so pack:8 would become pack.

Did you want to preserve this info across opens? A trailing :NUMBER allows you to open a file to a specific line.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should ship this as-is to start, can always tweak it later.

kevinsawicki added a commit that referenced this pull request Apr 30, 2015
Add option to preserve last search query
@kevinsawicki kevinsawicki merged commit 01b5f41 into atom:master Apr 30, 2015
@kevinsawicki
Copy link
Contributor

Thanks for this :shipit:

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants