-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
Minor image upload bug #41
Comments
Not sure of a good way to deal with this. Perhaps a way to toggle night mode, but how would the user know what they can't see? A checkerboard is also not ideal, since there's sometimes value to having transparency in images. Can you think of any, @kn0wmad ? There is one possible way to do this, however:
|
Cool, sounds good. Obviously not a pressing feature, but will investigate this. |
Honestly, I don't mind if this issue is solved in this way, it's a really elegant solution that no other browser or viewer I've seen other than image editing software that's simpler to use than GIMP or PS has solved. Like, seriously, most of them probably "solve" it by having off-white or off-black be the background color. For small, quality of life features like this, the approach I suggested poses few technical challenges, would stress the implementer's knowledge of Rust so that good knowledge is gained, and it could potentially someday impress an early adopter with an eye for this. @kn0wmad What I'm saying is, go to town, my man. I'm not gonna say no. |
Okay, what I was saying is "I will do this, after some in progress stuff gets done." |
I happened to be testing an upload with a white image that had a transparent background. At least in light mode, this obviously was invisible. Not sure a possible solution at the moment, and it's a very niche case, but I want to open this as a reminder to myself to sort it out at some point
The text was updated successfully, but these errors were encountered: