4 files changed, 74 insertions, 15 deletions

diff --git a/src/ui/sidebarwidget.c b/src/ui/sidebarwidget.c
index 82efb04e..aa2e185d 100644
--- a/src/ui/sidebarwidget.c
+++ b/src/ui/sidebarwidget.c
@@ -22,11 +22,12 @@ SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. */
 
 #include "sidebarwidget.h"
 
-#include "../gmdocument.h"
 #include "app.h"
 #include "bookmarks.h"
 #include "command.h"
 #include "documentwidget.h"
+#include "gmcerts.h"
+#include "gmdocument.h"
 #include "inputwidget.h"
 #include "labelwidget.h"
 #include "paint.h"
@@ -216,6 +217,33 @@ static void updateItems_SidebarWidget_(iSidebarWidget *d) {
                 }, 6);
             break;
         }
+        case identities_SidebarMode: {
+            iConstForEach(PtrArray, i, identities_GmCerts(certs_App())) {
+                const iGmIdentity *ident = i.ptr;
+                /* icon, common name, used URLs, expiration, isTemp, free-form notes */
+                iSidebarItem item;
+                init_SidebarItem(&item);
+                item.icon = ident->icon;
+                set_String(&item.label, collect_String(subject_TlsCertificate(ident->cert)));
+                iDate until;
+                validUntil_TlsCertificate(ident->cert, &until);
+                set_String(
+                    &item.meta,
+                    collectNewFormat_String(
+                        "%s \u2014 %s\n%s",
+                        isEmpty_SortedArray(&ident->useUrls)
+                            ? "Not used"
+                            : format_CStr("Used on %zu URLs", size_SortedArray(&ident->useUrls)),
+                        ident->flags & temporary_GmIdentityFlag
+                            ? "Temporary"
+                            : cstrCollect_String(format_Date(&until, "Expires %b %d, %Y")),
+                        cstr_String(&ident->notes)));
+                pushBack_Array(&d->items, &item);
+            }
+            /* menu: set icon, edit notes, view urls, reveal files, delete, activate on this page,
+             * deactivate on this page, deactivate everywhere  */
+            break;
+        }
         default:
             break;
     }
@@ -247,7 +275,7 @@ void setMode_SidebarWidget(iSidebarWidget *d, enum iSidebarMode mode) {
     for (enum iSidebarMode i = 0; i < max_SidebarMode; i++) {
         setFlags_Widget(as_Widget(d->modeButtons[i]), selected_WidgetFlag, i == d->mode);
     }
-    const float heights[max_SidebarMode] = { 1.333f, 1.333f, 2.5f, 1.2f };
+    const float heights[max_SidebarMode] = { 1.333f, 1.333f, 3.0f, 1.2f };
     d->itemHeight = heights[mode] * lineHeight_Text(uiContent_FontId);
     invalidate_SidebarWidget_(d);
     /* Restore previous scroll position. */
@@ -529,7 +557,10 @@ static iBool processEvent_SidebarWidget_(iSidebarWidget *d, const SDL_Event *ev)
             }
             return iTrue;
         }
-        else if (equal_Command(cmd, "bookmarks.changed")) {
+        else if (equal_Command(cmd, "bookmarks.changed") && d->mode == bookmarks_SidebarMode) {
+            updateItems_SidebarWidget_(d);
+        }
+        else if (equal_Command(cmd, "idents.changed") && d->mode == identities_SidebarMode) {
             updateItems_SidebarWidget_(d);
         }
         else if (equal_Command(cmd, "history.delete")) {
@@ -661,6 +692,9 @@ static void draw_SidebarWidget_(const iSidebarWidget *d) {
                 const iSidebarItem *item     = constAt_Array(&d->items, i);
                 const iRect         itemRect = { pos, init_I2(width_Rect(bufBounds), d->itemHeight) };
                 const iBool         isHover  = (d->hoverItem == i);
+                const int           iconColor =
+                    isHover ? (isPressing ? uiTextPressed_ColorId : uiIconHover_ColorId)
+                            : uiIcon_ColorId;
                 setClip_Paint(&p, intersect_Rect(itemRect, bufBounds));
                 if (isHover && !item->isSeparator) {
                     fillRect_Paint(&p,
@@ -691,8 +725,7 @@ static void draw_SidebarWidget_(const iSidebarWidget *d) {
                         font,
                         iconArea,
                         iTrue,
-                        isHover ? (isPressing ? uiTextPressed_ColorId : uiIconHover_ColorId)
-                                : uiIcon_ColorId,
+                        iconColor,
                         "%s",
                         cstr_String(&str));
                     deinit_String(&str);
@@ -737,6 +770,25 @@ static void draw_SidebarWidget_(const iSidebarWidget *d) {
                     }
                     iEndCollect();
                 }
+                else if (d->mode == identities_SidebarMode) {
+                    const int fg = isHover ? (isPressing ? uiTextPressed_ColorId
+                                                         : uiTextFramelessHover_ColorId)
+                                           : uiText_ColorId;
+                    iString icon;
+                    initUnicodeN_String(&icon, &item->icon, 1);
+                    drawRange_Text(font, add_I2(topLeft_Rect(itemRect), init_I2(3 * gap_UI, 0)),
+                                   iconColor, range_String(&icon));
+                    deinit_String(&icon);
+                    drawRange_Text(font, add_I2(topLeft_Rect(itemRect), init_I2(10 * gap_UI, 0)),
+                                   fg, range_String(&item->label));
+                    drawRange_Text(
+                        font,
+                        add_I2(topLeft_Rect(itemRect), init_I2(3 * gap_UI, lineHeight_Text(font))),
+                        isHover
+                            ? (isPressing ? uiTextPressed_ColorId : uiTextFramelessHover_ColorId)
+                            : uiAnnotation_ColorId,
+                        range_String(&item->meta));
+                }
                 unsetClip_Paint(&p);
                 pos.y += d->itemHeight;
             }
diff --git a/src/ui/util.c b/src/ui/util.c
index b3178e34..e6067ab9 100644
--- a/src/ui/util.c
+++ b/src/ui/util.c
@@ -689,9 +689,11 @@ iWidget *makeQuestion_Widget(const char *title,
 }
 
 void setToggle_Widget(iWidget *d, iBool active) {
-    setFlags_Widget(d, selected_WidgetFlag, active);
-    updateText_LabelWidget((iLabelWidget *) d,
-                           collectNewFormat_String("%s", isSelected_Widget(d) ? "YES" : "NO"));
+    if (d) {
+        setFlags_Widget(d, selected_WidgetFlag, active);
+        updateText_LabelWidget((iLabelWidget *) d,
+                               collectNewFormat_String("%s", isSelected_Widget(d) ? "YES" : "NO"));
+    }
 }
 
 static iBool toggleHandler_(iWidget *d, const char *cmd) {
diff --git a/src/ui/widget.c b/src/ui/widget.c
index f187bb82..b5ea3b0f 100644
--- a/src/ui/widget.c
+++ b/src/ui/widget.c
@@ -637,7 +637,7 @@ iBool isHover_Widget(const iWidget *d) {
 }
 
 iBool isSelected_Widget(const iWidget *d) {
-    return (d->flags & selected_WidgetFlag) != 0;
+    return d && (d->flags & selected_WidgetFlag) != 0;
 }
 
 iBool equalWidget_Command(const char *cmd, const iWidget *widget, const char *checkCommand) {
diff --git a/src/ui/window.c b/src/ui/window.c
index fad87f08..709fc0e0 100644
--- a/src/ui/window.c
+++ b/src/ui/window.c
@@ -102,6 +102,8 @@ static const iMenuItem navMenuItems[] = {
     { "Reset Zoom", SDLK_0, KMOD_PRIMARY, "zoom.set arg:100" },
     { "---", 0, 0, NULL },
     { "Preferences...", SDLK_COMMA, KMOD_PRIMARY, "preferences" },
+    { "Help", 0, 0, "!open url:about:help" },
+    { "Release Notes", 0, 0, "!open url:about:version" },
     { "---", 0, 0, NULL },
     { "Quit Lagrange", 'q', KMOD_PRIMARY, "quit" }
 };
@@ -121,10 +123,6 @@ static const iMenuItem editMenuItems[] = {
     { "Bookmark This Page...", SDLK_d, KMOD_PRIMARY, "bookmark.add" },
 };
 
-static const iMenuItem identityMenuItems[] = {
-    { "New Identity...", SDLK_n, KMOD_PRIMARY | KMOD_SHIFT, "ident.new" },
-};
-
 static const iMenuItem viewMenuItems[] = {
     { "Show Bookmarks", '1', KMOD_PRIMARY, "sidebar.mode arg:0 show:1" },
     { "Show History", '2', KMOD_PRIMARY, "sidebar.mode arg:1 show:1" },
@@ -147,6 +145,10 @@ static const iMenuItem helpMenuItems[] = {
 };
 #endif
 
+static const iMenuItem identityMenuItems[] = {
+    { "New Identity...", SDLK_n, KMOD_PRIMARY | KMOD_SHIFT, "ident.new" },
+};
+
 static const char *reloadCStr_ = "\U0001f503";
 static const char *stopCStr_   = uiTextCaution_ColorEscape "\U0001f310";
 
@@ -312,8 +314,11 @@ static void setupUserInterface_Window(iWindow *d) {
         setId_Widget(
             addChild_Widget(navBar, iClob(newIcon_LabelWidget(reloadCStr_, 0, 0, "navigate.reload"))),
             "reload");
-        addChild_Widget(navBar, iClob(newIcon_LabelWidget("\U0001f464", 0, 0, "cert.client")));
-
+        iLabelWidget *idMenu =
+            makeMenuButton_LabelWidget("\U0001f464", identityMenuItems, iElemCount(identityMenuItems));
+        setAlignVisually_LabelWidget(idMenu, iTrue);
+        addChild_Widget(navBar, iClob(idMenu));
+        //addChild_Widget(navBar, iClob(newIcon_LabelWidget("\U0001f464", 0, 0, "cert.client")));
 #if !defined (iHaveNativeMenus)
         iLabelWidget *navMenu =
             makeMenuButton_LabelWidget("\U0001d362", navMenuItems, iElemCount(navMenuItems));