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

Tidy up the exclusion/inclusion token field for catalog entries #865

Merged
merged 2 commits into from May 4, 2012

Conversation

pjrobertson
Copy link
Member

@pjrobertson pjrobertson commented May 4, 2012

  • Update the .xib to 10.6
  • Show nice types for strings beginning with a '.' e.g. show '.jpg' as 'JPEG image' (previously would stay as '.jpg')
  • Rescan the current item if any of the settings change. Gives immediate response of the 'contents' number when e.g. the depth or types fields are changed

I initially wanted to fix #335, but it appears to be working for me now. Can somebody else confirm.
Instead I just made the current system even better ;)

* Update the .xib to 10.6
* Show nice types for strings beginning with a '.' e.g. show '.jpg' as 'JPEG image' (previously would stay as '.jpg')
* Rescan the current item if any of the settings change. Gives immediate response of the 'contents' number when e.g. the depth or types fields are changed
@skurfer
Copy link
Member

@skurfer skurfer commented May 4, 2012

Cool. I love the real-time updates. All seems to work.

I wish we could make adding folders more obvious. Typing 'fold' is not that intuitive. Any ideas there?

skurfer added a commit that referenced this issue May 4, 2012
Tidy up the exclusion/inclusion token field for catalog entries
@skurfer skurfer merged commit 08c699c into quicksilver:master May 4, 2012
@lovequicksilver
Copy link

@lovequicksilver lovequicksilver commented May 9, 2012

It's a lot more obvious that the exclusions are working, the label changes show the user that QS has recognised the input. :)

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.

3 participants