5 files changed, 39 insertions, 9 deletions

diff --git a/res/about/version.gmi b/res/about/version.gmi
index f3cd21c0..08f39691 100644
--- a/res/about/version.gmi
+++ b/res/about/version.gmi
@@ -7,6 +7,10 @@
 # Release notes
 
 ## 1.9.2
+* Fixed possible crash when switching Feeds sidebar filter mode via the popup menu.
+* Shorter label for "Mark All as Read" in Feeds sidebar actions.
+* In popup menus, differentiate between selected item and hover item.
+* Updated UI translations.
 * Windows: Use the correct version number for update checks.
 
 ## 1.9.1
@@ -20,7 +24,6 @@
 * Fixed the New Tab button not staying at the right edge of the window, depending on how many tabs are open.
 
 ## 1.9
-
 New features:
 * Added a toolbar button for toggling the left sidebar.
 * Added an unsplit button in the toolbar when in split view mode.
diff --git a/src/ui/labelwidget.c b/src/ui/labelwidget.c
index f6ddfd43..4dd66a28 100644
--- a/src/ui/labelwidget.c
+++ b/src/ui/labelwidget.c
@@ -198,6 +198,7 @@ static void getColors_LabelWidget_(const iLabelWidget *d, int *bg, int *fg, int
     const iBool    isSel       = (flags & selected_WidgetFlag) != 0;
     const iBool    isFrameless = (flags & frameless_WidgetFlag) != 0;
     const iBool    isButton    = d->click.button != 0;
+    const iBool    isMenuItem  = !cmp_String(id_Widget(parent_Widget(d)), "menu");
     const iBool    isKeyRoot   = (w->root == get_Window()->keyRoot);
     const iBool    isDarkTheme = isDark_ColorTheme(colorTheme_App());
     /* Default color state. */
@@ -215,7 +216,12 @@ static void getColors_LabelWidget_(const iLabelWidget *d, int *bg, int *fg, int
         *meta = uiTextDisabled_ColorId;
     }
     if (isSel) {
-        *bg = uiBackgroundSelected_ColorId;
+        if (isMenuItem) {
+            *bg = uiBackgroundUnfocusedSelection_ColorId;
+        }
+        else {
+            *bg = uiBackgroundSelected_ColorId;            
+        }
 //        if (!isKeyRoot) {
 //            *bg = uiEmbossSelected1_ColorId; //uiBackgroundUnfocusedSelection_ColorId;
 //        }
diff --git a/src/ui/sidebarwidget.c b/src/ui/sidebarwidget.c
index 3d663afd..2219eba9 100644
--- a/src/ui/sidebarwidget.c
+++ b/src/ui/sidebarwidget.c
@@ -264,10 +264,12 @@ static iBool isBookmarkFolded_SidebarWidget_(const iSidebarWidget *d, const iBoo
     return iFalse;
 }
 
-static void updateItems_SidebarWidget_(iSidebarWidget *d) {
+static void updateItemsWithFlags_SidebarWidget_(iSidebarWidget *d, iBool keepActions) {
     clear_ListWidget(d->list);
     releaseChildren_Widget(d->blank);
+    if (!keepActions) {
     releaseChildren_Widget(d->actions);
+    }
     d->actions->rect.size.y = 0;
     destroy_Widget(d->menu);
     destroy_Widget(d->modeMenu);
@@ -348,7 +350,8 @@ static void updateItems_SidebarWidget_(iSidebarWidget *d) {
                     break;
                 }
             }
-            /* Actions. */ {
+            /* Actions. */
+            if (!keepActions) {
                 addActionButton_SidebarWidget_(
                     d, check_Icon " ${sidebar.action.feeds.markallread}", "feeds.markallread", expand_WidgetFlag |
                     tight_WidgetFlag);
@@ -362,6 +365,8 @@ static void updateItems_SidebarWidget_(iSidebarWidget *d) {
                 iWidget *dropButton = addChild_Widget(
                     d->actions,
                     iClob(makeMenuButton_LabelWidget(items[d->feedsMode].label, items, 2)));
+                setId_Widget(dropButton, "feeds.modebutton");
+                checkIcon_LabelWidget((iLabelWidget *) dropButton);
                 setFixedSize_Widget(
                     dropButton,
                     init_I2(iMaxi(20 * gap_UI, measure_Text(
@@ -371,6 +376,10 @@ static void updateItems_SidebarWidget_(iSidebarWidget *d) {
                                                    6 * gap_UI),
                             -1));
             }
+            else {
+                updateDropdownSelection_LabelWidget(findChild_Widget(d->actions, "feeds.modebutton"),
+                                                    format_CStr(" arg:%d", d->feedsMode));
+            }
             d->menu = makeMenu_Widget(
                 as_Widget(d),
                 (iMenuItem[]){ { openTab_Icon " ${feeds.entry.newtab}", 0, 0, "feed.entry.opentab" },
@@ -653,6 +662,10 @@ static void updateItems_SidebarWidget_(iSidebarWidget *d) {
     updateMouseHover_ListWidget(d->list);
 }
 
+static void updateItems_SidebarWidget_(iSidebarWidget *d) {
+    updateItemsWithFlags_SidebarWidget_(d, iFalse);
+}
+
 static size_t findItem_SidebarWidget_(const iSidebarWidget *d, int id) {
     /* Note that this is O(n), so only meant for infrequent use. */
     for (size_t i = 0; i < numItems_ListWidget(d->list); i++) {
@@ -1315,9 +1328,9 @@ static iBool processEvent_SidebarWidget_(iSidebarWidget *d, const SDL_Event *ev)
             }
             return iTrue;
         }
-        else if (isCommand_Widget(w, ev, "menu.closed")) {
+//        else if (isCommand_Widget(w, ev, "menu.closed")) {
          //   invalidateItem_ListWidget(d->list, d->contextIndex);
-        }
+//        }
         else if (isCommand_Widget(w, ev, "bookmark.open")) {
             const iSidebarItem *item = d->contextItem;
             if (d->mode == bookmarks_SidebarMode && item) {
@@ -1474,7 +1487,7 @@ static iBool processEvent_SidebarWidget_(iSidebarWidget *d, const SDL_Event *ev)
         }
         else if (equalWidget_Command(cmd, w, "feeds.mode")) {
             d->feedsMode = arg_Command(cmd);
-            updateItems_SidebarWidget_(d);
+            updateItemsWithFlags_SidebarWidget_(d, iTrue);
             return iTrue;
         }
         else if (equal_Command(cmd, "feeds.markallread") && d->mode == feeds_SidebarMode) {
diff --git a/src/ui/util.c b/src/ui/util.c
index 84de4b74..cfb2b230 100644
--- a/src/ui/util.c
+++ b/src/ui/util.c
@@ -1036,7 +1036,8 @@ void openMenuFlags_Widget(iWidget *d, iInt2 windowCoord, int menuOpenFlags) {
     setFlags_Widget(findChild_Widget(d, "menu.cancel"), disabled_WidgetFlag, iFalse);
     arrange_Widget(d); /* need to know the height */
     iBool allowOverflow = iFalse;
-    /* A vertical offset determined by a possible selected label in the menu. */ {
+    /* A vertical offset determined by a possible selected label in the menu. */ 
+    if (windowCoord.y < rootSize.y - lineHeight_Text(uiNormal_FontSize) * 3) {
         iConstForEach(ObjectList, child, children_Widget(d)) {
             const iWidget *item = constAs_Widget(child.object);
             if (flags_Widget(item) & selected_WidgetFlag) {
@@ -1269,6 +1270,9 @@ const iString *removeMenuItemLabelPrefixes_String(const iString *d) {
 }
 
 void updateDropdownSelection_LabelWidget(iLabelWidget *dropButton, const char *selectedCommand) {
+    if (!dropButton) {
+        return;
+    }
     iWidget *menu = findChild_Widget(as_Widget(dropButton), "menu");
     if (flags_Widget(menu) & nativeMenu_WidgetFlag) {
         unselectAllNativeMenuItems_Widget(menu);
@@ -1277,6 +1281,7 @@ void updateDropdownSelection_LabelWidget(iLabelWidget *dropButton, const char *s
             setSelected_NativeMenuItem(item, iTrue);
             updateText_LabelWidget(
                 dropButton, removeMenuItemLabelPrefixes_String(collectNewCStr_String(item->label)));
+            checkIcon_LabelWidget(dropButton);
         }
         return;
     }
@@ -1287,6 +1292,7 @@ void updateDropdownSelection_LabelWidget(iLabelWidget *dropButton, const char *s
             setFlags_Widget(as_Widget(item), selected_WidgetFlag, isSelected);
             if (isSelected) {
                 updateText_LabelWidget(dropButton, sourceText_LabelWidget(item));
+                checkIcon_LabelWidget(dropButton);
             }
         }
     }
@@ -2352,6 +2358,7 @@ iWidget *makePreferences_Widget(void) {
             { "radio device:1 id:prefs.pinsplit", 0, 0, (const void *) pinSplitItems },
             { "padding" },
             { "dropdown id:prefs.uilang", 0, 0, (const void *) langItems },
+            { "toggle id:prefs.time.24h" },
             { NULL }
         };
         const iMenuItem uiPanelItems[] = {
@@ -2436,6 +2443,7 @@ iWidget *makePreferences_Widget(void) {
         const iMenuItem aboutPanelItems[] = {
             { format_CStr("heading text:%s", cstr_String(aboutText)) },
             { "button text:" clock_Icon " ${menu.releasenotes}", 0, 0, "!open url:about:version" },
+            { "padding" },
             { "button text:" globe_Icon " ${menu.website}", 0, 0, "!open url:https://gmi.skyjake.fi/lagrange" },
             { "button text:" envelope_Icon " @jk@skyjake.fi", 0, 0, "!open url:https://skyjake.fi/@jk" },
             { "padding" },
diff --git a/src/ui/widget.c b/src/ui/widget.c
index a171a6cd..cedda461 100644
--- a/src/ui/widget.c
+++ b/src/ui/widget.c
@@ -1528,7 +1528,7 @@ static void addToPotentiallyVisible_Widget_(const iWidget *d, iPtrArray *pvs, iR
         }
         if (isFullyContainedByOther_Rect(bounds, *fullyMasked)) {
             return; /* can't be seen */
-        }                
+        }
         pushBack_PtrArray(pvs, d);
         if (d->bgColor >= 0 && ~d->flags & noBackground_WidgetFlag &&
             isFullyContainedByOther_Rect(*fullyMasked, bounds)) {