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

Amend T-Deck variant.h to improve readability #3926

Closed
wants to merge 2 commits into from

Conversation

joneyes
Copy link

@joneyes joneyes commented May 17, 2024

Changes to improve readability of T-Deck TFT:
Set T-Deck screen colour to white rather than green. Increase default brightness to 200

Changes to improve readability of T-Deck TFT:
Set T-Deck screen colour to white rather than green.
Increase default brightness to 200
@joneyes joneyes changed the title Amend variant.h to: Amend T-Deck variant.h to improve readability May 17, 2024
@rcarteraz
Copy link
Contributor

Changes to improve readability of T-Deck TFT: Set T-Deck screen colour to white rather than green. Increase default brightness to 200

I don't think there's a need to change the default brightness level considering you can now change the brightness level using the keyboard. I also don't think we should change the color scheme for one person's perceived readability level. If anything, an option to select what you are proposing in the display configs seems to be more appropriate

@joneyes
Copy link
Author

joneyes commented May 17, 2024

I had previously achieved the same result in a different way. When I posted images on a Facebook group, there was significant interest.
It's not just me, a lot of users find the default green Tdeck characters unreadable in sunlight.
This was the simplest fix I could do. If you can implement it a better way, please go ahead, but I suspect it will be a lot more complex?

@mverch67
Copy link
Collaborator

Increasing the default brightness will also increase the power consumption of the TFT display and thus reduce the runtime on batteries. As already suggested you can use the keyboard control to adapt the brightness, but I would not touch the default as not all users know how to change the brightness.

@joneyes
Copy link
Author

joneyes commented May 22, 2024

Ok, I'll take the adjustable brightness comment.

Thoughts on the colour change? Everyone that has seen it prefers white text as it is much more readable in bright sunlight.

I am happy to raise a retrospective enhancement request, to use it to gauge user opinions?

@rcarteraz
Copy link
Contributor

Ok, I'll take the adjustable brightness comment.

Thoughts on the colour change? Everyone that has seen it prefers white text as it is much more readable in bright sunlight.

I am happy to raise a retrospective enhancement request, to use it to gauge user opinions?

I don’t see the need to change it. There’s been little to no issue so far. If there’s a personal preference, then it would be something folks can manually compile for themselves.

@ChrDel
Copy link

ChrDel commented May 22, 2024

Ok, I'll take the adjustable brightness comment.
Thoughts on the colour change? Everyone that has seen it prefers white text as it is much more readable in bright sunlight.
I am happy to raise a retrospective enhancement request, to use it to gauge user opinions?

I don’t see the need to change it. There’s been little to no issue so far. If there’s a personal preference, then it would be something folks can manually compile for themselves.

I dont agree with that statement. I’m actually running custom builds now only to have a white text… For people using devices in sunny places there is no way to see anything on this screen even at full brightness… white does make it more readable… If I understood how to code this I would even go with an actual white background and black text which is the most readable outdoor in the sun…

@joneyes
Copy link
Author

joneyes commented May 22, 2024

I've already made a change to support it, and submitted the pull request.
I don't think it's going anywhere as there's no accompanying feature request.
It will need submitting again to back out the increased default brightness and I'll do that tomorrow when I am back home. Thank-you for supporting my opinion demonstrating that it's not a single person whim.

@rcarteraz
Copy link
Contributor

Ok, I'll take the adjustable brightness comment.
Thoughts on the colour change? Everyone that has seen it prefers white text as it is much more readable in bright sunlight.
I am happy to raise a retrospective enhancement request, to use it to gauge user opinions?

I don’t see the need to change it. There’s been little to no issue so far. If there’s a personal preference, then it would be something folks can manually compile for themselves.

I dont agree with that statement. I’m actually running custom builds now only to have a white text… For people using devices in sunny places there is no way to see anything on this screen even at full brightness… white does make it more readable… If I understood how to code this I would even go with an actual white background and black text which is the most readable outdoor in the sun…

And I don’t agree with yours. I use mine outdoors. Is it sometimes difficult to see? Sure, as is any screen. But I don’t think it’s as difficult as is being made out to be.

@thebentern thebentern closed this May 22, 2024
@ChrDel
Copy link

ChrDel commented May 22, 2024

Ok, I'll take the adjustable brightness comment.
Thoughts on the colour change? Everyone that has seen it prefers white text as it is much more readable in bright sunlight.
I am happy to raise a retrospective enhancement request, to use it to gauge user opinions?

I don’t see the need to change it. There’s been little to no issue so far. If there’s a personal preference, then it would be something folks can manually compile for themselves.

I dont agree with that statement. I’m actually running custom builds now only to have a white text… For people using devices in sunny places there is no way to see anything on this screen even at full brightness… white does make it more readable… If I understood how to code this I would even go with an actual white background and black text which is the most readable outdoor in the sun…

And I don’t agree with yours. I use mine outdoors. Is it sometimes difficult to see? Sure, as is any screen. But I don’t think it’s as difficult as is being made out to be.

I think it's sad to dismiss people's opinion based on your own and not to offer options...
This is an open source project that feels very close to the opinion of a few people and not open to what others things or may need... I'm really glad it works for you. Unfortunately for others it doesn't but this is being dismissed and ignored...
If you really don't have a problem either your eye sight is amazing or the sun where you live is not that bright.
Some people have disabilities and other live in scorching deserts burned by the sun others in rainy places. Why is this supposed to be only working for people that are young, healthy and live in low sun to no sun settings?
If you do not like white just because green is cooler and is the meshtastic color, why not just offer an option for users to chose what works best for them and be all inclusivee instead of discriminate amongst people disabilities, or place they live...

@rcarteraz
Copy link
Contributor

I dont agree with that statement. I’m actually running custom builds now only to have a white text… For people using devices in sunny places there is no way to see anything on this screen even at full brightness… white does make it more readable… If I understood how to code this I would even go with an actual white background and black text which is the most readable outdoor in the sun…

And I don’t agree with yours. I use mine outdoors. Is it sometimes difficult to see? Sure, as is any screen. But I don’t think it’s as difficult as is being made out to be.

I think it's sad to dismiss people's opinion based on your own and not to offer options... This is an open source project that feels very close to the opinion of a few people and not open to what others things or may need... I'm really glad it works for you. Unfortunately for others it doesn't but this is being dismissed and ignored... If you really don't have a problem either your eye sight is amazing or the sun where you live is not that bright. Some people have disabilities and other live in scorching deserts burned by the sun others in rainy places. Why is this supposed to be only working for people that are young, healthy and live in low sun to no sun settings? If you do not like white just because green is cooler and is the meshtastic color, why not just offer an option for users to chose what works best for them and be all inclusivee instead of discriminate amongst people disabilities, or place they live...

I’m not dismissing your opinion, simply stating I don’t agree with it. As you had just done with yours regarding mine as noted above.

I in fact live in a “scorching desert burned by the sun”. Also, I even tested this change and personally see no difference in the readability in the sun. Personally I think the green sticks out a little more to be honest and is easier to see than the white.

Also, I’ve suggested making it a config option. It’s not so easy to do so, so probably not a route that would be taken.

Personally, I don’t think we should be making changes because a select few want something different. If this was as big as an issue as it’s being made out to be in this discussion, we would have seen numerous complaints about this beforehand. We have not.

Lastly, I’m bit a decision maker. Merely offering my opinion as are you. Surely there shouldn’t be an issue with that just because I don’t agree with you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants