Skip to content
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

Blanking screensaver with 90 min timeout #123

Closed
wants to merge 7 commits into from

Conversation

@burnsauce
Copy link
Contributor

@burnsauce burnsauce commented Oct 23, 2017

What does this PR do?

Blanks the Teletype display after 90 minutes of keystroke activity. Requires a keystroke to wake the display. That keystroke is dropped.

Provide links to any related discussion on lines.

https://llllllll.co/t/teletype-screen-saver/9769

How should this be manually tested?

Set the timeout (see screensaver_mode.h) to 10 seconds, then cycle through modes, use the teletype as normal. Every 10 seconds after your last keystroke, the display should blank. Press any key (except for mods SHIFT, ALT, and CTRL) to wake the display. It should reveal the last screen you were on, although it will be as if you had just navigated to the screen (input and message lines will be blank, etc).

Any background context you want to provide?

Static patterns on the display will cause bright cells to deteriorate more quickly than black cells. It's somewhat healable with OLED, but would need a noisy animation to play for a while to do so.

There is another strategy to code this: by shortcutting handler_ScreenRefresh. It would be slightly more efficient, but it won't allow the screensaver interface to be extensible. I erred on the side of flexibility.

If the related Github issues aren't referenced in your commits, please link to them here.

I have,

  • updated CHANGELOG.md
  • updated the documentation
Except in pattern / tracker mode, which would require a refactor.
@burnsauce
Copy link
Contributor Author

@burnsauce burnsauce commented Oct 23, 2017

Don't merge this until #122 is merged. I need to update the code to reflect the new UI protocol. If it is merged, I will push a new commit to this PR.

@burnsauce
Copy link
Contributor Author

@burnsauce burnsauce commented Oct 25, 2017

Additionally, I will be adding button press and knob movement as conditions to wake the screensaver.

burnsauce added 6 commits Oct 23, 2017
Syntax: IN.SCALE -100 100

All future calls to IN will scale linearly to between -100 and 100.
Drops the keystroke that caused the wake-up.
@burnsauce burnsauce force-pushed the burnsauce:screensaver branch from f6376ea to 3805513 Oct 28, 2017
@burnsauce burnsauce closed this Oct 28, 2017
@burnsauce burnsauce deleted the burnsauce:screensaver branch Oct 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant