Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Desktop: Refactor audio and brightness controls #876

Merged
merged 1 commit into from
Nov 1, 2024

Conversation

kajusnau
Copy link
Contributor

Description of changes

  1. Taskbar Audio Control and LabWC (F1, F2, F3) Hotkeys Now Use Pamixer

    • The audio-ctrl script has been removed, as it was simply a wrapper for existing pamixer commands.
  2. Improved Responsiveness of Taskbar Audio and Brightness Controls:

    • Volume and brightness sliders in the quick settings widget are now more responsive.
    • Volume and brightness popups (displayed near the bottom of the screen) now appear more consistently and react to changes more quickly.
  3. Removed audio-ctrl Script

Checklist for things done

  • Summary of the proposed changes in the PR description
  • More detailed description in the commit message(s)
  • Commits are squashed into relevant entities - avoid a lot of minimal dev time commits in the PR
  • Contribution guidelines followed
  • Ghaf documentation updated with the commit - https://tiiuae.github.io/ghaf/
  • PR linked to architecture documentation and requirement(s) (ticket id)
  • Test procedure described (or includes tests). Select one or more:
    • Tested on Lenovo X1 x86_64
    • Tested on Jetson Orin NX or AGX aarch64
    • Tested on Polarfire riscv64
  • Author has run make-checks and it passes
  • All automatic Github Action checks pass - see actions
  • Author has added reviewers and removed PR draft status
  • Change requires full re-installation
  • Change can be updated with nixos-rebuild ... switch

Instructions for Testing

  • List all targets that this applies to:
  • Is this a new feature
    • List the test steps to verify:
      • Adjust system volume and display brightness using taskbar sliders and keyboard hotkeys
  • If it is an improvement how does it impact existing functionality?

@kajusnau kajusnau temporarily deployed to internal-build-workflow October 30, 2024 12:47 — with GitHub Actions Inactive
@kajusnau kajusnau requested a review from josa41 October 30, 2024 12:47
@kajusnau kajusnau added the Needs Testing CI Team to pre-verify label Oct 30, 2024
Copy link
Contributor

@josa41 josa41 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe a bit minor issues..

modules/desktop/graphics/ewwbar.nix Show resolved Hide resolved
modules/desktop/graphics/ewwbar.nix Show resolved Hide resolved
modules/desktop/graphics/ewwbar.nix Outdated Show resolved Hide resolved
modules/desktop/graphics/labwc.config.nix Outdated Show resolved Hide resolved
modules/desktop/graphics/ewwbar.nix Show resolved Hide resolved
@johannarautanen
Copy link

Tested on Lenovo-X1 (flashed to the SSD)

Working:

  • Volume and brightness sliders in the quick settings are more responsive when with older version

  • Volume and brightness popups now appear more consistently and react to changes more quickly.

  • all other ghaf apps can be launched

  • ci-test automation cases ok

Isssue:

  • there are times when the volume or brightness popup isn't visible, but the functionality(volume and brightness) working ok.
    Not easy to reproduced, but when happened when tapping the the volume and brightness buttons and then ie volume popup is not anymore visible, but the F1, F2 and F3 changed the volume.
    And when you tap the brightness F5 and F6, the brightness popup comes visible and after that the volume popup is visible again when changing the volume.

@kajusnau
Copy link
Contributor Author

Tested on Lenovo-X1 (flashed to the SSD)

Working:

  • Volume and brightness sliders in the quick settings are more responsive when with older version
  • Volume and brightness popups now appear more consistently and react to changes more quickly.
  • all other ghaf apps can be launched
  • ci-test automation cases ok

Isssue:

  • there are times when the volume or brightness popup isn't visible, but the functionality(volume and brightness) working ok.
    Not easy to reproduced, but when happened when tapping the the volume and brightness buttons and then ie volume popup is not anymore visible, but the F1, F2 and F3 changed the volume.
    And when you tap the brightness F5 and F6, the brightness popup comes visible and after that the volume popup is visible again when changing the volume.

The issue described seems to be the expected behavior of how the popups appear in the stack.
Since both popups are defined to appear in the overlay, whichever one opens last appears on top of the other.
For example, adjusting volume and brightness in that order makes the volume popup appear then get immediately covered by the brightness popup in the same layer. Both popups are on a 2s timer, which can be interrupted.

@johannarautanen johannarautanen added Tested on Lenovo X1 Carbon This PR has been tested on Lenovo X1 Carbon and removed Needs Testing CI Team to pre-verify labels Oct 31, 2024
- Adjust labwc and ewwbar to use pamixer directly
- Simplify ewwbar scripts
- Remove unused audio-ctrl script

Signed-off-by: Kajus Naujokaitis <kajus.naujokaitis@unikie.com>
@kajusnau
Copy link
Contributor Author

Maybe a bit minor issues..

Updated PR, most of the mentioned issues have been addressed.

@johannarautanen johannarautanen added the Needs Testing CI Team to pre-verify label Oct 31, 2024
@johannarautanen
Copy link

Tested on Lenovo-X1 (flashed to the SSD) (Commit: 1950991)

Working:

  • Volume and brightness sliders working
  • Popups appeared when changing volume and brightness
    - all other ghaf apps can be launched
    - ci-test automation cases ok

Notes:

  • when the "Terminal"-menu is opened and stayed the screen, the volume and brightness cannot be changed via F1, F2, F3, F5 and F6. After you really open the Terminal-app or close the Terminal-menu, then hotkeys worked again.
  • When you open then the youtube-page via chromium or microsoft teams meeting, then there appeared 3 times volume popups and when you close the page or stop the meeting, again volume popups appeared couple of times.
  • Once when "playing" with the "Terminal"-menu and volume popups, then popup stayed on the screen, doesn't go way time being or after lock the screen or closing other apps from the screen. After logout the volume popup disappeared.
    Will create separate issues of these notes.

@johannarautanen johannarautanen removed the Needs Testing CI Team to pre-verify label Nov 1, 2024
@brianmcgillion brianmcgillion merged commit aea05b5 into tiiuae:main Nov 1, 2024
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tested on Lenovo X1 Carbon This PR has been tested on Lenovo X1 Carbon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants