fixed issue #22736 - Cursor position not in right side of search keyword in mobile

[ghost]

Description (*)

For mobile view when search is focused I have set with only found solution https://www.google.com/search?q=jquery+set+cursor+at+end+of+input

Fixed Issues (if relevant)

1. Cursor position not in right side of search keyword in search box when click on search again (Mobile issue) #22736:
   Cursor position not in right side of search keyword in search box when click on search again (Mobile issue)

Manual testing scenarios (*)

1. Load website in mobile view
2. Search anything.
3. Click on search icon to focus search box
4. Check the cursor position

Contribution checklist (*)

• Pull request has a meaningful description of its purpose
• All commits are accompanied by meaningful commit messages
• All new or changed code is covered with unit/integration tests (if applicable)
• All automated tests passed successfully (all builds on Travis CI are green)

[m2-assistant]

Hi @sanjaychouhan-webkul. Thank you for your contribution
Here is some useful tips how you can test your changes using Magento test environment.
Add the comment under your pull request to deploy test or vanilla Magento instance:

• @magento-engcom-team give me test instance - deploy test instance based on PR changes
• @magento-engcom-team give me 2.3-develop instance - deploy vanilla Magento instance

For more details, please, review the Magento Contributor Assistant documentation

[VladimirZaets]

Hi @sanjaychouhan-webkul, this can get even simpler: this.element.focus().val(this.element.val())

[VladimirZaets]

Hi @sanjaychouhan-webkul, this can get even simpler: this.element.focus().val(this.element.val())

[ghost]

@VladimirZaets this does not work. also getting js error because we are triggering focus() event inside onFocus event handler.

[ghost]

@VladimirZaets also this.element.val(this.element.val()); doesn't work.

[magento-engcom-team]

Hi @VladimirZaets, thank you for the review.
ENGCOM-5126 has been created to process this Pull Request

[VladimirZaets]

Hi @sanjaychouhan-webkul.
Currently, we support only ES5 syntax, so can you please change let to var and according to our technical guidelines variables declaration should be on the top of the function.

[ghost]

@VladimirZaets I have applied the changes

[magento-engcom-team]

Hi @VladimirZaets, thank you for the review.
ENGCOM-5126 has been created to process this Pull Request

[soleksii]

✔️ QA Passed

Before:

After:

[m2-assistant]

Hi @sanjaychouhan-webkul, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.