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

Let tweet cards be True Dark as well #129

Closed
mikael-uhl opened this issue Dec 9, 2023 · 12 comments
Closed

Let tweet cards be True Dark as well #129

mikael-uhl opened this issue Dec 9, 2023 · 12 comments
Labels
enhancement New feature or request

Comments

@mikael-uhl
Copy link

Describe the problem

When True Black is active, the cards still follows the current theme color. Since the cards take up so much space in the screen the true dark option is barely taking any effect on the feed.
E.g.: I have the "RedWine" theme active and True Dark enabled, when i open my feed each card background is tinted red, i appreciate the colored icons and buttons but not the card background.

Describe the solution

Please let the background for tweet cards be true dark when TD is enabled. Hopefully it's as simple as changing the background color. Maybe adding a specific theme/themes for TD is also a solution.

@mikael-uhl mikael-uhl added the enhancement New feature or request label Dec 9, 2023
@TheHCJ
Copy link

TheHCJ commented Dec 22, 2023

I have fixed this in the mentioned draft pull request above No apk is available for testing at the moment

@TheHCJ
Copy link

TheHCJ commented Apr 2, 2024

I just submitted an pull request to fix this issue: #276

@TheHCJ
Copy link

TheHCJ commented Apr 3, 2024

This issue has been fixed and should be closed

@j-fbriere
Copy link
Owner

j-fbriere commented Apr 3, 2024

This issue has been fixed and should be closed

It has been fixed in the last build, thank you for your help, but it has not been released yet.
So I would wait for the next release before closing it.

@j-fbriere
Copy link
Owner

The version v3.8.0 has implemented this feature.

@TheHCJ
Copy link

TheHCJ commented Apr 4, 2024

@j-fbriere We should keep this issue open as it does not seem to be True Dark when you have the system theme on

@j-fbriere
Copy link
Owner

j-fbriere commented Apr 4, 2024

On my Android when I have the System theme mode on is like having the Light theme mode on.
So for me the Cards color should not be true black when the System theme mode is on.
Is your System theme mode like the Dark theme mode on your Android?
Maybe I should check the current theme brightness instead of the theme mode property?

@k2helix
Copy link

k2helix commented Apr 4, 2024

For me it works when explicitly setting the Theme Mode to dark and in true dark mode. When the theme mode is set to system, the tweet cards are shown as always.

By the way, would it be too hard to make this optional? I think in some themes it may look better to have some contrast between the tweets and the background.

@TheHCJ
Copy link

TheHCJ commented May 4, 2024

Is this issue resolved @mikael-uhl?

@TheHCJ TheHCJ mentioned this issue May 4, 2024
@mikael-uhl
Copy link
Author

mikael-uhl commented May 4, 2024

@TheHCJ

Is this issue resolved @mikael-uhl?

Tested on v3.8.2. "Theme Mode" is Dark and "True Black?" is checked.
The cards are still colored and not True Black, in comparison the background behind the cards is TB but it has been since before I've created this issue. However I did notice a diffence in colors, I think the cards are darker than previous versions but they are still tinted enough to light up AMOLED screens.
Since they are darker, I'm not sure how to proceed, was the intention just to darken the cards? Should I close this issue?
This kinda feature is a little hard for me to find in apps even today and it's why I appreciate the effort on making it work, it would be a big plus for me

App Info
"abis": [ "arm64-v8a", "armeabi-v7a", "armeabi" ]
"device": "miel", "flavor": "github", "locale": "en",
"os": "android",
"system": "33", "version": "300015940"

@j-fbriere
Copy link
Owner

The version v3.8.3 fixes the issue.

@mikael-uhl
Copy link
Author

The version v3.8.3 fixes the issue.

Can confirm! Cards are true dark, thank you

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

No branches or pull requests

4 participants