-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
feat: explore UI #4262
feat: explore UI #4262
Conversation
feat(explore): add a top tokens hook with mock data
feat(explore): add useTokenPrice hook and dumby data
* feat: add time selector dropdown UI * update time selector style
* chore(deps): bump token-lists (#3929) * feat: empty to deploy 628417f (#3962) feat: empty to deploy * feat: fix metamask mobile browser connection (#3964) * fix metamask * forceActivate * remove forceActivate * unused change * feat(risk): cache risk check with ttl (#3965) Co-authored-by: matteenm <105068213+matteenm@users.noreply.github.com> Co-authored-by: Vignesh Mohankumar <vignesh@vigneshmohankumar.com>
* add token table UI * update token table with intial data pipeline * feat: Load token table with initial dummy data TODO: get token information (token name and symbol) * add token table UI and token row components * update table with token logo * update table with correct arrow * update table border * runs prettier (#3971) prettier * add header to tokenRow, format dollar util, add responsiveness * update table styling * update table styling and components setup * add back side padding * create header cell component * update table styling * fix padding * update css styling * Alphabetize styles Co-authored-by: Jordan Frankfurt <jordanwfrankfurt@gmail.com>
update width mobile breakpoint
hide header on token table when mobile view
* stack token name and symbol * style: clean up CSS
* add token table loading state * make token row components reusable * change typing and CSS styling * remove key props
* fix conflicts * style: CSS cleaning 2 * clean divs * add media breakpoint constants
* add favorites button frontend component * fix height and width CSS
fix small arrow sizing detail
* filter favorite tokens * fix atom * make showFavorite an atom * implement atom and clean CSS * change naming schema
* search bar CSS * style css fix * change from atom to useState
* fix favoriting bug * fix code styling * minor change
* search responsiveness * hide placeholder * css fix * shared file
* token detail draft * initial route path and some info * reduce PR * fixes * token null fix
* token detail draft * initial route path and some info * reduce PR * token header * remove flex * font sizes * fix CSS
* add timeframe options * map times * list times
* link routing * fix focus * Update index.tsx
* skeleton * padding change * fix link styling * add resource component
* remove swap button & responsiveness * center sparkline * remove margin
fix hover
* fix hover * initial network balance * fix network * checking 0 balance * add unsupported chain check * add network selector * multiple netwrk logic * change polygon logo * fix * naming
* for demo * link protocol info * colors in shared file
* animate chart mwaha * get rid of comments * add timeout * add fake widget * style * move loading into own file
* moved token price and delta into chart, expanded hoverability for crosshair
Need to fix a build issue (Kaylee is owning) and then this should be mergable. Zach will take a final look over it tomorrow for any deps issues or things I missed, then we can merge! |
* add flyout * fix hover
merge main into explore
… page (#4194) * initial * initial footer * network balances * alphabetize * add smallest media breakpoint * Update colors.ts * rm console log * add loading and error state * fix multiple vs single * updates * updates * fix * Update settings.json * import fix
function ToolTip() { | ||
return ( | ||
<ToolTipWrapper> | ||
<TooltipTriangle /> |
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.
nit: This is overkill, and it should instead be done using a pseudo-element in CSS in the wrapper.
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.
good point. @kayleegeorge can you update
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.
what specifically do you mean by this @zzmp ?
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.
this can be done in pure css without bringing in an svg dependency
resolved "https://registry.yarnpkg.com/@types/lodash/-/lodash-4.14.177.tgz#f70c0d19c30fab101cad46b52be60363c43c4578" | ||
integrity sha512-0fDwydE2clKe9MNfvXHBHF9WEahRuj+msTuQqOmAApNORFvhMYZKNGGJdCzuhheVjMps/ti0Ak/iJPACMaevvw== | ||
"@types/lodash@^4.14.168", "@types/lodash@^4.14.172": | ||
version "4.14.182" |
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.
This doesn't actually need to be updated (but it doesn't hurt).
@@ -13891,6 +14127,13 @@ number-is-nan@^1.0.0: | |||
resolved "https://registry.yarnpkg.com/number-is-nan/-/number-is-nan-1.0.1.tgz#097b602b53422a522c1afb8790318336941a011d" | |||
integrity sha1-CXtgK1NCKlIsGvuHkDGDNpQaAR0= | |||
|
|||
numbro@^2.3.6: |
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.
It's not great that this brings in yet another big number library. There's no easy alternative here?
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.
we can switch to something else, i don't mind
merging explore into main!