Skip to content

Conversation

@fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Jan 19, 2025

Before this commit, when opening a channel on mobile the composer gets automatically focused.
This is not a problem on desktop, but when on mobile it triggers the opening of the system keyboard, occupying half the screen and requiring a "back" command to close it.

This happens because the function that opens the chat window will put focus on the composer.
This commit fixes the issue by only focusing the composer when not on mobile.

task-4115728

Forward-Port-Of: #194274
Forward-Port-Of: #194122

Before this commit, when opening a channel on mobile the composer gets
automatically focused.
This is not a problem on desktop, but when on mobile it triggers the
opening of the system keyboard, occupying half the screen and
requiring a "back" command to close it.

This happens because the function that opens the chat window will put
focus on the composer.
This commit fixes the issue by only focusing the composer when not on
mobile.

task-4115728

X-original-commit: b5741f1
@robodoo
Copy link
Contributor

robodoo commented Jan 19, 2025

Pull request status dashboard

@robodoo robodoo added forwardport This PR was created by @fw-bot conflict There was an error while creating this forward-port PR labels Jan 19, 2025
@fw-bot
Copy link
Contributor Author

fw-bot commented Jan 19, 2025

@Atovange @alexkuhn cherrypicking of pull request #194122 failed.

stdout:

CONFLICT (modify/delete): addons/mail/static/src/core/web/thread_service_patch.js deleted in ea44301837494697fdcd3f4c43871b9511d4cd84 and modified in 5f441c63d0b9f8b089b308d352e2d152ac15da5e.  Version 5f441c63d0b9f8b089b308d352e2d152ac15da5e of addons/mail/static/src/core/web/thread_service_patch.js left in tree.

Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?).

In the former case, you may want to edit this PR message as well.

⚠️ after resolving this conflict, you will need to merge it via @robodoo.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@C3POdoo C3POdoo added the RD research & development, internal work label Jan 19, 2025
@alexkuhn
Copy link
Contributor

Already fixed by #176557

@alexkuhn alexkuhn closed this Jan 19, 2025
@alexkuhn alexkuhn deleted the saas-17.4-17.0-autofocus-composer-mobile-gito-HM8R-fw branch January 19, 2025 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conflict There was an error while creating this forward-port PR forwardport This PR was created by @fw-bot RD research & development, internal work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants