diff options
author | Aimee Linden <aimee@lindenlab.com> | 2010-05-27 13:58:11 +0100 |
---|---|---|
committer | Aimee Linden <aimee@lindenlab.com> | 2010-05-27 13:58:11 +0100 |
commit | 9fa2447c59f6f0ff44b2127576af91f9b864f7aa (patch) | |
tree | 305e3620f1343c3164cc012d5ce29ecc2e3c082e /indra | |
parent | 617a80ee81cd1c8514594854036f6f543a571e95 (diff) |
EXT-7138 WIP Kill the magic column numbers!
Remove hardcoded font selections.
Diffstat (limited to 'indra')
-rw-r--r-- | indra/newview/llfloatervoiceeffect.cpp | 23 | ||||
-rw-r--r-- | indra/newview/llfloatervoiceeffect.h | 6 |
2 files changed, 16 insertions, 13 deletions
diff --git a/indra/newview/llfloatervoiceeffect.cpp b/indra/newview/llfloatervoiceeffect.cpp index 22392dca8b..2243f0bf9a 100644 --- a/indra/newview/llfloatervoiceeffect.cpp +++ b/indra/newview/llfloatervoiceeffect.cpp @@ -132,10 +132,9 @@ void LLFloaterVoiceEffect::refreshEffectList() LLSD element; element["id"] = LLUUID::null; - element["columns"][0]["column"] = "name"; - element["columns"][0]["value"] = getString("no_voice_effect"); - element["columns"][0]["font"]["name"] = "SANSSERIF"; - element["columns"][0]["font"]["style"] = "ITALIC"; + element["columns"][NAME_COLUMN]["column"] = "name"; + element["columns"][NAME_COLUMN]["value"] = getString("no_voice_effect"); + element["columns"][NAME_COLUMN]["font"]["style"] = "BOLD"; LLScrollListItem* sl_item = mVoiceEffectList->addElement(element, ADD_BOTTOM); // *HACK: Copied from llfloatergesture.cpp : ["font"]["style"] does not affect font style :( @@ -180,22 +179,20 @@ void LLFloaterVoiceEffect::refreshEffectList() LLSD element; element["id"] = effect_id; - element["columns"][0]["column"] = "name"; - element["columns"][0]["value"] = effect_name; - element["columns"][0]["font"]["name"] = "SANSSERIF"; - element["columns"][0]["font"]["style"] = font_style; + element["columns"][NAME_COLUMN]["column"] = "name"; + element["columns"][NAME_COLUMN]["value"] = effect_name; + element["columns"][NAME_COLUMN]["font"]["style"] = font_style; element["columns"][1]["column"] = "expires"; if (!is_template_only) { - element["columns"][1]["value"] = expiry_date; - element["columns"][1]["type"] = "date"; + element["columns"][DATE_COLUMN]["value"] = expiry_date; + element["columns"][DATE_COLUMN]["type"] = "date"; } else { - element["columns"][1]["value"] = ""; + element["columns"][DATE_COLUMN]["value"] = ""; } - element["columns"][1]["font"]["name"] = "SANSSERIF"; - element["columns"][1]["font"]["style"] = "NORMAL"; +// element["columns"][DATE_COLUMN]["font"]["style"] = "NORMAL"; LLScrollListItem* sl_item = mVoiceEffectList->addElement(element, ADD_BOTTOM); // *HACK: Copied from llfloatergesture.cpp : ["font"]["style"] does not affect font style :( diff --git a/indra/newview/llfloatervoiceeffect.h b/indra/newview/llfloatervoiceeffect.h index 2a8330c72a..a3429577bb 100644 --- a/indra/newview/llfloatervoiceeffect.h +++ b/indra/newview/llfloatervoiceeffect.h @@ -54,6 +54,12 @@ public: virtual void onClose(bool app_quitting); private: + enum ColumnIndex + { + NAME_COLUMN = 0, + DATE_COLUMN = 1, + }; + void refreshEffectList(); void updateControls(); |