]> git.gir.st - VimFx.git/commit
Make button click depend on current mode. Fix #243
authorSimon Lydell <simon.lydell@gmail.com>
Mon, 30 Dec 2013 11:02:45 +0000 (12:02 +0100)
committerSimon Lydell <simon.lydell@gmail.com>
Mon, 30 Dec 2013 11:02:45 +0000 (12:02 +0100)
commitf3365cb8fe6b7a362db05ec94b84d0d07fa39e4d
tree8c7051967065da54b208467d7bca42a3bdd887c6
parent17b0f92859ee5729f9813b2ac1c35dd24566882e
Make button click depend on current mode. Fix #243

Both insert mode and disabled mode use the same greyed out icon for the
toolbar button. If you're in insert mode and click the button, we used to
disable the extension as normal. However, it felt like clicking the button
did nothing, because the icon didn't change. Even if it _would_ change, it
feels more natural if the extension is _enabled_ again at that point. So
now, if you're in insert mode and click the button, we enter normal mode.
(#243)

If the current site is blacklisted, clicking the button used to disable
the extension. Well, it was already disabled, but the button icon changed
from the blacklisted icon (red) to the disabled icon (grey). Clicking
again went back to blacklisted mode. That's pretty weird. Now we open the
menu popup when clicked instead, which will show the trash can icon which
is used to un-blacklist the site. Un-blacklisting directly when clicking
the button would be to go too far, since that can potentially remove more
blacklisting rules than the user wants to. Using the popup you have a
chance to review what will be removed when un-blacklisting.

To sum up, the button clicking is more intuitive now:

- If enabled: Disable
- Otherwise: Enable (almost)
12 files changed:
extension/bootstrap.coffee
extension/locale/de/vimfx.properties
extension/locale/el-GR/vimfx.properties
extension/locale/en-US/vimfx.properties
extension/locale/hu/vimfx.properties
extension/locale/id/vimfx.properties
extension/locale/nl/vimfx.properties
extension/locale/pl/vimfx.properties
extension/locale/ru/vimfx.properties
extension/locale/zh-CN/vimfx.properties
extension/packages/button.coffee
extension/packages/events.coffee
Imprint / Impressum