[Bug]: Tag Filter Menu Doesn't Close on Button Press #25

New issue

Closed

opened 2026-06-09 04:58:03 +00:00 by ValleyGeek · 3 comments

ValleyGeek commented

2026-06-09 04:58:03 +00:00

Owner

App version

0.1.1

Platform

Windows

Steps to reproduce

Click the tag filter button to open the tag filter menu
While the tag filer menu is open, click the tag filter button a second time

Expected behavior

The tag filter menu should close

Actual behavior

The tag filter menu closes briefly but reopens immediately. The only way to close is to click elsewhere in the application

Additional context

The bug is not present on Linux

### App version 0.1.1 ### Platform Windows ### Steps to reproduce 1. Click the `tag filter` button to open the tag filter menu 2. While the `tag filer` menu is open, click the `tag filter` button a second time ### Expected behavior The tag filter menu should close ### Actual behavior The tag filter menu closes briefly but reopens immediately. The only way to close is to click elsewhere in the application ### Additional context The bug is not present on Linux

ValleyGeek added the

Kind/Bug

Priority

Medium

labels

2026-06-09 04:58:03 +00:00

Grok commented

2026-06-09 06:02:40 +00:00

Member

Fixed on branch 0.1.2/issue-25-tag-filter-close (VERSION 0.1.3).

Root cause: On Windows, Qt.Popup auto-dismisses when the Tag Filter button is clicked (outside-click handling) before the button's clicked handler runs. The handler then saw isVisible() == False and reopened the popup.

Fix: When the popup is dismissed with the cursor over the anchor button, set _tag_filter_ignore_next_click so the subsequent button click closes cleanly instead of reopening. Linux behavior is unchanged.

Verification: Regression test test_tag_filter_anchor_click_stays_closed_after_popup_auto_dismiss simulates the Windows event ordering; smoke-ui-exit.sh passes.

Fixed on branch `0.1.2/issue-25-tag-filter-close` (VERSION 0.1.3). **Root cause:** On Windows, `Qt.Popup` auto-dismisses when the Tag Filter button is clicked (outside-click handling) *before* the button's `clicked` handler runs. The handler then saw `isVisible() == False` and reopened the popup. **Fix:** When the popup is dismissed with the cursor over the anchor button, set `_tag_filter_ignore_next_click` so the subsequent button click closes cleanly instead of reopening. Linux behavior is unchanged. **Verification:** Regression test `test_tag_filter_anchor_click_stays_closed_after_popup_auto_dismiss` simulates the Windows event ordering; `smoke-ui-exit.sh` passes.

Grok commented

2026-06-09 06:02:40 +00:00

Member

Fixed on branch 0.1.2/issue-25-tag-filter-close (VERSION 0.1.3).

Root cause: On Windows, Qt.Popup auto-dismisses when the Tag Filter button is clicked before the button clicked handler runs, so the popup briefly closed and immediately reopened.

Fix: When the popup is dismissed with the cursor over the anchor button, ignore the follow-up open request on the next button click.

Verification: Regression test test_tag_filter_anchor_click_stays_closed_after_popup_auto_dismiss; smoke-ui-exit.sh passes.

Fixed on branch `0.1.2/issue-25-tag-filter-close` (VERSION 0.1.3). **Root cause:** On Windows, `Qt.Popup` auto-dismisses when the Tag Filter button is clicked before the button `clicked` handler runs, so the popup briefly closed and immediately reopened. **Fix:** When the popup is dismissed with the cursor over the anchor button, ignore the follow-up open request on the next button click. **Verification:** Regression test `test_tag_filter_anchor_click_stays_closed_after_popup_auto_dismiss`; `smoke-ui-exit.sh` passes.