Ameerul /Bug #65052 Inconsistent scrolling behaviour

[ameerul-deriv]

Changes:

Please include a summary of the change and which issue is fixed below:

• Added styling changes for when the user scrolls on the P2P page on the mobile view.
• Fixed overflow as some tabs showed 2 scroll bars.

When you need to add unit test

• If this change disrupt current flow
• If this change is adding new flow

When you need to add integration test

• If components from external libraries are being used to define the flow, e.g. @deriv/components
• If it relies on a very specific set of props with no default behavior for the current component.

Test coverage checklist (for reviewer)

• Ensure utility / function has a test case
• Ensure all the tests are passing

Type of change

• [ x] Bug fix
• New feature
• Update feature
• Refactor code
• Translation to code
• Translation to crowdin
• Script configuration
• Improve performance
• Style only
• Dependency update
• Documentation update
• Release

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated
deriv-app	✅ Ready (Inspect)	Visit Preview	Aug 11, 2022 at 7:57AM (UTC)

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.4% Duplication

[github-actions]

A production App ID was automatically generated for this PR. (log)

• PR: Ameerul /Bug #65052 Inconsistent scrolling behaviour #5505
• URLs:
  • w/ App ID + Server: https://deriv-app-git-fork-ameerul-deriv-tabs-scrolling-issue.binary.sx?qa_server=frontend.binaryws.com&app_id=31227
  • Original: https://deriv-app-git-fork-ameerul-deriv-tabs-scrolling-issue.binary.sx
• App ID: 31227

Click here to copy & paste above information.

- **PR**: [https://github.com/binary-com/deriv-app/pull/5505](https://github.com/binary-com/deriv-app/pull/5505)
- **URLs**:
    - **w/ App ID + Server**: https://deriv-app-git-fork-ameerul-deriv-tabs-scrolling-issue.binary.sx?qa_server=frontend.binaryws.com&app_id=31227
    - **Original**: https://deriv-app-git-fork-ameerul-deriv-tabs-scrolling-issue.binary.sx
- **App ID**: `31227`

[codecov-commenter]

Codecov Report

Merging #5505 (b08009a) into master (7af898d) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master    #5505   +/-   ##
=======================================
  Coverage   21.28%   21.28%           
=======================================
  Files        1385     1385           
  Lines       31471    31471           
  Branches     5360     5360           
=======================================
  Hits         6700     6700           
  Misses      24393    24393           
  Partials      378      378           

Impacted Files	Coverage Δ
...ackages/cashier/src/containers/cashier/cashier.jsx	97.22% <ø> (ø)
...kages/p2p/src/components/my-profile/my-profile.jsx	0.00% <ø> (ø)
...thods/add-payment-method/select-payment-method.jsx	0.00% <0.00%> (ø)
...ds/payment-methods-empty/payment-methods-empty.jsx	0.00% <ø> (ø)
...hods/payment-methods-list/payment-methods-list.jsx	0.00% <ø> (ø)
...components/order-details/order-details-wrapper.jsx	87.50% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
1.5% Duplication

[adrienne-deriv]

LGTM

[adrienne-deriv]

LGTM

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
3.1% Duplication