ms-iot / samples

Windows 10 IoT Core Samples
MIT License
1.2k stars 1.33k forks source link

Add screensaver to DefaultApp #117

Closed dotMorten closed 8 years ago

dotMorten commented 9 years ago

When the default app runs for a long time, it has the risk of burning itself into the LCDs we connect to the devices. It would be good to have a screensaver for the app to prevent this - I already have a sample done (and will be posting on hackster.io soon), so I would be happy to contribute this to the app, but before doing the work want to know if this is something that you agree on would be a good idea?

There is one down side of course: If you don't have a mouse, keyboard or functional touchscreen, once the screen saver has started, it's not possible to disable it again. However it might be possible to detect if any of these devices are available, and only if so, enable the screensaver.

stephenhuang commented 8 years ago

Hi dotMorten, I'm part of the Microsoft IoT Team. This is a great idea and we'd love to see your sample. Please submit a pull request and we'll take care of the rest.

dotMorten commented 8 years ago

@stephenhuang Yes I can provide you with this, however, can you help guide me to solve the issue on the last paragraph?

stephenhuang commented 8 years ago

@dotMorten My suggestion would be to have the screensaver disabled by default and to add an option to enable/disable it under settings (the gear in the top right hand corner). In the coming months, we can take your work and improve it so it can be set remotely from a computer. This will solve your problem :)

dotMorten commented 8 years ago

Got it! What do you want on the screensaver? Do you have a PNG or JPEG I can use? Perhaps a Windows IoT Core logo?

dotMorten commented 8 years ago

@stephenhuang Alright! Check out the PR and let me know what you think.

stephenhuang commented 8 years ago

Will do, thanks for helping out. We can always change the jpeg on our end (We'll need to comply with branding guidelines so it's best if we change it anyway).

Will take a look shortly

stephenhuang commented 8 years ago

Looks good. We'll pull this shortly

IoTGirl commented 8 years ago

Confirmed - Closed