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

64bit notice #979

Merged
merged 2 commits into from
Jul 6, 2012
Merged

64bit notice #979

merged 2 commits into from
Jul 6, 2012

Conversation

skurfer
Copy link
Member

@skurfer skurfer commented Jul 5, 2012

This should show a one-time alert on start-up to let users know many plug-ins won’t work.

I wanted to have another button that took you to the plug-in prefs, but it wouldn’t work this early in the launch process.

We can alter wording if needed, and add localizations.

@skurfer
Copy link
Member Author

skurfer commented Jul 6, 2012

I'd suggest removing the word 'completely' from the parenthesis at the end,

Done.

Nice trick on the lastVersion < xxxx btw, good idea :)

I didn’t want to write additional stuff to the prefs since this should only be around for a couple of releases, so I found a way to use what’s already there. :-)

@pjrobertson
Copy link
Member

Even with the spotlight module installed? (The one being distributed by the plugins system)

If I disable the plugin I get weird behaviour, and no exceptions/crashes. As soon as I re-enable it they come back

@pjrobertson pjrobertson closed this Jul 6, 2012
@pjrobertson pjrobertson reopened this Jul 6, 2012
pjrobertson added a commit that referenced this pull request Jul 6, 2012
@pjrobertson pjrobertson merged commit 5553e88 into quicksilver:release Jul 6, 2012
@pjrobertson
Copy link
Member

Merged.

If you've got time, I think we need a new release :)

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.

None yet

2 participants