-
Notifications
You must be signed in to change notification settings - Fork 6
feat: simple tour #705
base: main
Are you sure you want to change the base?
feat: simple tour #705
Conversation
pyphilia
left a 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.
Thanks for this PR!
| const iconInTextStyle = { | ||
| width: 20, | ||
| height: 20, | ||
| marginBottom: '-4px', |
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.
Can this be a bit more dynamic?
| content: 'Welcome', | ||
| locale: { skip: 'Skip' }, |
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.
Translations? If it's a prototype it can be done later.
| 'With this button, you can close the side bar to free some screen space.', | ||
| target: `#${TOUR_NAVIGATION_SIDEBAR_CLOSE_BUTTON_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.
Everything in there should have translations?
| // position: 'absolute', | ||
| // bottom: 0, |
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.
Remove.
| import ImportZip from './ImportZip'; | ||
| import ItemTypeTabs from './ItemTypeTabs'; | ||
|
|
||
| // import TourContext, { TourContextData } from './TourContext'; |
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.
Remove.
| const NewItemModal = ({ open, handleClose }: Props): JSX.Element => { | ||
| const { t: translateBuilder } = useBuilderTranslation(); | ||
| const { t: translateCommon } = useCommonTranslation(); | ||
| // const { isTourOpen } = useContext(TourContext) as TourContextData; |
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.
Remove.
| onClose={handleClose} | ||
| maxWidth="md" | ||
| fullWidth | ||
| // disableEnforceFocus={isTourOpen} |
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.
Remove.
0791683 to
b513c2f
Compare
No description provided.