* add contexttype, so the subst can precheck the context pointer for us, and bad...
[citadel.git] / webcit / iconbar.c
index 8d325d988eee2cd60831b4bfed84f7a7225349f9..99350602b47902378fcbcb91a73a42729b677b96 100644 (file)
@@ -7,12 +7,18 @@
 #include "webcit.h"
 
 
-/* Values for ib_displayas */
-#define IB_PICTEXT     0
-#define IB_PICONLY     1
-#define IB_TEXTONLY    2
+/** Values for ib_displayas */
+#define IB_PICTEXT     0 /**< picture and text */
+#define IB_PICONLY     1 /**< just a picture */
+#define IB_TEXTONLY    2 /**< just text */
 
 
+void do_iconbar(void);
+void do_iconbar_roomlist(void);
+
+/**
+ * \brief draw the icon bar?????
+ */
 void do_selected_iconbar(void) {
        if (WC->current_iconbar == current_iconbar_roomlist) {
                do_iconbar_roomlist();
@@ -22,110 +28,133 @@ void do_selected_iconbar(void) {
        }
 }
 
+void DontDeleteThis(void *Data){};
 
-void do_iconbar(void) {
-       char iconbar[SIZ];
-       char buf[SIZ];
-       char key[SIZ], value[SIZ];
-       int i;
+#define IconbarIsEnabled(a, b) IconbarIsENABLED(a, sizeof(a) - 1, b)
 
-       WC->current_iconbar = current_iconbar_menu;
+long IconbarIsENABLED(const char *key, size_t keylen, long defval)
+{
+       void *Data;
+       if (GetHash(WC->IconBarSetttings, key, keylen,
+                   &Data))
+               return (long) Data;
+       else 
+               return defval;
+}
 
-       /* The initialized values of these variables also happen to
+#ifdef DBG_ICONBAR_HASH
+static char nbuf[32];
+inline const char *PrintInt(void *Prefstr)
+{
+       snprintf(nbuf, sizeof(nbuf), "%ld", (long)Prefstr);
+       return nbuf;
+}
+#endif
+
+void LoadIconSettings(void)
+{
+       struct wcsession *WCC = WC;
+       StrBuf *iconbar = NULL;
+       StrBuf *buf = NewStrBuf();;
+       StrBuf *key = NewStrBuf();
+       long val;
+       int i, nTokens;
+
+       WCC->current_iconbar = current_iconbar_menu;
+       if (WCC->IconBarSetttings == NULL)
+               WCC->IconBarSetttings = NewHash(1, NULL);
+       /**
+        * The initialized values of these variables also happen to
         * specify the default values for users who haven't customized
         * their iconbars.  These should probably be set in a master
         * configuration somewhere.
         */
-       int ib_displayas = 0;   /* pictures and text, pictures, text */
-       int ib_logo = 0;        /* Site logo */
-       int ib_summary = 1;     /* Summary page icon */
-       int ib_inbox = 1;       /* Inbox icon */
-       int ib_calendar = 1;    /* Calendar icon */
-       int ib_contacts = 1;    /* Contacts icon */
-       int ib_notes = 1;       /* Notes icon */
-       int ib_tasks = 1;       /* Tasks icon */
-       int ib_rooms = 1;       /* Rooms icon */
-       int ib_users = 1;       /* Users icon */
-       int ib_chat = 1;        /* Chat icon */
-       int ib_advanced = 1;    /* Advanced Options icon */
-       int ib_citadel = 1;     /* 'Powered by Citadel' logo */
-       /*
-        */
 
-       get_preference("iconbar", iconbar, sizeof iconbar);
-       for (i=0; i<num_tokens(iconbar, ','); ++i) {
-               extract_token(buf, iconbar, i, ',', sizeof buf);
-               extract_token(key, buf, 0, '=', sizeof key);
-               extract_token(value, buf, 1, '=', sizeof value);
-
-               if (!strcasecmp(key, "ib_displayas")) ib_displayas = atoi(value);
-               if (!strcasecmp(key, "ib_logo")) ib_logo = atoi(value);
-               if (!strcasecmp(key, "ib_summary")) ib_summary = atoi(value);
-               if (!strcasecmp(key, "ib_inbox")) ib_inbox = atoi(value);
-               if (!strcasecmp(key, "ib_calendar")) ib_calendar = atoi(value);
-               if (!strcasecmp(key, "ib_contacts")) ib_contacts = atoi(value);
-               if (!strcasecmp(key, "ib_notes")) ib_notes = atoi(value);
-               if (!strcasecmp(key, "ib_tasks")) ib_tasks = atoi(value);
-               if (!strcasecmp(key, "ib_rooms")) ib_rooms = atoi(value);
-               if (!strcasecmp(key, "ib_users")) ib_users = atoi(value);
-               if (!strcasecmp(key, "ib_chat")) ib_chat = atoi(value);
-               if (!strcasecmp(key, "ib_advanced")) ib_advanced = atoi(value);
-               if (!strcasecmp(key, "ib_citadel")) ib_citadel = atoi(value);
+       if (get_preference("iconbar", &iconbar)) {
+               nTokens = StrBufNum_tokens(iconbar, ',');
+               for (i=0; i<nTokens; ++i) {
+                       StrBufExtract_token(buf, iconbar, i, ',');
+                       StrBufExtract_token(key, buf, 0, '=');
+                       val = StrBufExtract_long(buf, 1, '=');
+                       Put(WCC->IconBarSetttings, 
+                           ChrPtr(key), StrLength(key),
+                           (void*)val, DontDeleteThis);
+               }
        }
 
-       wprintf("<div id=\"button\">\n"
-               "<ul>\n"
-       );
+#ifdef DBG_ICONBAR_HASH
+       dbg_PrintHash(WCC->IconBarSetttings, PrintInt, NULL);
+#endif
 
-       if (ib_logo) {
-               wprintf("<li>");
-               if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<IMG BORDER=\"0\" WIDTH=\"32\" "
-                               "HEIGHT=\"32\" src=\"image&name=hello\" ALT=\"&nbsp;\">\n"
-                       );
-               }
-               wprintf("</li>\n");
-       }
+       FreeStrBuf(&key);
+       FreeStrBuf(&buf);
+}
 
-       if (ib_citadel) if (ib_displayas != IB_TEXTONLY) wprintf(
-               "<li><div align=\"center\">"
-               "<a href=\"http://www.citadel.org\" "
-               "title=\"%s\" target=\"aboutcit\">"
-               "<img border=\"0\" "
-               "src=\"static/citadel-logo.gif\" ALT=\"%s\"></a>"
-               "</div></li>\n",
-               _("Find out more about Citadel"),
-               _("CITADEL")
-       );
 
-       wprintf("<li><div align=\"center\"><a href=\"javascript:switch_to_room_list()\">");
+/**
+ * \brief draw the icon bar???
+ */
+void do_iconbar(void) {
+       int ib_displayas = IB_PICTEXT;  /**< pictures and text, pictures, text */
+
+       LoadIconSettings();
+       ib_displayas = IconbarIsEnabled("ib_displayas", IB_PICTEXT);
+
+/** Site logo */
+       if (IconbarIsEnabled("ib_logo", 0)) {
+                if (ib_displayas != IB_TEXTONLY) {
+                        wprintf("<div class=\"logo\"> <img "
+                                "src=\"image&name=hello\" alt=\"&nbsp;\"> "
+                                "</div>\n"
+                        );
+                }
+                wprintf("\n");
+        }
+
+/** 'Powered by Citadel' logo */
+        if (IconbarIsEnabled("ib_citadel", 1) && (ib_displayas != IB_TEXTONLY)) wprintf(
+                "<div class=\"logo_citadel\"> "
+                "<a href=\"http://www.citadel.org\" "
+                "title=\"%s\"> "
+                "<img "
+                "src=\"static/citadel-logo.gif\" alt=\"%s\"></a> "
+                "</div>\n",
+                _("Find out more about Citadel"),
+                _("CITADEL")
+        );
+
+       wprintf("<ul id=\"button\">\n");
+
+       wprintf("<li class=\"switch\"><a href=\"javascript:switch_to_room_list()\">");
        wprintf(_("switch to room list"));
-       wprintf("</a></div>");
+       wprintf("</a></li>");
 
-       if (ib_summary) {
+/** Summary page icon */
+       if (IconbarIsEnabled("ib_summary", 1)) {
                wprintf("<li><a href=\"summary\" "
-                       "TITLE=\"%s\" "
+                       "title=\"%s\" "
                        ">", _("Your summary page")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<IMG BORDER=\"0\" WIDTH=\"32\" HEIGHT=\"32\" "
+                       wprintf("<img alt=\"\" "
                                "src=\"static/summscreen_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
                        wprintf(_("Summary"));
                }
-               wprintf("</A></li>\n");
+               wprintf("</a></li>\n");
        }
 
-       if (ib_inbox) {
+/** Inbox icon */
+       if (IconbarIsEnabled("ib_inbox", 1)) {
                wprintf("<li>"
                        "<a href=\"dotgoto?room=_MAIL_\" "
-                       "TITLE=\"%s\" "
+                       "title=\"%s\" "
                        ">",
                        _("Go to your email inbox")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<IMG BORDER=\"0\" WIDTH=\"32\" HEIGHT=\"32\" "
+                       wprintf("<img alt=\"\" "
                                "src=\"static/privatemess_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
@@ -139,109 +168,122 @@ void do_iconbar(void) {
                                WC->remember_new_mail = WC->new_mail;
                        }
                }
-               wprintf("</A></li>\n");
+               wprintf("</a></li>\n");
        }
 
-       if (ib_calendar) {
+/** Calendar icon */
+       if (IconbarIsEnabled("ib_calendar", 1)) {
                wprintf("<li>"
                        "<a href=\"dotgoto?room=_CALENDAR_\" "
-                       "TITLE=\"%s\" "
+                       "title=\"%s\" "
                        ">",
                        _("Go to your personal calendar")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<IMG BORDER=\"0\" WIDTH=\"32\" HEIGHT=\"32\" "
+                       wprintf("<img alt=\"\" "
                        "src=\"static/calarea_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
                        wprintf(_("Calendar"));
                }
-               wprintf("</A></li>\n");
+               wprintf("</a></li>\n");
        }
 
-       if (ib_contacts) {
+/** Contacts icon */
+       if (IconbarIsEnabled("ib_contacts", 1)) {
                wprintf("<li>"
                        "<a href=\"dotgoto?room=_CONTACTS_\" "
-                       "TITLE=\"%s\" "
+                       "title=\"%s\" "
                        ">",
                        _("Go to your personal address book")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<IMG BORDER=\"0\" WIDTH=\"32\" HEIGHT=\"32\" "
+                       wprintf("<img alt=\"\" "
                        "src=\"static/viewcontacts_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
                        wprintf(_("Contacts"));
                }
-               wprintf("</A></li>\n");
+               wprintf("</a></li>\n");
        }
 
-       if (ib_notes) {
+/** Notes icon */
+       if (IconbarIsEnabled("ib_notes", 1)) {
                wprintf("<li>"
                        "<a href=\"dotgoto?room=_NOTES_\" "
-                       "TITLE=\"%s\" "
+                       "title=\"%s\" "
                        ">",
                        _("Go to your personal notes")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<IMG BORDER=\"0\" WIDTH=\"32\" HEIGHT=\"32\" "
+                       wprintf("<img alt=\"\" "
                        "src=\"static/storenotes_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
                        wprintf(_("Notes"));
                }
-               wprintf("</A></li>\n");
+               wprintf("</a></li>\n");
        }
 
-       if (ib_tasks)  {
+/** Tasks icon */
+       if (IconbarIsEnabled("ib_tasks", 1))  {
                wprintf("<li>"
                        "<a href=\"dotgoto?room=_TASKS_\" "
-                       "TITLE=\"%s\" "
+                       "title=\"%s\" "
                        ">",
                        _("Go to your personal task list")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<IMG BORDER=\"0\" WIDTH=\"32\" HEIGHT=\"32\" "
+                       wprintf("<img alt=\"\" "
                        "src=\"static/taskmanag_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
                        wprintf(_("Tasks"));
                }
-               wprintf("</A></li>\n");
+               wprintf("</a></li>\n");
        }
 
-       if (ib_rooms) {
+/** Rooms icon */
+       if (IconbarIsEnabled("ib_rooms", 1)) {
                wprintf("<li>"
-                       "<a href=\"knrooms\" TITLE=\"%s\" >",
+                       "<a href=\"knrooms\" title=\"%s\" >",
                        _("List all of your accessible rooms")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<IMG BORDER=\"0\" WIDTH=\"32\" HEIGHT=\"32\" "
+                       wprintf("<img alt=\"\" "
                        "src=\"static/chatrooms_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
                        wprintf(_("Rooms"));
                }
-               wprintf("</A></li>\n");
+               wprintf("</a></li>\n");
        }
 
-       if (ib_users) {
+/** Users icon */
+       if (IconbarIsEnabled("ib_users", 1)) {
                wprintf("<li>"
-                       "<a href=\"who\" TITLE=\"%s\" "
+                       "<a href=\"do_template?template=who\" title=\"%s\" "
                        ">",
                        _("See who is online right now")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<IMG BORDER=\"0\" WIDTH=\"32\" HEIGHT=\"32\" "
+                       wprintf("<img alt=\"\" "
                        "src=\"static/usermanag_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
                        wprintf(_("Who is online?"));
                }
-               wprintf("</A></li>\n");
+                
+               wprintf("</a>\n");
+
+               if (IconbarIsEnabled("ib_users", 0)) {
+                       wprintf("<ul id=\"wholist\">");
+                       wprintf("</ul></li>\n");
+               }
        }
 
-       if (ib_chat) {
+/** Chat icon */
+       if (IconbarIsEnabled("ib_chat", 1)) {
                wprintf("<li>"
                        "<a href=\"#\" onClick=\"window.open('chat', "
                        "'ctdl_chat_window', "
@@ -250,237 +292,194 @@ void do_iconbar(void) {
                        ">"
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<IMG BORDER=\"0\" WIDTH=\"32\" HEIGHT=\"32\" "
+                       wprintf("<img alt=\"\" "
                        "src=\"static/citadelchat_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
                        wprintf(_("Chat"));
                }
-               wprintf("</A></li>\n");
+               wprintf("</a></li>\n");
        }
 
-       if (ib_advanced) {
+/** Advanced Options icon */
+       if (IconbarIsEnabled("ib_advanced", 1)) {
                wprintf("<li>"
-                       "<a href=\"display_main_menu\" "
-                       "TITLE=\"%s\" "
+                       "<a href=\"do_template?template=display_main_menu\" "
+                       "title=\"%s\" "
                        ">",
                        _("Advanced Options Menu: Advanced Room commands, Account Info, and Chat")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<IMG BORDER=\"0\" WIDTH=\"32\" HEIGHT=\"32\" "
+                       wprintf("<img alt=\"\" "
                        "src=\"static/advanpage2_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
                        wprintf(_("Advanced"));
                }
-               wprintf("</A></li>\n");
+               wprintf("</a></li>\n");
        }
 
        if ((WC->axlevel >= 6) || (WC->is_room_aide)) {
                wprintf("<li>"
-                       "<a href=\"display_aide_menu\" "
-                       "TITLE=\"%s\" "
+                       "<a href=\"do_template?template=display_aide_menu\" "
+                       "title=\"%s\" "
                        ">",
                        _("Room and system administration functions")
                );
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<IMG BORDER=\"0\" WIDTH=\"32\" HEIGHT=\"32\" "
+                       wprintf("<img alt=\"\" "
                        "src=\"static/advanpage2_32x.gif\">");
                }
                if (ib_displayas != IB_PICONLY) {
                        wprintf(_("Administration"));
                }
-               wprintf("</A></li>\n");
+               wprintf("</a></li>\n");
        }
 
        wprintf("<li>"
-               "<a href=\"termquit\" TITLE=\"%s\" "
+               "<a href=\"termquit\" title=\"%s\" "
                "onClick=\"return confirm('%s');\">",
                _("Log off"),
                _("Log off now?")
                
        );
        if (ib_displayas != IB_TEXTONLY) {
-       wprintf("<IMG BORDER=\"0\" WIDTH=\"32\" HEIGHT=\"32\" "
+       wprintf("<img alt=\"\" "
                "src=\"static/logoff_32x.gif\">");
        }
        if (ib_displayas != IB_PICONLY) {
                wprintf(_("Log off"));
        }
-       wprintf("</A></li>\n");
+       wprintf("</a></li>\n");
 
        wprintf(
-               "<li><div align=\"center\">"
+               "<li class=\"switch\">"
                "<a href=\"display_customize_iconbar\" "
-               "TITLE=\"%s\" "
+               "title=\"%s\" "
                ">%s"
-               "</A></div></li>\n",
+               "</a></li>\n",
                _("Customize this menu"),
                _("customize this menu")
        );
 
-       wprintf("</ul></div>\n");
+       wprintf("</ul>\n");
+
+       if (IconbarIsEnabled("ib_users", 0)) {
+               wprintf(
+                       "<script type=\"text/javascript\"> "
+                       " new Ajax.PeriodicalUpdater('wholist', 'do_template?template=wholist_section', { method: 'get', frequency: 30 } );"
+                       "</script> \n"
+                       );
+       }
 }
 
 
-/*
+/**
+ * \brief roomtree view of the iconbar
  * If the user has toggled the icon bar over to a room list, here's where
  * we generate its innerHTML...
  */
 void do_iconbar_roomlist(void) {
-       char iconbar[SIZ];
-       char buf[SIZ];
-       char key[SIZ], value[SIZ];
-       int i;
-
+                               
        WC->current_iconbar = current_iconbar_roomlist;
 
-       /* The initialized values of these variables also happen to
+       /**
+        * The initialized values of these variables also happen to
         * specify the default values for users who haven't customized
         * their iconbars.  These should probably be set in a master
         * configuration somewhere.
         */
-       int ib_displayas = 0;   /* pictures and text, pictures, text */
-       int ib_logo = 0;        /* Site logo */
-       int ib_citadel = 1;     /* 'Powered by Citadel' logo */
-       /*
-        */
+       int ib_displayas;
 
-       get_preference("iconbar", iconbar, sizeof iconbar);
-       for (i=0; i<num_tokens(iconbar, ','); ++i) {
-               extract_token(buf, iconbar, i, ',', sizeof buf);
-               extract_token(key, buf, 0, '=', sizeof key);
-               extract_token(value, buf, 1, '=', sizeof value);
+       LoadIconSettings();
 
-               if (!strcasecmp(key, "ib_displayas")) ib_displayas = atoi(value);
-               if (!strcasecmp(key, "ib_logo")) ib_logo = atoi(value);
-               if (!strcasecmp(key, "ib_citadel")) ib_citadel = atoi(value);
-       }
+       ib_displayas = IconbarIsEnabled("ib_displayas", IB_PICTEXT);    /* pictures and text, pictures, text */
 
-       wprintf("<div id=\"button\">\n"
-               "<ul>\n"
-       );
-
-       if (ib_logo) {
-               wprintf("<li>");
+/** Site logo */
+       if (IconbarIsEnabled("ib_logo", 0)) {
                if (ib_displayas != IB_TEXTONLY) {
-                       wprintf("<IMG BORDER=\"0\" WIDTH=\"32\" "
-                               "HEIGHT=\"32\" src=\"image&name=hello\" ALT=\"&nbsp;\">\n"
+                        wprintf("<div class=\"logo\"> <img "
+                                "src=\"image&name=hello\" alt=\"&nbsp;\"> "
+                                "</div>\n"
                        );
                }
-               wprintf("</li>\n");
        }
 
-       if (ib_citadel) if (ib_displayas != IB_TEXTONLY) wprintf(
-               "<li><div align=\"center\">"
-               "<a href=\"http://www.citadel.org\" "
-               "title=\"%s\" target=\"aboutcit\">"
-               "<img border=\"0\" "
-               "src=\"static/citadel-logo.gif\" ALT=\"%s\"></a>"
-               "</div></li>\n",
-               _("Find out more about Citadel"),
-               _("CITADEL")
-       );
-
-       wprintf("<li><div align=\"center\"><a href=\"javascript:switch_to_menu_buttons()\">");
+/** 'Powered by Citadel' logo */
+        if (IconbarIsEnabled("ib_citadel", 1) && (ib_displayas != IB_TEXTONLY)) wprintf(
+                "<div class=\"logo_citadel\"> "
+                "<a href=\"http://www.citadel.org\" "
+                "title=\"%s\"> "
+                "<img "
+                "src=\"static/citadel-logo.gif\" alt=\"%s\"></a> "
+                "</div>\n",
+                _("Find out more about Citadel"),
+                _("CITADEL")
+        );
+
+       wprintf("<ul id=\"button\">\n");
+
+       wprintf("<li class=\"switch\"><a href=\"javascript:switch_to_menu_buttons()\">");
        wprintf(_("switch to menu"));
-       wprintf("</a></div>");
+       wprintf("</a></li>");
 
        wprintf("<li>"
-               "<a href=\"termquit\" TITLE=\"%s\" "
+               "<a href=\"termquit\" title=\"%s\" "
                "onClick=\"return confirm('%s');\">",
                _("Log off"),
                _("Log off now?")
                
        );
        if (ib_displayas != IB_TEXTONLY) {
-       wprintf("<IMG BORDER=\"0\" WIDTH=\"32\" HEIGHT=\"32\" "
+       wprintf("<img alt=\"\" "
                "src=\"static/logoff_32x.gif\">");
        }
        if (ib_displayas != IB_PICONLY) {
                wprintf(_("Log off"));
        }
-       wprintf("</A></li>\n");
+       wprintf("</a></li>\n");
 
-       wprintf("</ul></div>\n");
+       wprintf("</ul>\n");
 
-       /* embed the room list */
+       /** embed the room list */
        list_all_rooms_by_floor("iconbar");
 
        wprintf("</div>\n");
 }
 
 
-
+/**
+ * \brief display a customized version of the iconbar
+ */
 void display_customize_iconbar(void) {
-       char iconbar[SIZ];
-       char buf[SIZ];
-       char key[SIZ], value[SIZ];
        int i;
        int bar = 0;
+       long val;
 
-       /* The initialized values of these variables also happen to
-        * specify the default values for users who haven't customized
-        * their iconbars.  These should probably be set in a master
-        * configuration somewhere.
-        */
-       int ib_displayas = IB_PICTEXT;  /* pictures and text, pictures, text */
-       int ib_logo = 0;        /* Site logo */
-       int ib_summary = 1;     /* Summary page icon */
-       int ib_inbox = 1;       /* Inbox icon */
-       int ib_calendar = 1;    /* Calendar icon */
-       int ib_contacts = 1;    /* Contacts icon */
-       int ib_notes = 1;       /* Notes icon */
-       int ib_tasks = 1;       /* Tasks icon */
-       int ib_rooms = 1;       /* Rooms icon */
-       int ib_users = 1;       /* Users icon */
-       int ib_chat = 1;        /* Chat icon */
-       int ib_advanced = 1;    /* Advanced Options icon */
-       int ib_citadel = 1;     /* 'Powered by Citadel' logo */
-       /*
-        */
+       int ib_displayas;
 
-       get_preference("iconbar", iconbar, sizeof iconbar);
-       for (i=0; i<num_tokens(iconbar, ','); ++i) {
-               extract_token(buf, iconbar, i, ',', sizeof buf);
-               extract_token(key, buf, 0, '=', sizeof key);
-               extract_token(value, buf, 1, '=', sizeof value);
-
-               if (!strcasecmp(key, "ib_displayas")) ib_displayas = atoi(value);
-               if (!strcasecmp(key, "ib_logo")) ib_logo = atoi(value);
-               if (!strcasecmp(key, "ib_summary")) ib_summary = atoi(value);
-               if (!strcasecmp(key, "ib_inbox")) ib_inbox = atoi(value);
-               if (!strcasecmp(key, "ib_calendar")) ib_calendar = atoi(value);
-               if (!strcasecmp(key, "ib_contacts")) ib_contacts = atoi(value);
-               if (!strcasecmp(key, "ib_notes")) ib_notes = atoi(value);
-               if (!strcasecmp(key, "ib_tasks")) ib_tasks = atoi(value);
-               if (!strcasecmp(key, "ib_rooms")) ib_rooms = atoi(value);
-               if (!strcasecmp(key, "ib_users")) ib_users = atoi(value);
-               if (!strcasecmp(key, "ib_chat")) ib_chat = atoi(value);
-               if (!strcasecmp(key, "ib_advanced")) ib_advanced = atoi(value);
-               if (!strcasecmp(key, "ib_citadel")) ib_citadel = atoi(value);
-       }
+       LoadIconSettings();
 
        output_headers(1, 1, 2, 0, 0, 0);
-       wprintf("<div id=\"banner\">\n"
-               "<TABLE WIDTH=100%% BORDER=0 BGCOLOR=\"#444455\"><TR><TD>"
-               "<SPAN CLASS=\"titlebar\">");
+       wprintf("<div id=\"banner\">");
+       wprintf("<h1>");
        wprintf(_("Customize the icon bar"));
-       wprintf("</SPAN>"
-               "</TD></TR></TABLE>\n"
-               "</div>\n<div id=\"content\">\n"
-       );
+       wprintf("</h1></div>\n");
+
+       wprintf("<div id=\"content\" class=\"service\">\n");
 
-       wprintf("<div id=\"fix_scrollbar_bug\">"
-               "<table border=0 width=100%% bgcolor=\"#ffffff\"><tr><td>");
+       wprintf("<div class=\"fix_scrollbar_bug\">");
 
-       wprintf("<FORM METHOD=\"POST\" action=\"commit_iconbar\">\n");
+       wprintf("<form method=\"post\" action=\"commit_iconbar\">\n");
+       wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
 
-       wprintf("<CENTER>");
+       wprintf("<table class=\"altern\" >\n");
+       wprintf("<tr><td></td><td colspan=\"2\"><b>");
        wprintf(_("Display icons as:"));
-       wprintf(" ");
+       wprintf("</b>");
+       ib_displayas = IconbarIsEnabled("ib_displayas",IB_PICTEXT);
        for (i=0; i<=2; ++i) {
-               wprintf("<INPUT TYPE=\"radio\" NAME=\"ib_displayas\" VALUE=\"%d\"", i);
+               wprintf("<input type=\"radio\" name=\"ib_displayas\" value=\"%d\"", i);
                if (ib_displayas == i) wprintf(" CHECKED");
                wprintf(">");
                if (i == IB_PICTEXT)    wprintf(_("pictures and text"));
@@ -488,224 +487,257 @@ void display_customize_iconbar(void) {
                if (i == IB_TEXTONLY)   wprintf(_("text only"));
                wprintf("\n");
        }
-       wprintf("<br /><br />\n");
+       wprintf("<br />\n");
 
        wprintf(_("Select the icons you would like to see displayed "
                "in the 'icon bar' menu on the left side of the "
                "screen."));
-       wprintf("</CENTER><br />\n");
-
-       wprintf("<TABLE border=0 cellspacing=0 cellpadding=3 width=100%%>\n");
-
-       wprintf("<TR BGCOLOR=%s><TD>"
-               "<INPUT TYPE=\"checkbox\" NAME=\"ib_logo\" VALUE=\"yes\" %s>"
-               "</TD><TD>"
-               "<IMG BORDER=\"0\" WIDTH=\"48\" HEIGHT=\"48\" "
-               "src=\"image&name=hello\" ALT=\"&nbsp;\">"
-               "</TD><TD>"
-               "<B>%s</B><br />"
+       wprintf("</td></tr>\n");
+
+       bar = 1 - bar;
+       val = IconbarIsEnabled("ib_logo", 0);
+       wprintf("<tr class=\"%s\"><td>"
+               "<input type=\"radio\" name=\"ib_logo\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
+               "<input type=\"radio\" name=\"ib_logo\" value=\"no\" %s> %s <br />"
+               "</td><td>"
+               "<img src=\"image&name=hello\" width=\"48\" alt=\"&nbsp;\">"
+               "</td><td>"
+               "<b>%s</b><br />"
                "%s"
-               "</TD></TR>\n",
-               ((bar = 1 - bar), (bar ? "\"#CCCCCC\"" : "\"#FFFFFF\"")),
-               (ib_logo ? "CHECKED" : ""),
+               "</td></tr>\n",
+               (bar ? "even" : "odd"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Site logo"),
                _("An icon describing this site")
        );
 
-       wprintf("<TR bgcolor=%s><TD>"
-               "<INPUT TYPE=\"checkbox\" NAME=\"ib_summary\" VALUE=\"yes\" %s>"
-               "</TD><TD>"
-               "<IMG BORDER=\"0\" WIDTH=\"48\" HEIGHT=\"48\" "
-               "src=\"static/summscreen_48x.gif\" ALT=\"&nbsp;\">"
-               "</TD><TD>"
-               "<B>%s</B><br />"
+       bar = 1 - bar;
+       val = IconbarIsEnabled("ib_summary", 1);
+       wprintf("<tr class=\"%s\"><td>"
+               "<input type=\"radio\" name=\"ib_summary\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
+               "<input type=\"radio\" name=\"ib_summary\" value=\"no\" %s> %s <br />"
+               "</td><td>"
+               "<img src=\"static/summscreen_48x.gif\" alt=\"&nbsp;\">"
+               "</td><td>"
+               "<b>%s</b><br />"
                "%s"
-               "</TD></TR>\n",
-               ((bar = 1 - bar), (bar ? "\"#CCCCCC\"" : "\"#FFFFFF\"")),
-               (ib_summary ? "CHECKED" : ""),
+               "</td></tr>\n",
+               (bar ? "even" : "odd"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Summary"),
                _("Your summary page")
        );
 
-       wprintf("<TR bgcolor=%s><TD>"
-               "<INPUT TYPE=\"checkbox\" NAME=\"ib_inbox\" VALUE=\"yes\" %s>"
-               "</TD><TD>"
-               "<IMG BORDER=\"0\" WIDTH=\"48\" HEIGHT=\"48\" "
-               "src=\"static/privatemess_48x.gif\" ALT=\"&nbsp;\">"
-               "</TD><TD>"
-               "<B>%s</B><br />"
+       bar = 1 - bar;
+       val = IconbarIsEnabled("ib_inbox", 1);
+       wprintf("<tr class=\"%s\"><td>"
+               "<input type=\"radio\" name=\"ib_inbox\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
+               "<input type=\"radio\" name=\"ib_inbox\" value=\"no\" %s> %s <br />"
+               "</td><td>"
+               "<img src=\"static/privatemess_48x.gif\" alt=\"&nbsp;\">"
+               "</td><td>"
+               "<b>%s</b><br />"
                "%s"
-               "</TD></TR>\n",
-               ((bar = 1 - bar), (bar ? "\"#CCCCCC\"" : "\"#FFFFFF\"")),
-               (ib_inbox ? "CHECKED" : ""),
+               "</td></tr>\n",
+               (bar ? "even" : "odd"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Mail (inbox)"),
                _("A shortcut to your email Inbox")
        );
 
-       wprintf("<TR bgcolor=%s><TD>"
-               "<INPUT TYPE=\"checkbox\" NAME=\"ib_contacts\" "
-               "VALUE=\"yes\" %s>"
-               "</TD><TD>"
-               "<IMG BORDER=\"0\" WIDTH=\"48\" HEIGHT=\"48\" "
-               "src=\"static/viewcontacts_48x.gif\" ALT=\"&nbsp;\">"
-               "</TD><TD>"
-               "<B>%s</B><br />"
+       bar = 1 - bar;
+       val = IconbarIsEnabled("ib_contacts", 1);
+       wprintf("<tr class=\"%s\"><td>"
+               "<input type=\"radio\" name=\"ib_contacts\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
+               "<input type=\"radio\" name=\"ib_contacts\" value=\"no\" %s> %s <br />"
+               "</td><td>"
+               "<img src=\"static/viewcontacts_48x.gif\" alt=\"&nbsp;\">"
+               "</td><td>"
+               "<b>%s</b><br />"
                "%s"
-               "</TD></TR>\n",
-               ((bar = 1 - bar), (bar ? "\"#CCCCCC\"" : "\"#FFFFFF\"")),
-               (ib_contacts ? "CHECKED" : ""),
+               "</td></tr>\n",
+               (bar ? "even" : "odd"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Contacts"),
                _("Your personal address book")
        );
 
-       wprintf("<TR bgcolor=%s><TD>"
-               "<INPUT TYPE=\"checkbox\" NAME=\"ib_notes\" "
-               "VALUE=\"yes\" %s>"
-               "</TD><TD>"
-               "<IMG BORDER=\"0\" WIDTH=\"48\" HEIGHT=\"48\" "
-               "src=\"static/storenotes_48x.gif\" ALT=\"&nbsp;\">"
-               "</TD><TD>"
-               "<B>%s</B><br />"
+       bar = 1 - bar;
+       val = IconbarIsEnabled("ib_notes", 1);
+       wprintf("<tr class=\"%s\"><td>"
+               "<input type=\"radio\" name=\"ib_notes\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
+               "<input type=\"radio\" name=\"ib_notes\" value=\"no\" %s> %s <br />"
+               "</td><td>"
+               "<img src=\"static/storenotes_48x.gif\" alt=\"&nbsp;\">"
+               "</td><td>"
+               "<b>%s</b><br />"
                "%s"
-               "</TD></TR>\n",
-               ((bar = 1 - bar), (bar ? "\"#CCCCCC\"" : "\"#FFFFFF\"")),
-               (ib_notes ? "CHECKED" : ""),
+               "</td></tr>\n",
+               (bar ? "even" : "odd"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Notes"),
                _("Your personal notes")
        );
 
-#ifdef WEBCIT_WITH_CALENDAR_SERVICE
-       wprintf("<TR bgcolor=%s><TD>"
-               "<INPUT TYPE=\"checkbox\" NAME=\"ib_calendar\" "
-               "VALUE=\"yes\" %s>"
-               "</TD><TD>"
-               "<IMG BORDER=\"0\" WIDTH=\"48\" HEIGHT=\"48\" "
-               "src=\"static/calarea_48x.gif\" ALT=\"&nbsp;\">"
-               "</TD><TD>"
-               "<B>%s</B><br />"
+       bar = 1 - bar;
+       val = IconbarIsEnabled("ib_calendar", 1);
+       wprintf("<tr class=\"%s\"><td>"
+               "<input type=\"radio\" name=\"ib_calendar\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
+               "<input type=\"radio\" name=\"ib_calendar\" value=\"no\" %s> %s <br />"
+               "</td><td>"
+               "<img src=\"static/calarea_48x.gif\" alt=\"&nbsp;\">"
+               "</td><td>"
+               "<b>%s</b><br />"
                "%s"
-               "</TD></TR>\n",
-               ((bar = 1 - bar), (bar ? "\"#CCCCCC\"" : "\"#FFFFFF\"")),
-               (ib_calendar ? "CHECKED" : ""),
+               "</td></tr>\n",
+               (bar ? "even" : "odd"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Calendar"),
                _("A shortcut to your personal calendar")
        );
 
-       wprintf("<TR bgcolor=%s><TD>"
-               "<INPUT TYPE=\"checkbox\" NAME=\"ib_tasks\" VALUE=\"yes\" %s>"
-               "</TD><TD>"
-               "<IMG BORDER=\"0\" WIDTH=\"48\" HEIGHT=\"48\" "
-               "src=\"static/taskmanag_48x.gif\" ALT=\"&nbsp;\">"
-               "</TD><TD>"
-               "<B>%s</B><br />"
+       bar = 1 - bar;
+       val = IconbarIsEnabled("ib_tasks", 1);
+       wprintf("<tr class=\"%s\"><td>"
+               "<input type=\"radio\" name=\"ib_tasks\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
+               "<input type=\"radio\" name=\"ib_tasks\" value=\"no\" %s> %s <br />"
+               "</td><td>"
+               "<img src=\"static/taskmanag_48x.gif\" alt=\"&nbsp;\">"
+               "</td><td>"
+               "<b>%s</b><br />"
                "%s"
-               "</TD></TR>\n",
-               ((bar = 1 - bar), (bar ? "\"#CCCCCC\"" : "\"#FFFFFF\"")),
-               (ib_tasks ? "CHECKED" : ""),
+               "</td></tr>\n",
+               (bar ? "even" : "odd"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Tasks"),
                _("A shortcut to your personal task list")
        );
-#endif /* WEBCIT_WITH_CALENDAR_SERVICE */
-
-       wprintf("<TR bgcolor=%s><TD>"
-               "<INPUT TYPE=\"checkbox\" NAME=\"ib_rooms\" VALUE=\"yes\" %s>"
-               "</TD><TD>"
-               "<IMG BORDER=\"0\" WIDTH=\"48\" HEIGHT=\"48\" "
-               "src=\"static/chatrooms_48x.gif\" ALT=\"&nbsp;\">"
-               "</TD><TD>"
-               "<B>%s</B><br />"
+
+       bar = 1 - bar;
+       val = IconbarIsEnabled("ib_rooms", 1);
+       wprintf("<tr class=\"%s\"><td>"
+               "<input type=\"radio\" name=\"ib_rooms\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
+               "<input type=\"radio\" name=\"ib_rooms\" value=\"no\" %s> %s <br />"
+               "</td><td>"
+               "<img src=\"static/chatrooms_48x.gif\" alt=\"&nbsp;\">"
+               "</td><td>"
+               "<b>%s</b><br />"
                "%s"
-               "</TD></TR>\n",
-               ((bar = 1 - bar), (bar ? "\"#CCCCCC\"" : "\"#FFFFFF\"")),
-               (ib_rooms ? "CHECKED" : ""),
+               "</td></tr>\n",
+               (bar ? "even" : "odd"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Rooms"),
-               _("Clicking this icon displays a list of all accesible "
+               _("Clicking this icon displays a list of all accessible "
                "rooms (or folders) available.")
        );
 
-       wprintf("<TR bgcolor=%s><TD>"
-               "<INPUT TYPE=\"checkbox\" NAME=\"ib_users\" VALUE=\"yes\" %s>"
-               "</TD><TD>"
-               "<IMG BORDER=\"0\" WIDTH=\"48\" HEIGHT=\"48\" "
-               "src=\"static/usermanag_48x.gif\" ALT=\"&nbsp;\">"
-               "</TD><TD>"
-               "<B>%s</B><br />"
-               "%s"
-               "</TD></TR>\n",
-               ((bar = 1 - bar), (bar ? "\"#CCCCCC\"" : "\"#FFFFFF\"")),
-               (ib_users ? "CHECKED" : ""),
+       bar = 1 - bar;
+       val = IconbarIsEnabled("ib_users", 1);
+       wprintf("<tr class=\"%s\"><td>"
+               "<input type=\"radio\" name=\"ib_users\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
+               "<input type=\"radio\" name=\"ib_users\" value=\"no\" %s> %s <br />"
+               "<input type=\"radio\" name=\"ib_users\" value=\"yeslist\" %s> %s"
+               "</td><td>"
+               "<img src=\"static/usermanag_48x.gif\" alt=\"&nbsp;\">"
+               "</td><td>"
+               "<b>%s</b>"
+               "<br />%s"
+               "</td></tr>\n",
+               (bar ? "even" : "odd"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
+               ((val > 1) ? "CHECKED" : ""),_("Yes with users list"),
                _("Who is online?"),
                _("Clicking this icon displays a list of all users "
                "currently logged in.")
        );
 
-       wprintf("<TR bgcolor=%s><TD>"
-               "<INPUT TYPE=\"checkbox\" NAME=\"ib_chat\" VALUE=\"yes\" %s>"
-               "</TD><TD>"
-               "<IMG BORDER=\"0\" WIDTH=\"48\" HEIGHT=\"48\" "
-               "src=\"static/citadelchat_48x.gif\" ALT=\"&nbsp;\">"
-               "</TD><TD>"
-               "<B>%s</B><br />"
+       bar = 1 - bar;
+       val = IconbarIsEnabled("ib_chat", 1);
+       wprintf("<tr class=\"%s\"><td>"
+               "<input type=\"radio\" name=\"ib_chat\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
+               "<input type=\"radio\" name=\"ib_chat\" value=\"no\" %s> %s <br />"
+               "</td><td>"
+               "<img src=\"static/citadelchat_48x.gif\" alt=\"&nbsp;\">"
+               "</td><td>"
+               "<b>%s</b><br />"
                "%s"
-               "</TD></TR>\n",
-               ((bar = 1 - bar), (bar ? "\"#CCCCCC\"" : "\"#FFFFFF\"")),
-               (ib_chat ? "CHECKED" : ""),
+               "</td></tr>\n",
+               (bar ? "even" : "odd"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Chat"),
                _("Clicking this icon enters real-time chat mode "
                "with other users in the same room.")
                
        );
 
-       wprintf("<TR bgcolor=%s><TD>"
-               "<INPUT TYPE=\"checkbox\" NAME=\"ib_advanced\" "
-               "VALUE=\"yes\" %s>"
-               "</TD><TD>"
-               "<IMG BORDER=\"0\" WIDTH=\"48\" HEIGHT=\"48\" "
-               "src=\"static/advanpage2_48x.gif\" ALT=\"&nbsp;\">"
-               "</TD><TD>"
-               "<B>%s</B><br />"
+       bar = 1 - bar;
+       val = IconbarIsEnabled("ib_advanced", 1);
+       wprintf("<tr class=\"%s\"><td>"
+               "<input type=\"radio\" name=\"ib_advanced\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
+               "<input type=\"radio\" name=\"ib_advanced\" value=\"no\" %s> %s <br />"
+               "</td><td>"
+               "<img src=\"static/advanpage2_48x.gif\" alt=\"&nbsp;\">"
+               "</td><td>"
+               "<b>%s</b><br />"
                "%s"
-               "</TD></TR>\n",
-               ((bar = 1 - bar), (bar ? "\"#CCCCCC\"" : "\"#FFFFFF\"")),
-               (ib_advanced ? "CHECKED" : ""),
+               "</td></tr>\n",
+               (bar ? "even" : "odd"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Advanced options"),
                _("Access to the complete menu of Citadel functions.")
 
        );
 
-       wprintf("<TR bgcolor=%s><TD>"
-               "<INPUT TYPE=\"checkbox\" NAME=\"ib_citadel\" "
-               "VALUE=\"yes\" %s>"
-               "</TD><TD>"
-               "<IMG BORDER=\"0\" WIDTH=\"48\" HEIGHT=\"48\" "
-               "src=\"static/citadel-logo.gif\" ALT=\"&nbsp;\">"
-               "</TD><TD>"
-               "<B>%s</B><br />"
+       bar = 1 - bar;
+       val = IconbarIsEnabled("ib_citadel", 1);
+       wprintf("<tr class=\"%s\"><td>"
+               "<input type=\"radio\" name=\"ib_citadel\" value=\"yes\" %s> %s &nbsp;&nbsp;&nbsp;"
+               "<input type=\"radio\" name=\"ib_citadel\" value=\"no\" %s> %s <br />"
+               "</td><td>"
+               "<img border=\"0\" width=\"48\" height=\"48\" "
+               "src=\"static/citadel-logo.gif\" alt=\"&nbsp;\">"
+               "</td><td>"
+               "<b>%s</b><br />"
                "%s"
-               "</TD></TR>\n",
-               ((bar = 1 - bar), (bar ? "\"#CCCCCC\"" : "\"#FFFFFF\"")),
-               (ib_citadel ? "CHECKED" : ""),
+               "</td></tr>\n",
+               (bar ? "even" : "odd"),
+               (val ? "CHECKED" : ""),_("Yes"),
+               (!val ? "CHECKED" : ""),_("No"),
                _("Citadel logo"),
                _("Displays the 'Powered by Citadel' icon")
        );
 
-       wprintf("</TABLE><br />\n"
-               "<CENTER>"
-               "<INPUT TYPE=\"submit\" NAME=\"ok_button\" VALUE=\"%s\">"
+       wprintf("</table><br />\n"
+               "<center>"
+               "<input type=\"submit\" name=\"ok_button\" value=\"%s\">"
                "&nbsp;"
-               "<INPUT TYPE=\"submit\" NAME=\"cancel_button\" VALUE=\"%s\">"
-               "</CENTER></FORM>\n",
+               "<input type=\"submit\" name=\"cancel_button\" value=\"%s\">"
+               "</center>\n",
                _("Save changes"),
                _("Cancel")
        );
 
-       wprintf("</td></tr></table></div>\n");
+       wprintf("</form></div>\n");
        wDumpContent(2);
 }
 
-
+/**
+ * \brief commit the changes of an edited iconbar ????
+ */
 void commit_iconbar(void) {
-       char iconbar[SIZ];
+       StrBuf *iconbar;
+       StrBuf *buf;
        int i;
 
        char *boxen[] = {
@@ -724,26 +756,39 @@ void commit_iconbar(void) {
                "ib_citadel"
        };
 
-       if (strlen(bstr("ok_button")) == 0) {
+       if (!havebstr("ok_button")) {
                display_main_menu();
                return;
        }
 
-       sprintf(iconbar, "ib_displayas=%d", atoi(bstr("ib_displayas")));
-
+       iconbar = NewStrBuf();
+       buf = NewStrBuf();
+       StrBufPrintf(iconbar, "ib_displayas=%d", ibstr("ib_displayas"));
        for (i=0; i<(sizeof(boxen)/sizeof(char *)); ++i) {
-               sprintf(&iconbar[strlen(iconbar)], ",%s=", boxen[i]);
-               if (!strcasecmp(bstr(boxen[i]), "yes")) {
-                       sprintf(&iconbar[strlen(iconbar)], "1");
+               char *Val;
+               if (!strcasecmp(BSTR(boxen[i]), "yes")) {
+                       Val = "1";
+               }
+               else if (!strcasecmp(BSTR(boxen[i]), "yeslist")) {
+                       Val = "2";
                }
                else {
-                       sprintf(&iconbar[strlen(iconbar)], "0");
+                       Val = "0";
                }
-       }
+               StrBufPrintf(buf, ",%s=%s", boxen[i], Val);
+               StrBufAppendBuf(iconbar, buf, 0);
 
+       }
+       FreeStrBuf(&buf);
        set_preference("iconbar", iconbar, 1);
 
-       output_headers(1, 1, 0, 0, 0, 0);
+       output_headers(1, 1, 2, 0, 0, 0);
+       wprintf("<div id=\"banner\">\n");
+       wprintf("<h1>");
+       wprintf(_("Customize the icon bar"));
+       wprintf("</h1></div>\n");
+
+       wprintf("<div id=\"content\" class=\"service\">\n");
        wprintf(
                "<center><table border=1 bgcolor=\"#ffffff\"><tr><td>"
                "<img src=\"static/advanpage2_48x.gif\">"
@@ -752,5 +797,36 @@ void commit_iconbar(void) {
                "choices to continue."));
        wprintf("</td></tr></table>\n");
        wDumpContent(2);
+#ifdef DBG_ICONBAR_HASH
+       dbg_PrintHash(WC->IconBarSetttings, PrintInt, NULL);
+#endif
 }
 
+
+void tmplput_iconbar(StrBuf *Target, int nArgs, WCTemplateToken *Tokens, void *Context, int ContextType)
+{
+       struct wcsession *WCC = WC;
+       
+       if ((WCC != NULL) && (WCC->logged_in)) {
+               wprintf("<div id=\"iconbar\">");
+               do_selected_iconbar();
+               /** check for instant messages (these display in a new window) */
+               page_popup();
+               wprintf("</div>");
+       }
+}
+
+void 
+InitModule_ICONBAR
+(void)
+{
+       WebcitAddUrlHandler(HKEY("iconbar_ajax_menu"), do_iconbar, AJAX);
+       WebcitAddUrlHandler(HKEY("iconbar_ajax_rooms"), do_iconbar_roomlist, AJAX);
+       WebcitAddUrlHandler(HKEY("display_customize_iconbar"), display_customize_iconbar, 0);
+       WebcitAddUrlHandler(HKEY("commit_iconbar"), commit_iconbar, 0);
+       RegisterNamespace("ICONBAR", 0, 0, tmplput_iconbar, 0);
+
+}
+
+
+/*@}*/