Fixing positional enter in text area.

[sakshi-arora1]

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• I have signed the Adobe Open Source CLA.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes and the overall coverage did not decrease.
• All unit tests pass on CircleCi.
• I ran all tests locally and they pass.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.34%. Comparing base (995a7b9) to head (b567a4f).
Report is 1 commits behind head on dev.

Additional details and impacted files

@@            Coverage Diff            @@
##                dev    #1610   +/-   ##
=========================================
  Coverage     82.34%   82.34%           
  Complexity     1003     1003           
=========================================
  Files           108      108           
  Lines          2611     2611           
  Branches        373      373           
=========================================
  Hits           2150     2150           
  Misses          272      272           
  Partials        189      189           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[rismehta]

Why was this merged without test case ?

[sakshi-arora1]

Why was this merged without test case ?

@rismehta we already have a test case for newline charachter https://github.com/adobe/aem-core-forms-components/blob/dev/ui.tests/test-module/specs/textinput/textinput.runtime.cy.js#L150

and it seems we can't go to a cursor location to type using cypress. I searched that, and all the solutions were to put the whole value including \n. The fix here is particularly to type \n in between.

[rismehta]

Why was this merged without test case ?

@rismehta we already have a test case for newline charachter https://github.com/adobe/aem-core-forms-components/blob/dev/ui.tests/test-module/specs/textinput/textinput.runtime.cy.js#L150

and it seems we can't go to a cursor location to type using cypress. I searched that, and all the solutions were to put the whole value including \n. The fix here is particularly to type \n in between.

If we already have a test case for this use-case, then why do we need this fix ?