search

marijnkampf / Random-Photo-Screensaver

Source of Random Photo Screensaver
http://www.abscreensavers.com/random-photo-screensaver/open-source
Other
44 stars 14 forks source link

Only 1 session of Configure Random Screensaver open at atime #130

Open Philip-Tory opened 3 years ago

Philip-Tory commented 3 years ago

I had selected a different Images folder in the Configure Random Screensaver screen, and thought I had Saved and Exited OK . . . but the selected images weren't showing. I opened Configure Random Screensaver again, made definite settings, Save and Close, but the selected images still weren't showing! It turned out that the first Configure Random Screensaver screen was still open, in the background. Can we fix it so that only one session can be open at a time? If a user selects it again, then make it go to the session that is already open. Thank you all.

marijnkampf commented 3 years ago

It does show a (small) message on the screen. I will look into making it more prominent as I think that making only one session possible at the time will create more issues than it solves.

Philip-Tory commented 3 years ago

Hello Marijn,

  1. Thank you for your prompt reply. How about a confirmation message box, RPS is already open in another session… Continue? Y/N .
  2. Enhancement request re. Videos… I don’t always want video sound to come on automatically, in case the noise disturbs someone else, or distracts me from a phone call. So I tick “Mute videos”. But sometimes I would like to listen to a video that appears on the screen, so I click the Loudspeaker icon … But! This triggers a transition to the next slide. Can you make it so that clicking the loudspeaker icon does not transition to the next slide, but only switches the sound to the loudspeakers? Best regards, Philip Tory

29 King Edwards Avenue, Gloucester GL1 5DD UK Tel: 01452 - 500 639

From: Marijn Kampf [mailto:notifications@github.com] Sent: 09 October 2020 13:30 To: marijnkampf/Random-Photo-Screensaver Cc: Philip-Tory; Author Subject: Re: [marijnkampf/Random-Photo-Screensaver] Only 1 session of Configure Random Screensaver open at a time (#130) It does show a message on the screen. I will look into making it more prominent as I think that making only one session possible at the time will create more issues than it solves. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/marijnkampf/Random-Photo-Screensaver/issues/130#issuecomment-706152676 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AIWCFQCXAZ5ZVQDIKTWO5UDSJ365PANCNFSM4SJ3RQUA . https://github.com/notifications/beacon/AIWCFQCDVFMV7ZERMPQZC7TSJ365PA5CNFSM4SJ3RQUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFILQRZA.gif

marijnkampf commented 3 years ago

I've got a mute shortcut key on my todo list at https://github.com/marijnkampf/Random-Photo-Screensaver/issues/106