-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[docs] Cleanup the doc pages of the community pickers #7339
[docs] Cleanup the doc pages of the community pickers #7339
Conversation
These are the results for the performance tests:
|
@@ -6,7 +6,7 @@ import { Unstable_StaticNextDatePicker as StaticNextDatePicker } from '@mui/x-da | |||
export default function StaticDatePickerDemo() { | |||
return ( | |||
<LocalizationProvider dateAdapter={AdapterDayjs}> | |||
<StaticNextDatePicker displayStaticWrapperAs="desktop" /> |
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.
Now that we have dedicated pages for the view components, I think that the static demos should display the toolbar, otherwise the diff between the two components is hard to understand.
packages/x-date-pickers-pro/src/internal/hooks/useStaticRangePicker/useStaticRangePicker.tsx
Outdated
Show resolved
Hide resolved
27acaa4
to
0c0edde
Compare
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.
packages/x-date-pickers-pro/src/internal/hooks/useStaticRangePicker/useStaticRangePicker.tsx
Outdated
Show resolved
Hide resolved
packages/x-date-pickers-pro/src/internal/hooks/useStaticRangePicker/useStaticRangePicker.tsx
Outdated
Show resolved
Hide resolved
4f2aabf
to
9d10de8
Compare
9d10de8
to
f9b3d70
Compare
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.
To me overall the new approach shown on date picker page seems like the most cohesive and if it were my decision—I'd go forth with updating other pages.
But we could also possibly wait/ask for @joserodolfofreitas opinion on this refactor/restructure. 🤔
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
597d421
to
af4d293
Compare
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.
LGTM other than those few nitpicks with time picker page/demos and the still unanswered question with position: 'relative'
... 🙈
A lot of small changes to unify the doc of the Date Picker, Time Picker and Date Time Picker.
I'll keep the Date Range Picker for later, as it needs a Date Range Calendar dedicated page.
The Date Picker page still have 2 demos I'd like to move to other pages (the helper text and the custom input one).
But I think this PR is big enough as it is.