Issue #147 - Log out confirmation #231

Merged
merged 2 commits into from Jan 12, 2012

Projects

None yet

2 participants

@wesleywiser
Contributor

Per #147, I added a confirmation message to the logout menu button and changed the Confirm Quit preference to include the logout option.

@talklittle

I think this would be a forward slash / rather than backslash \

@talklittle

this comment should say "logout" rather than "quit"

@talklittle

the title should be a logout string resource rather than a quit string resource

@talklittle

thanks for breaking logout() into a separate method

@wesleywiser
Contributor

@talklittle I've made the changes you've suggested. Good catch on the resources!

@talklittle
Owner

Great, thanks!

@talklittle talklittle merged commit 9619a65 into talklittle:master Jan 12, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment