OrangeJedi / Aerial

Apple TV screen saver for Windows
MIT License
702 stars 25 forks source link

Random text position doesn't actually cause random text position. #138

Closed ThinkSalat closed 1 year ago

ThinkSalat commented 1 year ago

Describe the bug I set the "random" text to what I want, but it never even shows up on the screen. using an lg c1 60"

To Reproduce select random location for text and test it

Expected behavior It puts the text in random locations to prevent burn in on oled

Aerial Version latest

Additional context Would be really nice because I like the text display. Unfortunately it comes with potential burn in issues on an oled

OrangeJedi commented 1 year ago

I was not able to reproduce this bug. The random text option is working just fine for me.

What type of text are you trying to display? What is Random Change Speed on the text options menu set to? Both these could be effecting it.

ThinkSalat commented 1 year ago

I'm trying to display the date and the info for the current playing.

It's set to 30

Message ID: @.***>

OrangeJedi commented 1 year ago

Ok, I think I found the bug. Aerial don't trigger the random text to load until after the Random Change Speed time has passed. If you keep Aerial open for 30 seconds it should work as expected. Obviously this needs to be fixed and I'll slate it for the next update. Thanks for pointing this out!

ThinkSalat commented 1 year ago

Great! Good find

On Thu, Mar 23, 2023 at 11:09 AM OrangeJedi @.***> wrote:

Ok, I think I found the bug. Aerial don't trigger the random text to load until after the Random Change Speed time has passed. If you keep Aerial open for 30 seconds it should work as expected. Obviously this needs to be fixed and I'll slate it for the next update. Thanks for pointing this out!

— Reply to this email directly, view it on GitHub https://github.com/OrangeJedi/Aerial/issues/138#issuecomment-1481567493, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHNEPJIIZY76E6D6W7B5YCLW5R7VPANCNFSM6AAAAAAV5ZU6ZA . You are receiving this because you authored the thread.Message ID: @.***>

OrangeJedi commented 1 year ago

Fixed in v1.2.1