-
Notifications
You must be signed in to change notification settings - Fork 6
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
Implement new main page #462
Conversation
bfa35a7
to
4fe0a63
Compare
4fe0a63
to
5446ca1
Compare
entryShortNamePlural: 'transactions', | ||
entryLongNamePlural: 'forced transactions', | ||
path: '/forced-transactions', | ||
} | ||
|
||
export const OFFER_TABLE_PROPS = { | ||
title: 'Available forced trade offers', | ||
title: 'Available offers', |
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.
Naming offers "forced trade offers" was something specifically requested by SW long ago. I'd only make this change if they approved it this way.
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.
They've accepted it 👍
} | ||
|
||
export function PerpetualL2TransactionsTable( | ||
props: PerpetualL2TransactionsTableProps | ||
) { | ||
const columns: Column[] = [ | ||
{ header: 'Age' }, | ||
{ header: `ID` }, |
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.
Maybe we could have an eslint rule for no unnecessary backticks? Not sure if a rule like this exists
@@ -0,0 +1,26 @@ | |||
import React from 'react' | |||
|
|||
export function BreakpointIndicator() { |
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.
Not sure if this is the best place to put this component
@@ -59,6 +57,7 @@ | |||
"postcss": "^8.4.14", | |||
"prettier": "^2.8.4", | |||
"prettier-plugin-tailwindcss": "^0.2.2", | |||
"tailwindcss": "^3.0.24" | |||
"@tailwindcss/container-queries": "^0.1.1", |
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.
Really cool that this is so widely supported https://caniuse.com/css-container-queries. Didn't even know this existed!
Resolves L2B-1975