Show indication when shortcuts are disabled

bustiiing / blazingstars

Automatically exported from code.google.com/p/blazingstars

0 stars 0 forks source link

Show indication when shortcuts are disabled #79

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago

It is possible to disable keyboard shortcuts (using the default shortcurt of 
alt-command-p), however it's not easy to tell whether or not they are disabled.

I propose either changing the Mac Os menu bar icon to include a red cross when 
disabled, or show a small (non-modal) popup in the center of the screen to 
indicate the change

Original issue reported on code.google.com by qzerv...@gmail.com on 4 Aug 2010 at 7:50

GoogleCodeExporter commented 8 years ago

This should be under "Enhancements" but I can't find the option to change it

Original comment by qzerv...@gmail.com on 4 Aug 2010 at 7:53

GoogleCodeExporter commented 8 years ago

Nice suggestion. I like the idea of indicating it in the menu bar icon.

Original comment by steve.mc...@gmail.com on 4 Aug 2010 at 12:13

Added labels: Type-Enhancement
Removed labels: Type-Defect

GoogleCodeExporter commented 8 years ago

Great idea.  As I've mentioned elsewhere, though (see the front page), I'm not 
developing BlazingStars right now. If anyone would like to implement this idea, 
I'll give them committing rights.

Original comment by steven.h...@gmail.com on 4 Aug 2010 at 10:11

GoogleCodeExporter commented 8 years ago

Hi Steve,

I'm a web and iPhone developer but haven't developed for OS X specifically. 
I'll see if I can figure it out in the next week or two, and if so I'll hit you 
up for commit access.

Original comment by qzerv...@gmail.com on 4 Aug 2010 at 10:51

GoogleCodeExporter commented 8 years ago

I've had a play around with the code and have managed to implement a working 
solution. I wasn't able to get the release build working (I guess it needs 
signing) so haven't been able to test things extensively.

Anyway, I'm happy to submit it back, just awaiting instructions.

Original comment by qzerv...@gmail.com on 5 Aug 2010 at 5:58

GoogleCodeExporter commented 8 years ago

Thanks for taking a look at it. I've given you committing rights for the svn, 
and you can go ahead and check in your changes. When I get a free moment, I'll 
do a release build. In the meantime, feel free to have a crack at other things 
if you like.

Original comment by steven.h...@gmail.com on 6 Aug 2010 at 1:07

GoogleCodeExporter commented 8 years ago

I've submitted this change in r121. Before I do anything else I'll await some 
feedback on this particular change ;)

Original comment by qzerv...@gmail.com on 10 Aug 2010 at 2:30

GoogleCodeExporter commented 8 years ago

It looks good, qzervaas!  Thanks for doing the work on this.  Sorry it took me 
so long to get the release out, but I've been pretty swamped.  If you feel like 
doing more work on the project, let me know, and I'll figure out a way for you 
to sign the releases.

Original comment by steven.h...@gmail.com on 19 Aug 2010 at 8:41

GoogleCodeExporter commented 8 years ago

Original comment by steven.h...@gmail.com on 19 Aug 2010 at 8:42

Changed state: Fixed