[PR #6384] [CLOSED] chore(deps): bump fyne.io/systray to v1.12.2 #29457

Closed
opened 2026-08-05 08:08:06 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/6384
Author: @scottharris
Created: 6/9/2026
Status: Closed

Base: mainHead: bump-fyne-systray-v1.12.2


📝 Commits (1)

  • f48bb51 chore(deps): bump fyne.io/systray to v1.12.2

📊 Changes

2 files changed (+3 additions, -1 deletions)

View changed files

📝 go.mod (+1 -1)
📝 go.sum (+2 -0)

📄 Description

Describe your changes

Bumps fyne.io/systray from v1.12.1-0.20260116214250-81f8e1a496f9 to v1.12.2.

Fixes blank submenus on Linux desktops using libdbusmenu-gtk3 (Cinnamon, Xfce, GNOME).

Background

libdbusmenu-gtk3 requests GetGroupProperties for grandchildren before parents on the first call after detecting a new tray service. GTK silently drops children inserted into parent containers that don't yet exist, resulting in permanently blank submenus.

v1.12.2 includes fyne-io/systray#118, which fixes this by returning only depth=1 on the first GetLayout call (parents only), then triggering a second cycle 150ms later so grandchildren are introduced after their parent containers exist.

Test plan

  • Build and run netbird-ui on Linux Mint Cinnamon and confirm submenus populate correctly on first load

Fixes #4186

Stack

Checklist

  • Is it a bug fix
  • Is a typo/documentation fix
  • Is a feature enhancement
  • It is a refactor
  • Created tests that fail without the change (if possible)
  • This change does not modify the public API, gRPC protocols, functionality behavior, CLI / service flags, or introduce a new feature — OR I have discussed it with the NetBird team beforehand (link the issue / Slack thread in the description). See CONTRIBUTING.md.

By submitting this pull request, you confirm that you have read and agree to the terms of the Contributor License Agreement.

Documentation

Select exactly one:

  • I added/updated documentation for this change
  • Documentation is not needed for this change (explain why)

Dependency version bump fixing a UI bug; no user-facing behavior or docs change.

Docs PR URL (required if "docs added" is checked)

Paste the PR link from https://github.com/netbirdio/docs here:

https://github.com/netbirdio/docs/pull/__


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/netbirdio/netbird/pull/6384 **Author:** [@scottharris](https://github.com/scottharris) **Created:** 6/9/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `bump-fyne-systray-v1.12.2` --- ### 📝 Commits (1) - [`f48bb51`](https://github.com/netbirdio/netbird/commit/f48bb511e7344a7998b78b6baabac7b662759c28) chore(deps): bump fyne.io/systray to v1.12.2 ### 📊 Changes **2 files changed** (+3 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `go.mod` (+1 -1) 📝 `go.sum` (+2 -0) </details> ### 📄 Description ## Describe your changes Bumps `fyne.io/systray` from `v1.12.1-0.20260116214250-81f8e1a496f9` to `v1.12.2`. Fixes blank submenus on Linux desktops using libdbusmenu-gtk3 (Cinnamon, Xfce, GNOME). ### Background libdbusmenu-gtk3 requests `GetGroupProperties` for grandchildren before parents on the first call after detecting a new tray service. GTK silently drops children inserted into parent containers that don't yet exist, resulting in permanently blank submenus. `v1.12.2` includes fyne-io/systray#118, which fixes this by returning only depth=1 on the first `GetLayout` call (parents only), then triggering a second cycle 150ms later so grandchildren are introduced after their parent containers exist. ### Test plan - [x] Build and run `netbird-ui` on Linux Mint Cinnamon and confirm submenus populate correctly on first load ## Issue ticket number and link Fixes #4186 ## Stack <!-- branch-stack --> ### Checklist - [x] Is it a bug fix - [ ] Is a typo/documentation fix - [ ] Is a feature enhancement - [ ] It is a refactor - [ ] Created tests that fail without the change (if possible) - [x] This change does **not** modify the public API, gRPC protocols, functionality behavior, CLI / service flags, or introduce a new feature — **OR** I have discussed it with the NetBird team beforehand (link the issue / Slack thread in the description). See [CONTRIBUTING.md](https://github.com/netbirdio/netbird/blob/main/CONTRIBUTING.md#discuss-changes-with-the-netbird-team-first). > By submitting this pull request, you confirm that you have read and agree to the terms of the [Contributor License Agreement](https://github.com/netbirdio/netbird/blob/main/CONTRIBUTOR_LICENSE_AGREEMENT.md). ## Documentation Select exactly one: - [ ] I added/updated documentation for this change - [x] Documentation is **not needed** for this change (explain why) Dependency version bump fixing a UI bug; no user-facing behavior or docs change. ### Docs PR URL (required if "docs added" is checked) Paste the PR link from https://github.com/netbirdio/docs here: https://github.com/netbirdio/docs/pull/__ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
saavagebueno added the pull-request label 2026-08-05 08:08:06 -04:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: DYNR/netbird#29457