Skip to content

Loading…

DRYing up your code #14

Merged
merged 1 commit into from

2 participants

@deiga

As you already had a method for this line method, it only made sense to use it :)

@deiga deiga DRYing up your code
As you already had a method for this line method, it only made sense to use it :)
802ed78
@netpro2k netpro2k merged commit 1a0f537 into netpro2k:master
@deiga deiga deleted the unknown repository branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jan 30, 2013
  1. @deiga

    DRYing up your code

    deiga committed
    As you already had a method for this line method, it only made sense to use it :)
Showing with 1 addition and 2 deletions.
  1. +1 −2 SublimeBlockCursor.py
View
3 SublimeBlockCursor.py
@@ -19,11 +19,10 @@ def show_block_cursor(self, view):
view.erase_regions('SublimeBlockCursorListener')
def on_selection_modified(self, view):
- is_widget = view.settings().get('is_widget')
is_vintage_mode = "Vintage" not in view.settings().get('ignored_packages', [])
command_mode = view.settings().get('command_mode')
- if is_widget or (is_vintage_mode and not command_mode):
+ if view_is_widget(view) or (is_vintage_mode and not command_mode):
view.erase_regions('SublimeBlockCursorListener')
return
Something went wrong with that request. Please try again.