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
Enhancement for arrows on image carousel #208
Comments
looping is a parameter which sometimes makes sense, sometimes not. so I would not override this with a hardocded bool honestly |
@Jupi007 would it be possible to a border-radius to the image? This way it would fit the other parts of the UI |
@madsrh Yes it would be possible 👍 :) |
@madsrh I forgotten about the keyboard thing, so reopening :) About the border-radius, imo this should be done by the app using the carousel (aka. Software). |
Yes this can be done inside the app |
@Jupi007 Are the (missing) keyboard bindings for fullscreen done here or in the Software app? 🤔 Also, can the left / right arrows be present in fullscreen mode? |
It's not missing in software you just need to click on the image :D |
* Add PopupMenuTheme and improve example
The difference between the active and deactivated arrows in the image carousel are too subtle IMHO. I think a better solution would be if the images could just loop ♾️ or alternatively, to hide the left arrow on the first image (and right arrow for the last ofc).
For accessibility the keyboard left/right arrows should work when I click the arrows, the dots and when the image is enlarged.
The text was updated successfully, but these errors were encountered: