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

Preview details in detail view of photo not fully displayed on mobile devices #9318

Closed
koebel opened this issue Jun 29, 2023 · 2 comments · Fixed by #9351
Closed

Preview details in detail view of photo not fully displayed on mobile devices #9318

koebel opened this issue Jun 29, 2023 · 2 comments · Fixed by #9351
Assignees
Labels
Topic:UX-Design Type:Bug Something isn't working

Comments

@koebel
Copy link
Contributor

koebel commented Jun 29, 2023

Steps to reproduce

  1. Open any resource of a file type that can be displayed in preview app on a mobile device
  2. preview details at the bottom are cut off on the right side because the div is not fully centered and thus some actions are cut off

Expected behaviour

preview detail bar is centered and all actions are visible even on smaller devices

Actual behaviour

on mobile devices rotate icon is cut off and thus this action is not accessible to the user

Screenshot_Previewdetails_Detailview

@koebel koebel added Type:Bug Something isn't working Topic:UX-Design labels Jun 29, 2023
@grimmoc grimmoc self-assigned this Jul 4, 2023
@koebel
Copy link
Contributor Author

koebel commented Jul 5, 2023

@grimmoc I think the issue are the 30px margin of .oc-position-medium
If you subtract that amount from .oc-position-bottom-center on the left, it seems to be ok

Screen Shot 2023-07-05 at 16 16 32.png

@grimmoc
Copy link
Contributor

grimmoc commented Jul 5, 2023

@koebel yes, I know. I am working on it. it also caused the bar to be off-center in desktop view

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Topic:UX-Design Type:Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants