| Summary: | Kpat plays sound when you push "New game" button even if "Play sounds" option is unchecked | ||
|---|---|---|---|
| Product: | [Applications] kpat | Reporter: | twaikyont |
| Component: | general | Assignee: | Stephan Kulow <coolo> |
| Status: | CONFIRMED --- | ||
| Severity: | normal | CC: | aacid, danje1, kde-games-bugs-null |
| Priority: | NOR | ||
| Version First Reported In: | 20.08 | ||
| Target Milestone: | --- | ||
| Platform: | Ubuntu | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
| Attachments: |
attachment-1831-0.html
attachment-10664-0.html |
||
|
Description
twaikyont
2021-05-01 16:46:13 UTC
I can't reproduce it. For me it doesn't play a sound when pressing "New Game" at all, the "Play sounds" toggle enabled or disabled. Or do you mean the notification sound, when pressing "New Game" while a game is already running and has been started. KPatience: 21.8.1 Created attachment 142178 [details] attachment-1831-0.html You are right, I meant notification sound. вт, 5 окт. 2021 г. в 11:34, Daniel Fichtner <bugzilla_noreply@kde.org>: > https://bugs.kde.org/show_bug.cgi?id=436462 > > Daniel Fichtner <danje1@outlook.de> changed: > > What |Removed |Added > > ---------------------------------------------------------------------------- > CC| |danje1@outlook.de > > --- Comment #1 from Daniel Fichtner <danje1@outlook.de> --- > I can't reproduce it. > For me it doesn't play a sound when pressing "New Game" at all, the "Play > sounds" toggle enabled or disabled. > > Or do you mean the notification sound, when pressing "New Game" while a > game is > already running and has been started. > > KPatience: 21.8.1 > > -- > You are receiving this mail because: > You reported the bug. Okay. I'll have a look at it, but I think it's a system sound (notification prompt) and therefore not affected by the "Play sounds" setting. > I think it's a system sound (notification prompt) and therefore not affected by the "Play sounds" setting.
That's defenitely the case, the question is if the user has a reasonable expectation that "Play sounds" also disables those kind of sounds or not.
Maybe the easy solution is just rename "Play Sounds" to something more obvious that is only about the game sounds?
Created attachment 142196 [details] attachment-10664-0.html I think it would be better to modify messagebox creating process. https://stackoverflow.com/a/16076209 . This way you can choose if you need sound or not. On Wed, Oct 6, 2021, 00:03 Albert Astals Cid <bugzilla_noreply@kde.org> wrote: > https://bugs.kde.org/show_bug.cgi?id=436462 > > Albert Astals Cid <aacid@kde.org> changed: > > What |Removed |Added > > ---------------------------------------------------------------------------- > CC| |aacid@kde.org > > --- Comment #4 from Albert Astals Cid <aacid@kde.org> --- > > I think it's a system sound (notification prompt) and therefore not > affected by the "Play sounds" setting. > > That's defenitely the case, the question is if the user has a reasonable > expectation that "Play sounds" also disables those kind of sounds or not. > > Maybe the easy solution is just rename "Play Sounds" to something more > obvious > that is only about the game sounds? > > -- > You are receiving this mail because: > You reported the bug. It's really easy to disable the Notification. It's just one added line in the KMessageBox that is opened. If you overwrite the Option = Notify with a nullptr the notification is no more :D It should also be possible to ask for the toggle status to create a case depending messagebox. So to remove the sound all together is really easy. I'll have closer look tomorrow if I can find a solution to toggle it as well. Btw. I'm new here. So please don't be mad at me if I do something wrong :) Merge request created: https://invent.kde.org/games/kpat/-/merge_requests/19 |