From cce01f1d75a43b943237a11f9695e8426425210a Mon Sep 17 00:00:00 2001 From: James Booth Date: Sun, 17 Jan 2016 02:17:12 +0000 Subject: Added /roster unread preferrence closes #712 --- src/command/command.c | 11 ++++++++++- src/command/commands.c | 27 +++++++++++++++++++++++++++ src/config/preferences.c | 5 +++++ src/config/preferences.h | 1 + src/config/theme.c | 1 + src/ui/console.c | 19 ++++++++++++++----- src/ui/rosterwin.c | 33 ++++++++++++++++++++++++--------- 7 files changed, 82 insertions(+), 15 deletions(-) (limited to 'src') diff --git a/src/command/command.c b/src/command/command.c index b6c00803..f3c4d3bb 100644 --- a/src/command/command.c +++ b/src/command/command.c @@ -282,6 +282,7 @@ static struct cmd_t command_defs[] = "/roster hide [offline|resource|presence|status|empty|count|priority|rooms]", "/roster by group|presence|none", "/roster order name|presence", + "/roster unread before|after|off", "/roster room order name|unread", "/roster room unread before|after|off", "/roster header char |none", @@ -326,11 +327,14 @@ static struct cmd_t command_defs[] = { "by none", "No grouping in the roster panel." }, { "order name", "Order roster items by name only." }, { "order presence", "Order roster items by presence, and then by name." }, + { "unread before", "Show unread message count before contact in roster." }, + { "unread after", "Show unread message count after contact in roster." }, + { "unread off", "Do not show unread message count for contacts in roster." }, { "room order name", "Order roster rooms by name." }, { "room order unread", "Order roster rooms by unread messages, and then by name." }, { "room unread before", "Show unread message count before room in roster." }, { "room unread after", "Show unread message count after room in roster." }, - { "room unread off", "Do not show unread message count for rooms in rosters." }, + { "room unread off", "Do not show unread message count for rooms in roster." }, { "header char ", "Prefix roster headers with specified character." }, { "header char none", "Remove roster header character prefix." }, { "contact char ", "Prefix roster contacts with specified character." }, @@ -2172,6 +2176,7 @@ cmd_init(void) autocomplete_add(roster_ac, "hide"); autocomplete_add(roster_ac, "by"); autocomplete_add(roster_ac, "order"); + autocomplete_add(roster_ac, "unread"); autocomplete_add(roster_ac, "room"); autocomplete_add(roster_ac, "size"); autocomplete_add(roster_ac, "wrap"); @@ -3169,6 +3174,10 @@ _roster_autocomplete(ProfWin *window, const char *const input) if (result) { return result; } + result = autocomplete_param_with_ac(input, "/roster unread", roster_unread_ac, TRUE); + if (result) { + return result; + } result = autocomplete_param_with_ac(input, "/roster room", roster_room_ac, TRUE); if (result) { return result; diff --git a/src/command/commands.c b/src/command/commands.c index 3ad79778..76904ef0 100644 --- a/src/command/commands.c +++ b/src/command/commands.c @@ -2310,6 +2310,33 @@ cmd_roster(ProfWin *window, const char *const command, gchar **args) return TRUE; } + } else if (g_strcmp0(args[0], "unread") == 0) { + if (g_strcmp0(args[1], "before") == 0) { + cons_show("Roster unread message count: before"); + prefs_set_string(PREF_ROSTER_UNREAD, "before"); + if (conn_status == JABBER_CONNECTED) { + rosterwin_roster(); + } + return TRUE; + } else if (g_strcmp0(args[1], "after") == 0) { + cons_show("Roster unread message count: after"); + prefs_set_string(PREF_ROSTER_UNREAD, "after"); + if (conn_status == JABBER_CONNECTED) { + rosterwin_roster(); + } + return TRUE; + } else if (g_strcmp0(args[1], "off") == 0) { + cons_show("Roster unread message count: off"); + prefs_set_string(PREF_ROSTER_UNREAD, "off"); + if (conn_status == JABBER_CONNECTED) { + rosterwin_roster(); + } + return TRUE; + } else { + cons_bad_cmd_usage(command); + return TRUE; + } + } else if (g_strcmp0(args[0], "room") == 0) { if (g_strcmp0(args[1], "order") == 0) { if (g_strcmp0(args[2], "name") == 0) { diff --git a/src/config/preferences.c b/src/config/preferences.c index 4b762c04..54bc5516 100644 --- a/src/config/preferences.c +++ b/src/config/preferences.c @@ -1042,6 +1042,7 @@ _get_group(preference_t pref) case PREF_ROSTER_EMPTY: case PREF_ROSTER_BY: case PREF_ROSTER_ORDER: + case PREF_ROSTER_UNREAD: case PREF_ROSTER_COUNT: case PREF_ROSTER_PRIORITY: case PREF_ROSTER_WRAP: @@ -1235,6 +1236,8 @@ _get_key(preference_t pref) return "roster.by"; case PREF_ROSTER_ORDER: return "roster.order"; + case PREF_ROSTER_UNREAD: + return "roster.unread"; case PREF_ROSTER_COUNT: return "roster.count"; case PREF_ROSTER_PRIORITY: @@ -1336,6 +1339,8 @@ _get_default_string(preference_t pref) return "presence"; case PREF_ROSTER_ORDER: return "presence"; + case PREF_ROSTER_UNREAD: + return "after"; case PREF_ROSTER_ROOMS_ORDER: return "name"; case PREF_ROSTER_ROOMS_UNREAD: diff --git a/src/config/preferences.h b/src/config/preferences.h index 06c2c67b..df6cd482 100644 --- a/src/config/preferences.h +++ b/src/config/preferences.h @@ -69,6 +69,7 @@ typedef enum { PREF_ROSTER_EMPTY, PREF_ROSTER_BY, PREF_ROSTER_ORDER, + PREF_ROSTER_UNREAD, PREF_ROSTER_COUNT, PREF_ROSTER_PRIORITY, PREF_ROSTER_WRAP, diff --git a/src/config/theme.c b/src/config/theme.c index 6460f65d..3dad150a 100644 --- a/src/config/theme.c +++ b/src/config/theme.c @@ -317,6 +317,7 @@ _load_preferences(void) _set_boolean_preference("roster.wrap", PREF_ROSTER_WRAP); _set_string_preference("roster.by", PREF_ROSTER_BY); _set_string_preference("roster.order", PREF_ROSTER_ORDER); + _set_string_preference("roster.unread", PREF_ROSTER_UNREAD); _set_boolean_preference("roster.count", PREF_ROSTER_COUNT); _set_boolean_preference("roster.priority", PREF_ROSTER_PRIORITY); _set_boolean_preference("roster.rooms", PREF_ROSTER_ROOMS); diff --git a/src/ui/console.c b/src/ui/console.c index 4656b4e3..78ab8d2a 100644 --- a/src/ui/console.c +++ b/src/ui/console.c @@ -1324,11 +1324,6 @@ cons_roster_setting(void) else cons_show("Roster priority (/roster) : hide"); - if (prefs_get_boolean(PREF_ROSTER_ROOMS)) - cons_show("Roster rooms (/roster) : show"); - else - cons_show("Roster rooms (/roster) : hide"); - char *by = prefs_get_string(PREF_ROSTER_BY); cons_show("Roster by (/roster) : %s", by); prefs_free_string(by); @@ -1337,6 +1332,20 @@ cons_roster_setting(void) cons_show("Roster order (/roster) : %s", order); prefs_free_string(order); + char *unread = prefs_get_string(PREF_ROSTER_UNREAD); + if (g_strcmp0(unread, "before") == 0) { + cons_show("Roster unread (/roster) : before"); + } else if (g_strcmp0(unread, "after") == 0) { + cons_show("Roster unread (/roster) : after"); + } else { + cons_show("Roster unread (/roster) : OFF"); + } + + if (prefs_get_boolean(PREF_ROSTER_ROOMS)) + cons_show("Roster rooms (/roster) : show"); + else + cons_show("Roster rooms (/roster) : hide"); + char *rooms_order = prefs_get_string(PREF_ROSTER_ROOMS_ORDER); cons_show("Roster rooms order (/roster) : %s", rooms_order); prefs_free_string(rooms_order); diff --git a/src/ui/rosterwin.c b/src/ui/rosterwin.c index 5b18ec95..7593b9ab 100644 --- a/src/ui/rosterwin.c +++ b/src/ui/rosterwin.c @@ -167,9 +167,11 @@ _rosterwin_resources(ProfLayoutSplit *layout, PContact contact, int current_inde g_string_append_printf(msg, " %d", resource->priority); } - if (unread > 0) { + char *unreadpos = prefs_get_string(PREF_ROSTER_UNREAD); + if ((g_strcmp0(unreadpos, "after") == 0) && unread > 0) { g_string_append_printf(msg, " (%d)", unread); } + prefs_free_string(unreadpos); gboolean wrap = prefs_get_boolean(PREF_ROSTER_WRAP); win_sub_print(layout->subwin, msg->str, FALSE, wrap, 0); @@ -182,7 +184,8 @@ _rosterwin_resources(ProfLayoutSplit *layout, PContact contact, int current_inde } else { gboolean wrap = prefs_get_boolean(PREF_ROSTER_WRAP); - if (unread > 0) { + char *unreadpos = prefs_get_string(PREF_ROSTER_UNREAD); + if ((g_strcmp0(unreadpos, "after") == 0) && unread > 0) { GString *unreadmsg = g_string_new(""); g_string_append_printf(unreadmsg, " (%d)", unread); @@ -193,6 +196,7 @@ _rosterwin_resources(ProfLayoutSplit *layout, PContact contact, int current_inde win_sub_print(layout->subwin, unreadmsg->str, FALSE, wrap, current_indent); wattroff(layout->subwin, theme_attrs(presence_colour)); } + prefs_free_string(unreadpos); int resource_indent = prefs_get_roster_resource_indent(); if (resource_indent > 0) { @@ -238,7 +242,8 @@ _rosterwin_resources(ProfLayoutSplit *layout, PContact contact, int current_inde theme_item_t presence_colour = _get_roster_theme(theme_type, presence); gboolean wrap = prefs_get_boolean(PREF_ROSTER_WRAP); - if (unread > 0) { + char *unreadpos = prefs_get_string(PREF_ROSTER_UNREAD); + if ((g_strcmp0(unreadpos, "after") == 0) && unread > 0) { GString *unreadmsg = g_string_new(""); g_string_append_printf(unreadmsg, " (%d)", unread); @@ -246,11 +251,13 @@ _rosterwin_resources(ProfLayoutSplit *layout, PContact contact, int current_inde win_sub_print(layout->subwin, unreadmsg->str, FALSE, wrap, current_indent); wattroff(layout->subwin, theme_attrs(presence_colour)); } + prefs_free_string(unreadpos); _rosterwin_presence(layout, presence_colour, presence, status, current_indent); } else { gboolean wrap = prefs_get_boolean(PREF_ROSTER_WRAP); - if (unread > 0) { + char *unreadpos = prefs_get_string(PREF_ROSTER_UNREAD); + if ((g_strcmp0(unreadpos, "after") == 0) && unread > 0) { GString *unreadmsg = g_string_new(""); g_string_append_printf(unreadmsg, " (%d)", unread); const char *presence = p_contact_presence(contact); @@ -260,6 +267,7 @@ _rosterwin_resources(ProfLayoutSplit *layout, PContact contact, int current_inde win_sub_print(layout->subwin, unreadmsg->str, FALSE, wrap, current_indent); wattroff(layout->subwin, theme_attrs(presence_colour)); } + prefs_free_string(unreadpos); } g_list_free(resources); @@ -303,14 +311,21 @@ _rosterwin_contact(ProfLayoutSplit *layout, PContact contact) if (ch) { g_string_append_printf(msg, "%c", ch); } - g_string_append(msg, name); - if (!prefs_get_boolean(PREF_ROSTER_RESOURCE)) { - if (unread > 0) { - g_string_append_printf(msg, " (%d)", unread); + char *unreadpos = prefs_get_string(PREF_ROSTER_UNREAD); + if ((g_strcmp0(unreadpos, "before") == 0) && unread > 0) { + g_string_append_printf(msg, "(%d) ", unread); + } + g_string_append(msg, name); + if (g_strcmp0(unreadpos, "after") == 0) { + if (!prefs_get_boolean(PREF_ROSTER_RESOURCE)) { + if (unread > 0) { + g_string_append_printf(msg, " (%d)", unread); + } + unread = 0; } - unread = 0; } + prefs_free_string(unreadpos); win_sub_newline_lazy(layout->subwin); gboolean wrap = prefs_get_boolean(PREF_ROSTER_WRAP); -- cgit 1.4.1-2-gfad0