Skip to content

Commit

Permalink
s/trigger/buskill_trigger in Config
Browse files Browse the repository at this point in the history
This commit replaces the Config item name "buskill" with "buskill_trigger"

Apparently I silently renamed this option in the Config file some months ago when I was switching the Settings screen to use RecycleView, for the font accessibility feature

 * #55 (comment)

I don't know exactly why I made the change, but apparently I already updated it in other places, except here. The result: a bug where the first time you "arm" BusKill, it always defaulted to the 'lock-screen' trigger, instead of the setting the user already set it to.

 * #77 (comment)
  • Loading branch information
maltfield committed Jul 29, 2024
1 parent 44b6de8 commit 924ca98
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/packages/buskill/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -944,8 +944,8 @@ def toggle(self):
# set the default trigger to what's defined in the config file
self.config = configparser.ConfigParser()
self.config.read( self.CONF_FILE )
if self.config.has_option('buskill', 'trigger'):
trigger = self.config.get('buskill', 'trigger')
if self.config.has_option('buskill', 'buskill_trigger'):
trigger = self.config.get('buskill', 'buskill_trigger')
else:
trigger = 'lock-screen'
self.set_trigger( trigger )
Expand Down

0 comments on commit 924ca98

Please sign in to comment.