codecov[bot]
commented
2 months ago Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 82.50%. Comparing base (
460caec) to head (5650431). Report is 13 commits behind head on release/4.9.x.develop.
Additional details and impacted files
```diff @@ Coverage Diff @@ ## release/4.9.x.develop #3274 +/- ## ====================================================== Coverage 82.50% 82.50% ====================================================== Files 157 157 Lines 7249 7249 Branches 1920 1920 ====================================================== Hits 5981 5981 Misses 996 996 Partials 272 272 ```:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Checklist
Issue Resolved / Feature Added
The last input type is not recognized properly when
requestLastInputTypeis called on some platforms since the function do nothing on unsupported platforms.Resolution
The issue occurs after #3218 that adds
requestLastInputType. At the moment we added this API, it is unclear that who is in charge of handling unsupported platforms among Enact, theme libraries, and apps. TherequestLastInputTypefunction's role is simply to call LS2Request so the usage of this API depends on apps' logic. We'd like to remove platform checking and make this function to call LS2Request always. It means that Enact does not involve checking if the platform is supported. So, theme libraries or apps should check the availability.Additional Considerations
On unsupported platforms, LS2Request API covers warnings so
requestLastInputTypedoes not need to cover the case.Links
WRR-1010 enactjs/sandstone#1682 Related: WRQ-6498 / #3218
Comments
Enact-DCO-1.0-Signed-off-by: Seungcheon Baek (sc.baek@lge.com)