-
Notifications
You must be signed in to change notification settings - Fork 32
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
[5369] Design and implement a Recycle Bin #3126
Conversation
@@ -97,6 +99,7 @@ export function ActionsBar(props: ActionsBarProps) { | |||
disabled={disabled} | |||
{...buttonProps} | |||
onClick={() => onOptionClicked(option.id)} | |||
startIcon={option.icon ? <SystemIcon icon={option.icon} /> : null} |
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's this for?
@@ -83,7 +83,7 @@ const states = { | |||
disabled: { stateMap: { disabled: true } } | |||
}; | |||
|
|||
const status = { | |||
export const status = { |
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 needs a much better name if it is to be exported
const id = systemLinkId === 'siteDashboardDialog' ? 'craftercms.components.Dashboard' : ( | ||
systemLinkId === 'siteToolsDialog' | ||
? 'craftercms.components.EmbeddedSiteTools' | ||
: systemLinkId === 'siteRecycleBin' | ||
? 'craftercms.components.RecycleBin' | ||
: 'craftercms.components.Search' | ||
); |
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.
Replace this with a lookup table
import { makeStyles } from 'tss-react/mui'; | ||
import { Theme } from '@mui/material/styles'; | ||
|
||
export const useStyles = makeStyles()((theme: Theme) => ({ |
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.
Use sx
|
||
import { defineMessages } from 'react-intl'; | ||
|
||
export const translations = defineMessages({ |
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.
I believe we can now define messages inline with the react-intl upgrade
}, | ||
itemsTable: { | ||
border: 'none !important', | ||
'& .MuiDataGrid-columnHeaders, & .MuiDataGrid-cell, & .MuiDataGrid-cell--withRenderer & .MuiDataGrid-footerContainer': |
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.
Use exported classes
objects
craftercms/craftercms#5369