Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEQ-2166]/Tablet view - Reports, Dtrader, Tradershub #15011

Merged
merged 41 commits into from
Jun 26, 2024

Conversation

habib-deriv
Copy link
Contributor

@habib-deriv habib-deriv commented May 7, 2024

Changes:

Tablet view for Dtrader, Reports and TradersHub

DTrader and Reports breakpoints:
remove blocking message about adjusting size;
if width <600 - responsive;
width between 600 - 1279 - tablet + web-version + lock users for landscape mode
if width > 1280 desktop
Footer + Header - responsive;

Users will open Dtrader from Trader's Hub - we will not show them landscape blocker

farhan-nurzi-deriv and others added 3 commits April 23, 2024 16:16
* feat: tablet view

* revert: trade params test cases

* chore: remove unused imports

* chore: added test case for usedevice

* fix: height

* fix: accumulators stats

* fix: layout

* fix: test

* refactor: update test

* refactor: change mixins names

* fix: platform switcher

* refactor: remove shared isnewmobile and fix styles in contract page

* fix: blocker showing in different page besides dtrader, comments

* fix: comments

* revert: unneded changes

* fix: test failed

* fix: breaking designs because of rtl

* feat: empty

* fix: account switcher

* fix: account switcher and add blocker on portrait

* chore: update test case

* refactor: replace with hook from deriv-com/ui

* fix: failing test cases

* chore: update deriv-com/ui and remove redundant hook

* fix: useonclickoutside
* feat: update to deriv-com/ui

* feat: delete landscape blocker

* feat: add some mixins

* feat: complete with mixins

* feat: modify traders hub main page

* feat: modify main title

* feat: modify asset-summary

* feat: modify asset-summary styles

* feat: modify listing-container and currency-switcher styles

* feat: modify header

* feat: get back package-lock.json

* fix: add some our library to ignore patterns for jest

* chore: empty commit

* feat: get back landscape_blocker

* feat: better to update it in base.config intead of writing it in both files

* feat: update package.json

* feat: create hook to hide landscape_blocker

* feat: move landscape blocker to the react compnent

* feat: add routes to LandscapeBlocker

* feat: add mock for deriv-com/ui in globals.js

* fix: fix one test

* feat: remove jest deriv-com/ui from globals

* fix: trying to fix tests again

* fix: fix again and again

* fix: fix test case again

* feat: add isDesktop check for cfds-listing

* feat: override some styles

* feat: change breakpoints for footer

* fix: change package version

* feat: update mixins based on Farhan PR

* fix: fix lint issue

* feat: delete useEffect in App of trader package and style for landscape blocker

* feat: modify footer

* chore: made decision to delete unreachable code

* feat: get changes back for app component in trader package

* feat: remove routes.root from disable blocker array

* feat: change condition to else

* feat: implement review suggestions and fix mixin

* feat: rename sccs file

* feat: get changes back for trading.scss

* feat: implemet review suggestions
…odal and Compare CFDs Accounts modal (#14799)

* feat: modify regulators compare modal component

* feat: add correct mixins for cfd-compare-accounts modal

* fix: add resolver for react/jsx-runtime

* fix: mock deriv-com/ui useDevice hook

* feat: implement review suggestions
Copy link

vercel bot commented May 7, 2024

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

Name Status Preview Updated (UTC)
deriv-app ✅ Ready (Inspect) Visit Preview Jun 26, 2024 7:40am

* chore: reports pages tablet view

* style: update width as per figma design

* refactor: landscape cloker component

* fix: comments

* chore: use bitwise to avoid nan value

* fix: comments

* fix: import

* chore: update function
Copy link
Contributor

github-actions bot commented May 7, 2024

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

Click here to copy & paste above information.
- **PR**: [https://github.com/binary-com/deriv-app/pull/15011](https://github.com/binary-com/deriv-app/pull/15011)
- **URLs**:
    - **w/ App ID + Server**: https://deriv-app-git-tabletview.binary.sx?qa_server=red.derivws.com&app_id=24072
    - **Original**: https://deriv-app-git-tabletview.binary.sx
- **App ID**: `24072`

Copy link
Contributor

github-actions bot commented May 7, 2024

🚨 Lighthouse report for the changes in this PR:

Category Score
🔺 Performance 31
🟧 Accessibility 70
🟧 Best practices 83
🟧 SEO 77
🟧 PWA 78

Lighthouse ran with https://deriv-app-git-tabletview.binary.sx/

* chore: lock landscape mode on tablet view

* chore: only show tablet view on real tablet

* fix: notification

* chore: change implementation

* fix: change root

* refactor: comments

* fix: comments

* fix: height

* fix: rtl language

* refactor: remove extra property

* fix: positions modal height based on figma

* chore: add loader before rotating the screen

* fix: failing test

* chore: add timeout before hiding loader

* style: move styles

* chore: another ref for loader timeout cleanup

* fix: hide footer for dtrader
)

* fix: creation banner and adding mt5 account

* feat: add deriv-com/ui to account package json

* fix: add react/jsx-runtime to webpack resolvers because i have added deriv-com/ui to package json
* feat: tablet view suggestion

* feat: implement new design for tablet
Copy link
Contributor

package old new diff pct change
account 231 KB 231 KB +486 B +0.2% 🟢
appstore 771 KB 775 KB +4 KB +0.6% 🟡
bot-web-ui 839 KB 839 KB +9 B +0.0% 🟢
cashier 682 KB 682 KB 0 B 0% 🟢
cashier-v2 3.0 MB 3.0 MB +3 KB +0.1% 🟢
cfd 494 KB 496 KB +2 KB +0.4% 🟢
core 6.6 MB 6.6 MB +2 KB +0.0% 🟢
p2p 985 KB 990 KB +5 KB +0.5% 🟡
p2p-v2 3.4 MB 3.4 MB +4 KB +0.1% 🟢
reports 94 KB 95 KB +394 B +0.4% 🟢
trader 318 KB 320 KB +2 KB +0.5% 🟡
wallets 3.7 MB 3.7 MB +3 KB +0.1% 🟢

Copy link
Contributor

package old new diff pct change
account 231 KB 231 KB +486 B +0.2% 🟢
appstore 771 KB 775 KB +4 KB +0.6% 🟡
bot-web-ui 839 KB 839 KB +9 B +0.0% 🟢
cashier 682 KB 682 KB 0 B 0% 🟢
cashier-v2 3.0 MB 3.0 MB +3 KB +0.1% 🟢
cfd 494 KB 496 KB +2 KB +0.4% 🟢
core 6.6 MB 6.6 MB +2 KB +0.0% 🟢
p2p 985 KB 990 KB +5 KB +0.5% 🟡
p2p-v2 3.4 MB 3.4 MB +4 KB +0.1% 🟢
reports 94 KB 95 KB +394 B +0.4% 🟢
trader 318 KB 320 KB +2 KB +0.5% 🟡
wallets 3.7 MB 3.7 MB +3 KB +0.1% 🟢

@coveralls
Copy link

Coverage Status

coverage: 45.545% (-0.006%) from 45.551%
when pulling a0f0229 on tablet_view
into 23354ea on master.

Copy link

sonarcloud bot commented Jun 26, 2024

Copy link
Contributor

package old new diff pct change
account 231 KB 231 KB +486 B +0.2% 🟢
appstore 775 KB 779 KB +4 KB +0.6% 🟡
bot-web-ui 839 KB 839 KB +9 B +0.0% 🟢
cashier 684 KB 684 KB 0 B 0% 🟢
cashier-v2 3.0 MB 3.0 MB +3 KB +0.1% 🟢
cfd 498 KB 500 KB +2 KB +0.4% 🟢
core 6.6 MB 6.6 MB +2 KB +0.0% 🟢
p2p 986 KB 996 KB +9 KB +1.0% 🟡
p2p-v2 3.4 MB 3.4 MB +4 KB +0.1% 🟢
reports 94 KB 95 KB +394 B +0.4% 🟢
trader 318 KB 320 KB +2 KB +0.5% 🟡
wallets 3.7 MB 3.7 MB +3 KB +0.1% 🟢

@coveralls
Copy link

Coverage Status

coverage: 45.522% (-0.004%) from 45.526%
when pulling e540069 on tablet_view
into 249894c on master.

@balakrishna-deriv balakrishna-deriv merged commit 9765c27 into master Jun 26, 2024
9 checks passed
@ali-hosseini-deriv ali-hosseini-deriv deleted the tablet_view branch August 16, 2024 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants