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

don't have translucent background on image post thumbnail click #625

Merged

Conversation

vijaykramesh
Copy link
Contributor

right now there's this backColorTranslucent applied when you click on the image link preview for a post:

this looks kind of funky as the preview is then centered over the original post and you get doubling of the image with the translucent on in the background, until you click and then it black out the background.

this proposed change would just always have the blacked out background and not do the translucent thing until you click:

@@ -73,7 +73,6 @@ fun ImageViewerDialog(url: String, onBackRequest: () -> Unit) {
}

val backColor = MaterialTheme.colorScheme.scrim
val backColorTranslucent = MaterialTheme.colorScheme.scrim.copy(alpha = 0.4f)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I actually do really like the background translucence, but I agree 40% in way too little... it should be like 10-20% . Could you try that out and see how it looks?

@vijaykramesh vijaykramesh force-pushed the no_translucent_image_post_preview branch from 9722324 to 3662bed Compare June 15, 2023 05:36
…his makes it easier to see the image without the background interfering so much
@vijaykramesh vijaykramesh force-pushed the no_translucent_image_post_preview branch from 3662bed to df357cd Compare June 15, 2023 05:39
@vijaykramesh
Copy link
Contributor Author

OK updated this, the alpha is for the black overlay not for the underlying background, so 0.8 seems to look the nicest (the underlying background is still visible but doesn't interfere with the image you have in the ImageViewerDialog

it seems like when I screenshot it on my phone it doesn't quite capture the effect but let me know if you think this is a reasonable change (at least with 0.8 it seems way less distracting to me than 0.4 is)

@vijaykramesh
Copy link
Contributor Author

also just super impressed with how fast people are fixing bugs here, in this same MR i went to fix the small card click for images and then upon rebasing saw #614 already fixed it!

Copy link
Member

@dessalines dessalines left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Ya things are moving really fast, I can barely keep up.

@dessalines dessalines merged commit f680226 into LemmyNet:main Jun 15, 2023
Chris-Kropp pushed a commit to Chris-Kropp/jerboa that referenced this pull request Jun 16, 2023
…his makes it easier to see the image without the background interfering so much (LemmyNet#625)

Co-authored-by: Dessalines <dessalines@users.noreply.github.com>
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.

2 participants