Waybar icon indicating copy to clipboard operation
Waybar copied to clipboard

Privacy module does not hide itself when not in use [Bug]

Open emiliathemaid opened this issue 1 year ago • 3 comments

Privacy module does not hide itself when not in use when "audio-out" is omitted from configuration. Config of privacy module:

  "privacy": {
    "icon-spacing": 14,
    "icon-size": 14,
    "transition-duration": 250,
    "modules": [
      {
        "type": "screenshare",
        "tooltip": true,
        "tooltip-icon-size": 24
      },
      //{
      //  "type": "audio-out",
      //  "tooltip": true,
      //  "tooltip-icon-size": 24
      //},
      {
        "type": "audio-in",
        "tooltip": true,
        "tooltip-icon-size": 24
      }
    ]
  },

(Opened OBS) 20240524_18h16m24s_grim (no software is recording sound/screen or outputting audio) 20240524_18h13m41s_grim

If "audio-out" module is included in the config, privacy module disappears from waybar as expected

emiliathemaid avatar May 24 '24 16:05 emiliathemaid

forgot to mention, I use arch with hyprland, ask me if you need more info

emiliathemaid avatar May 24 '24 16:05 emiliathemaid

I can confirm this, I also use arch with hyprland. I don't think that should matter tho?

joshuajeschek avatar Jun 08 '24 17:06 joshuajeschek

I can confirm this, I also use arch with hyprland. I don't think that should matter tho?

Arch probably don’t matter, hyprland might, however, in general pointing out your environment is good practice when reporting a bug.

emiliathemaid avatar Jun 09 '24 03:06 emiliathemaid

Which version are you using? This was recently changed with 4fbd4f212a80009e92d179f9e96a50562891c2e8. I have not attempted a repro, but the code looks to me like it should work. ;)

RobertMueller2 avatar Jul 07 '24 17:07 RobertMueller2

Which version are you using? This was recently changed with 4fbd4f2. I have not attempted a repro, but the code looks to me like it should work. ;)

Issue persists. I am using version 0.10.3-1 that is packaged in pacman extra repo, perhaps pacman package is not up to date, but I would rather not bother with building from source to verify that the bug is fixed and wait for the next update in the package manager. I'll take a note to check this once again when there is a package update.

emiliathemaid avatar Jul 08 '24 09:07 emiliathemaid

Fair enough. Fwiw, the commit is newer than last waybar release (0.10.3). I have no knowledge about what specific distro packages have or do, though.

RobertMueller2 avatar Jul 08 '24 10:07 RobertMueller2

I just installed waybar throught the AUR, it does actually work now. I guess it will just take a bit until it arrives in the repo :)

joshuajeschek avatar Jul 18 '24 21:07 joshuajeschek

Upgraded the package, issue resolved.

emiliathemaid avatar Jul 23 '24 12:07 emiliathemaid