From 3d7cb7305b6ffb75ab35a74b86f4d33784221ef5 Mon Sep 17 00:00:00 2001 From: Dan Guzek Date: Mon, 6 Apr 2015 21:21:23 -0400 Subject: [PATCH] change broadcasting of ThemePref changes I moved a MESSAGEMAN:Broadcast() call so that it only broadcasts the setting that has been saved. This seems more sensible to me, but it is possible I'm misinterpreting the original design here. Feel free to merge or not merge this commit. :) --- Themes/_fallback/Scripts/02 ThemePrefsRows.lua | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/Themes/_fallback/Scripts/02 ThemePrefsRows.lua b/Themes/_fallback/Scripts/02 ThemePrefsRows.lua index 21ad8973c2..d6e00381bb 100644 --- a/Themes/_fallback/Scripts/02 ThemePrefsRows.lua +++ b/Themes/_fallback/Scripts/02 ThemePrefsRows.lua @@ -57,8 +57,11 @@ local function DefaultSave( pref, choices, values ) return function(self, list, pn) for i=1, #choices do - if list[i] then ThemePrefs.Set( pref, values[i] ) break end - MESSAGEMAN:Broadcast( msg, params ) + if list[i] then + ThemePrefs.Set( pref, values[i] ) + MESSAGEMAN:Broadcast( msg, params ) + break + end end end end