Skip to content

fix: always use the internal back button#1690

Merged
susnux merged 2 commits intomasterfrom
fix/show-back-button
Sep 12, 2025
Merged

fix: always use the internal back button#1690
susnux merged 2 commits intomasterfrom
fix/show-back-button

Conversation

@susnux
Copy link
Copy Markdown
Contributor

@susnux susnux commented Sep 10, 2025

@susnux
Copy link
Copy Markdown
Contributor Author

susnux commented Sep 10, 2025

/backport to stable32

@szaimen
Copy link
Copy Markdown
Collaborator

szaimen commented Sep 11, 2025

/backport to stable32

Copy link
Copy Markdown
Collaborator

@szaimen szaimen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works!
But didnt review the code

The downside of this approach is that the arrow keys do not work anymore. However I'd say this is fine. WDYT @nextcloud/designers?

@marcoambrosini
Copy link
Copy Markdown
Member

marcoambrosini commented Sep 12, 2025

I'm fine with that, thanks @susnux :)

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux force-pushed the fix/show-back-button branch from 1acfdd5 to b210ba5 Compare September 12, 2025 10:45
@susnux
Copy link
Copy Markdown
Contributor Author

susnux commented Sep 12, 2025

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@susnux susnux merged commit 9ee2eaf into master Sep 12, 2025
41 checks passed
@susnux susnux deleted the fix/show-back-button branch September 12, 2025 10:49
@skjnldsv skjnldsv modified the milestones: Nextcloud 33, Nextcloud 32 Oct 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

the back button is missing from modal in desktop mode

5 participants