-
Notifications
You must be signed in to change notification settings - Fork 65
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
Make background colour switcher optional #226
Comments
Unfortunately, that doesn't work with white content. |
This is how it looks in the Wikipedia viewer: https://commons.wikimedia.org/wiki/Category:White_logos?uselang=de#/media/File:Disqus_logo_%28white%29.svg And even then, I would say if anything a background switcher like that should be a shortcut or something else, but not a prominent button like it is now. Because it really is an edge-case. |
We've had this discussion before :). Some people have folders filled with such content and the slideshow becomes useless if they can't present it in a decent way. I thought about making the checkboard darker, but then things start looking strange, as it can interfere with darker images. I'll just make the feature optional, via the config file first. |
@deMattin that’s not the point. It is there, and the majority of people don’t need it. I’m sure we can think of a more elegant solution here. |
I understand, why you don't like the button and also would like to have a solution without it. |
There is one party trick I like, but which is not easy to implement because pictures may fill the screen. It's where you present to the user a "band" going from dark to light. The user clicks on the shade and it changes the background. But the elegant solution is here anyway #84 :P |
Maybe I have an idea - not a real solution but a way to get rid of the button: Clicking on the picture itself circles through background colors: Ok, it's not intuitive, but if you have read the readme or by doing it accidently you will know about such a "hidden" feature ... @oparoz Automatic feature is great but how do you know, that this automatic color stands for transparency? |
@deMattin I think it's not a bad idea to enable the change via a click and it's definitely something which can be tested for usability. And the automatic feature does some calculations and based on the result picks either the white or black background. It can be extended of course to pick an exact shade, but I don't think it's worth it. |
@oparoz: In my above example you see, that a black background won't show that the "MK" isn't transparent as it should be. You only see this by switching to a grey or white background! |
Yeah, but those are details. The solution can use a light or a dark checkboard, clearly indicating where the picture is transparent. |
This could also already be implemented with the current switching solution. |
Checkerboard doesn't show details ... ;) So, what do you think about a "click on picture color circle switch" to get rid of the button? |
My english ... I think, it has to be named "cycle switch" and not "circle switch" ... ;) |
I think there is one problem. We have to prevent it from being triggered when the user is panning. I think that involves launching a timer and then testing if the cursor has moved and it has to work with old browsers... But I don't see the difference between the automatic chooser and the current switch. You get the same result, except that with the current solution, there is a button and you need to click on it. |
@deMattin @oparoz this will not work as clicking on the image in a slideshow is the common pattern for switching to the next image – which we also do now. How about this:
|
Ok - I havn't thought about panning, you are right. So in my opinion:
|
@jancborchardt: we posted simultaneously: |
Haha .. I like good solutions - but sometimes it's a hard way to find this simple thing, that suits to nearly anybody. Edit: I hope, @oparoz or someone others doesn't find a showstopper for this solution ... ;) |
I did think about it earlier, but discarded it because you're always going to enable the alternative background with large pictures as the mouse pointer will be over it. And imagine that you're zooming in and the background changes all the sudden. Not a great UX. |
So, there is double click left, but users will really have to had read the doc to know about this. And auto mode. |
I think it's a nice project for a junior designer. Build all the solutions into the slideshow and get them tested to see what works most of the time. |
@oparoz: In only automatic mode you are not able to see, if the color is background (transparent) or a solid color of the picture (see my "MK" in above example - there neither black nor white background is visualizing the transparency appropriate). Only switching shows the error (lack of transparency in "MK") I'm able to do some static css or html and am able to understand or slightly modify php or js code. So sadly I'm too junior to do this. :/ |
Auto mode can tell if the picture is transparent and the level of luminance. |
The background color switcher for transparent images is really not necessary. We should do it like Wikipedia and just insert the white-grey pattern background: https://en.wikipedia.org/wiki/Wikipedia#/media/File:Wikipedia-logo-v2.svg
The text was updated successfully, but these errors were encountered: