search

netbymatt / ws4kp

WeatherStar 4000+
https://weatherstar.netbymatt.com
MIT License
273 stars 47 forks source link

Share link broken #47

Closed hzm170 closed 4 months ago

hzm170 commented 5 months ago

share link ( copy to clipboard ) is not working ; also how can one add the music to the player?

netbymatt commented 5 months ago

Can you provide more information on what's not working such as what web browser you're running on and what platform?

Previous disucssion regarding music: https://github.com/netbymatt/ws4kp/issues/43#issuecomment-2107612071

ray-frey commented 4 months ago

I think he meant the permalink button. Mine doesn't seem to copy the permalink to the clipboard

gmoran1016 commented 4 months ago

Also having issues with the permalink button, This error is displayed in the console when the copy permalink button is pressed. (Firefox 127.0.2) Screenshot 2024-07-07 175323

netbymatt commented 4 months ago

@gmoran1016 Thanks for the screenshot. This problem has had me stumped because MDN was reporting all the current browsers support the clipboard. But your screenshot shows a non-secure address (HTTP). Normally the secure requirement is relaxed for localhost (and 127.0.0.1) but I've learned a lot of users run this in docker or other local server methods and run in to similar problems, such as GPS location lookups.

Since this throws an undefined error I can detect it and just write the URL to the page to be copied manually.