Fixes #2255 with some lambda fun

This commit is contained in:
mathieui 2011-11-05 21:25:17 +01:00
parent 3f08e235a1
commit a49af71f7c
2 changed files with 26 additions and 26 deletions

View file

@ -58,12 +58,12 @@ SHOW_NAME = {
NS_MUC_USER = 'http://jabber.org/protocol/muc#user'
STATE_COLORS = {
'disconnected': 'COLOR_TAB_DISCONNECTED',
'message': 'COLOR_TAB_NEW_MESSAGE',
'highlight': 'COLOR_TAB_HIGHLIGHT',
'private': 'COLOR_TAB_PRIVATE',
'normal': 'COLOR_TAB_NORMAL',
'current': 'COLOR_TAB_CURRENT',
'disconnected': lambda: get_theme().COLOR_TAB_DISCONNECTED,
'message': lambda: get_theme().COLOR_TAB_NEW_MESSAGE,
'highlight': lambda: get_theme().COLOR_TAB_HIGHLIGHT,
'private': lambda: get_theme().COLOR_TAB_PRIVATE,
'normal': lambda: get_theme().COLOR_TAB_NORMAL,
'current': lambda: get_theme().COLOR_TAB_CURRENT,
}
class Tab(object):
@ -98,7 +98,7 @@ class Tab(object):
@property
def color(self):
return getattr(get_theme(), STATE_COLORS[self.state])
return STATE_COLORS[self.state]()
@property
def state(self):
@ -826,7 +826,7 @@ class MucTab(ChatTab):
@property
def color(self):
return getattr(get_theme(), STATE_COLORS[self._room.state])
return STATE_COLORS[self._room.state]()
def on_lose_focus(self):
self._room.state = 'normal'
@ -1181,8 +1181,8 @@ class PrivateTab(ChatTab):
def color(self):
if self._room.state == 'normal' or\
self._room.state == 'current':
return getattr(get_theme(), STATE_COLORS[self._room.state])
return getattr(get_theme(), 'COLOR_TAB_PRIVATE')
return STATE_COLORS[self._room.state]()
return get_theme().COLOR_TAB_PRIVATE
def get_name(self):
return self._room.name
@ -1822,7 +1822,7 @@ class ConversationTab(ChatTab):
def color(self):
if self.state == 'normal' or \
self.state == 'current':
return getattr(get_theme(), STATE_COLORS[self.state])
return STATE_COLORS[self.state]()
return get_theme().COLOR_TAB_PRIVATE
def get_name(self):

View file

@ -191,19 +191,19 @@ class UserList(Win):
def __init__(self):
Win.__init__(self)
self.pos = 0
self.color_role = {'moderator': get_theme().COLOR_USER_MODERATOR,
'participant':get_theme().COLOR_USER_PARTICIPANT,
'visitor':get_theme().COLOR_USER_VISITOR,
'none':get_theme().COLOR_USER_NONE,
'':get_theme().COLOR_USER_NONE
}
self.color_show = {'xa':get_theme().COLOR_STATUS_XA,
'none':get_theme().COLOR_STATUS_NONE,
'':get_theme().COLOR_STATUS_NONE,
'dnd':get_theme().COLOR_STATUS_DND,
'away':get_theme().COLOR_STATUS_AWAY,
'chat':get_theme().COLOR_STATUS_CHAT
}
self.color_role = {'moderator': lambda: get_theme().COLOR_USER_MODERATOR,
'participant': lambda: get_theme().COLOR_USER_PARTICIPANT,
'visitor': lambda: get_theme().COLOR_USER_VISITOR,
'none': lambda: get_theme().COLOR_USER_NONE,
'': lambda: get_theme().COLOR_USER_NONE
}
self.color_show = {'xa': lambda: get_theme().COLOR_STATUS_XA,
'none': lambda: get_theme().COLOR_STATUS_NONE,
'': lambda: get_theme().COLOR_STATUS_NONE,
'dnd': lambda: get_theme().COLOR_STATUS_DND,
'away': lambda: get_theme().COLOR_STATUS_AWAY,
'chat': lambda: get_theme().COLOR_STATUS_CHAT
}
def scroll_up(self):
self.pos += self.height-1
@ -228,11 +228,11 @@ class UserList(Win):
if not user.role in self.color_role:
role_col = get_theme().COLOR_USER_NONE
else:
role_col = self.color_role[user.role]
role_col = self.color_role[user.role]()
if not user.show in self.color_show:
show_col = get_theme().COLOR_STATUS_NONE
else:
show_col = self.color_show[user.show]
show_col = self.color_show[user.show]()
if user.chatstate == 'composing':
char = 'X'
elif user.chatstate == 'active':