-
-
Notifications
You must be signed in to change notification settings - Fork 285
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
BUG: Sound Effect broken on v1.18 #2682
Comments
Please check if you have deleted Windows Media Player from the system. Since our framework uses WMP, it should not be deleted for the normal operation of flow. (In fact, windows warns you that deleting WMP may affect other programs.) A lot of people know this as a tip and delete it, but it's very often used in other programs as well as ours, so you shouldn't delete it. 😂 |
hmm, yes i have removed WMP, a year ago. and the sound works on v1.17 i tired to leave it but it couldnt handle not being the "default" and keep popping up like a red-head-step-child. sooo oh well i guess? |
I confirm, I deleted WMP years ago and the opening sound effect worked fine till v1.17. Is it possible to make the sound work without Windows dependencies or at least make it use the system's default player if possible (VLC in my case for example). |
im not the one to answer this... thou. i can say i reverted back to v1.17 and the sound is working as it was, without WMP. so i doubt it that causing the bug. also windows doesn't "remove or uninstall" any features from the "additional features" list, it only hides them, WMP is still in the system, its just not trying to trying to set itself as the default anymore. |
We use a framework called WPF, and there are two functions for playing sound. There are two functions: soundplay() and mediaplay(). We used soundplay() until version 1.17, and it didn't have volume control. There was a volume adjustment request pr and we changed the function to mediaplay()(It has a volume control function.) function. But mediaplay relies on WMP to work. this is what happens. As @realgooseman said, we can add something to play the sound without any dependencies. But I feel it's unreasonable to put this in just to play "a single sound". Uninstalling WMP is not recommended as it will cause problems for programs other than ours. (Just curious, Why is it so common to uninstall WMP? It is intentionally made inconvenient to delete, and tells you that it may cause problems when deleted.) We're going to take care of it by putting these warnings in for now. p.s) While writing this answer, I had an idea. we could just change function to work with soundplay when user don't have WMP (although user won't be able to use volume control)... |
Thanks for clarifying @onesounds, I like the idea to switch from mediaplay to soundplay if WMP is not detected on the system. One solution that makes everyone happy. Volume control is not a big deal, just leave it at 100% when switching to soundplay. |
Thanks for the clarification. @onesounds As there are many tools to meet an ends in windows. one can override a programs audio volume with something like the system volume mixer or a more robust option https://eartrumpet.app/ As for everyone uninstalling WMP is because.... It sucks. I've been "turning it off" via the system settings (not uninstalling it) sense the windows 7 days, over a decade, and very, very rarely has it ever caused an issue. This might be first in many years. |
That's sounds great. It's not a very loud sound effect anyway. Most of the time I almost can't hear it over the clicking of my cherry blue switches mechanical keyboard. 🤣 Generally I don't like to many sounds from my apps, I change all of windows default notifications sound to softer, more quiet sounds. But I do like the sound effect y'all added to flow so I keep it on. And I love this app! It's better than powertoys and because of it I haven't used windows start menu in months! Doubt I ever will again. Windows mutilated that back on win8 anyway. This is why we use apps other than what windows provides. There are better, quicker, more productive and fun options other than what Microsoft can or will produce. If they designed a functional or customizible start menu we would be seeking out better tools such as Flow. They're in the business of spyware for adds these days, not development. Ahh but I'm rambling off top. Lol In any case THANK YOU for you time here and on this app, I love it |
Added soundplay() to use if you don't have WMP, with "volume not adjustable" warning. It will play the same way it did in previous versions. https://ci.appveyor.com/project/JohnTheGr8/flow-launcher/builds/49820434/artifacts This is test build. Get the portable here, run it, and let me know if it plays sound normally without WMP. (And warning message in setting window) If it works, it will be included in the next release. |
Tried it, works great and the warning shows up in the settings. Thanks for the update.
|
yeah! thank you kindly! the sound is working. even with the volume turn down to 0. an new bug thou. 😆 first time ive noticed this. i just installed two new plugins using this v1.18 and it install and close itself, but then not restart itself as it used to do. |
@indigofairyx This is not a production build, so there may be issues with such handling. It is not recommended for mainstream use. Thank you for testing. We'll apply it after code review. +) I created this sound feature at first. Because the window is small and its position is not definite, it was difficult to recognize when flow was opened, so I added a sound and an animation to confirm the position. (Both are purely to make it easier for the user to recognize the state and position of the flow). I'm glad to see this feature is so popular. :) |
its great! thanks for your hard work. as i said i dont have many sounds going off in my system thou i like this one, very fitting. |
Checks
I have checked that this issue has not already been reported.
I am using the latest version of Flow Launcher.
Problem Description
theres no longer a sound effect when opening the search box. tired changing the new volume slide. no luck.
To Reproduce
Screenshots
No response
Flow Launcher Version
1.18.0
Windows Build Number
Microsoft Windows [Version 10.0.19045.4291]
Error Log
The text was updated successfully, but these errors were encountered: