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

make buttons visible in mobile version of presentation mode in slides #1112

Open
wants to merge 1 commit into
base: staging
Choose a base branch
from

Conversation

lemondevxyz
Copy link
Contributor

@lemondevxyz lemondevxyz commented Jul 6, 2023

Fixes #1105

Slides hides the buttons needed for navigation and exiting in presentation mode. This makes sense because when doing a presentation, a user would prefer to not have button icons all over their presentation content. When a user hovers on the presentation, the needed buttons become visible again.

This commit makes mobile touch act similarily to hovering over the presentation.

Note: I tested this on Firefox responsive mode and iOS Chrome.

Slides hides the buttons needed for navigation and exiting in presentation
mode. This makes sense because when doing a presentation, a user would
prefer to not have button icons all over their presentation content. When
a user hovers on the presentation, the needed buttons become visible
again.

This commit makes mobile touch act similarily to hovering over the
presentation.
@ghost ghost changed the base branch from main to staging July 7, 2023 07:10
@ghost ghost added Mobile support Small screens? Virtual keyboards? UI/UX Interfaces & user experience labels Jul 7, 2023
@ghost ghost requested a review from yflory July 7, 2023 07:10
@mathilde-cryptpad mathilde-cryptpad added the Markdown slides Related to the Markdown Sildes app label Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Markdown slides Related to the Markdown Sildes app Mobile support Small screens? Virtual keyboards? UI/UX Interfaces & user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Slide: can't exit presentation mode (PR open)
2 participants