-
Notifications
You must be signed in to change notification settings - Fork 3
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
Information message if user clicks Try the Tgrade App (Desktop only)
using "Mobile" devices
#510
Conversation
Try the Tgrade App (Desktop only)
using "Mobile" devices
Try the Tgrade App (Desktop only)
using "Mobile" devicesTry the Tgrade App (Desktop only)
using "Mobile" devices
src/App/index.tsx
Outdated
@@ -21,57 +23,71 @@ const Engagement = lazy(() => import("./pages/Engagement")); | |||
const OcHome = lazy(() => import("./pages/OcHome")); | |||
const TMarketHome = lazy(() => import("App/pages/TMarket")); | |||
|
|||
export const WarningMessage = styled.h1` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! but please move this to components as a component and then import it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks, done bd9b710
please let me know if I have to use different name for folder like 'TextAlert' or ...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well technically it is a informationMessage now when you ask
@@ -0,0 +1,10 @@ | |||
import styled from "styled-components"; | |||
|
|||
/** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove this comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done, thx
3823aee
Try the Tgrade App (Desktop only)
using "Mobile" devicesTry the Tgrade App (Desktop only)
using "Mobile" devices
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
This PR about adding warning message for the moment when user trying to start using 'Tgrade App' on mobile devices
(fyi we are waiting for design based on this ticket https://github.com/confio/design/issues/52)
Test scenarios when user click 'Try the Tgrade App (Desktop only)' button:
screencast.2022-03-23.14-56-00.mp4