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

Printers should have useful names #475

Closed
skurfer opened this issue Aug 29, 2011 · 4 comments · Fixed by #793
Closed

Printers should have useful names #475

skurfer opened this issue Aug 29, 2011 · 4 comments · Fixed by #793

Comments

@skurfer
Copy link
Member

skurfer commented Aug 29, 2011

In the catalog preferences, under System → Devices → Printers, all of the indexed printers are named “PrinterProxy”. Ideally, they’d be assigned the name in System Preferences.

@pjrobertson
Copy link
Member

Confirmed on Snow Leopard (assuming you're using Lion?)

It should also remove duplicates. I have the same printer listed as a 'printer' and 'scanner' in the Print & Fax prefs, it shows up twice in QS's catalog.q

@ghost ghost assigned pjrobertson Nov 16, 2011
@pjrobertson
Copy link
Member

At the moment, a basic file scanner is used to populate this catalog list.

I don't understand why the names PrinterProxy are used, as a basic file object is created which initially has the name of the file.

@skurfer
Copy link
Member Author

skurfer commented Nov 17, 2011

Yeah, I just checked and the string “PrinterProxy” doesn’t appear anywhere in the code. But here’s a clue:

% mdls ~/Library/Printers/*.app | grep PrinterProxy
kMDItemCFBundleIdentifier      = "com.apple.print.PrinterProxy"
kMDItemCFBundleIdentifier      = "com.apple.print.PrinterProxy"
kMDItemCFBundleIdentifier      = "com.apple.print.PrinterProxy"
kMDItemCFBundleIdentifier      = "com.apple.print.PrinterProxy"
kMDItemCFBundleIdentifier      = "com.apple.print.PrinterProxy"

@pjrobertson
Copy link
Member

Found the problem, should also improve the names of various other apps. See #793

pjrobertson pushed a commit to pjrobertson/Quicksilver that referenced this issue Apr 24, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants