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

feat: audio applet over-amplification support #700

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

justintime4tea
Copy link

@justintime4tea justintime4tea commented Oct 29, 2024

The COSMIC system settings provide a choice between 0-150% for the volume of both output and input. The COSMIC audio applet does not.

This proposed change aligns the audio applet UI/UX with the system settings, allowing the user to "over-amplify" volume directly from the audio applet.

Updated COSMIC Audio Applet (this PR)

screenshot-2024-10-29-16-40-24

Existing COSMIC System Settings

screenshot-2024-10-29-04-41-27

This change relates to the following issues:

Considerations

Bug/Bad experience?

Without this proposed change, or one of similar impact/change, a user could set volume to 150% in COSMIC settings and afterward, when adjusting volume via the audio applet, the volume will snap-back/cap to 100 and restrict volume to between 0-100%.

Forward thinking

I understand there is a broader topic of how exactly to support "over-amplification", however, at minimum, the COSMIC audio applet should at least align with the COSMIC system settings so that whether audio is set via settings, or audio applet, the user has the same "choice" (0-150%).

@justintime4tea justintime4tea changed the title feat(484): audio applet over-amplification support (like cosmic system) feat: audio applet over-amplification support (like cosmic system) Oct 29, 2024
@justintime4tea justintime4tea changed the title feat: audio applet over-amplification support (like cosmic system) feat: audio applet over-amplification support Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant