Skip to content

do not focus input on mobile#8898

Merged
nickvergessen merged 1 commit intomasterfrom
fix/noid/mobile-focus
Mar 2, 2023
Merged

do not focus input on mobile#8898
nickvergessen merged 1 commit intomasterfrom
fix/noid/mobile-focus

Conversation

@szaimen
Copy link
Contributor

@szaimen szaimen commented Mar 1, 2023

Follow-up to #8058

Regression of #4333

πŸ–ΌοΈ Screenshots

🏚️ Before 🏑 After
B A

🚧 TODO

  • ...

🏁 Checklist

For my own testing
docker run -it \
--name nextcloud-easy-test \
-p 8443:443 \
-e TRUSTED_DOMAIN=192.168.24.128 \
--volume="nextcloud_easy_test_npm_cache_volume:/var/www/.npm" \
-e TALK_BRANCH=fix/noid/mobile-focus \
ghcr.io/szaimen/nextcloud-easy-test:latest

@szaimen szaimen marked this pull request as ready for review March 1, 2023 13:30
@szaimen
Copy link
Contributor Author

szaimen commented Mar 1, 2023

Tested and works. Ready for review πŸ‘

@szaimen szaimen force-pushed the fix/noid/mobile-focus branch from e40ed85 to 8a3fb5c Compare March 1, 2023 16:53
@szaimen
Copy link
Contributor Author

szaimen commented Mar 1, 2023

Rebased

@szaimen szaimen requested a review from ShGKme March 1, 2023 17:10
@nickvergessen nickvergessen merged commit 915ab9d into master Mar 2, 2023
@nickvergessen nickvergessen deleted the fix/noid/mobile-focus branch March 2, 2023 08:06
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.

2 participants