Comment #3 on issue 25957 by [email protected]: View and its sub  
menus should stay on clicking the option "Text Encoding"
http://code.google.com/p/chromium/issues/detail?id=25957

The problem is that giving Text Encoding an action (-commandDispatch:)  
makes it
selectable in that way. Removing it seems to break our deactivation of the  
menu item
(e.g., on the NTP).

*However*, deactivating Text Encoding is almost certainly the wrong thing  
to do on
the Mac; instead, we should be going through the submenu and disabling all  
its
entries (which is what every other Mac app does). This is presumably in  
Apple's HIG
somewhere, but I'm too lazy to look at the moment.

I probably have a bit-rotted change somewhere which does the latter, which  
I can
update, and then hopefully we can remove the -commandDispatch: action from  
the
menu item.

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to