Skip to content

[scheduler] Render multi-days event the same way as all-day events in all views #20242

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 23 commits into
base: master
Choose a base branch
from
Open

[scheduler] Render multi-days event the same way as all-day events in all views #20242

wants to merge 23 commits into from
+311 −324

Conversation

@flaviendelangle
Copy link
Member

@flaviendelangle flaviendelangle commented Nov 7, 2025 •
edited
Loading

Closes #19685

You can play with the car-rental dataset to see the effect of the PR 👍

@flaviendelangle flaviendelangle self-assigned this Nov 7, 2025
@flaviendelangle flaviendelangle added type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. scope: scheduler Changes related to the scheduler. labels Nov 7, 2025
@mui-bot
Copy link

mui-bot commented Nov 7, 2025 •
edited
Loading

Deploy preview: https://deploy-preview-20242--material-ui-x.netlify.app/

Bundle size report

Bundle Parsed size Gzip size
@mui/x-data-grid 0B(0.00%) 0B(0.00%)
@mui/x-data-grid-pro 0B(0.00%) 0B(0.00%)
@mui/x-data-grid-premium 0B(0.00%) 0B(0.00%)
@mui/x-charts 0B(0.00%) 0B(0.00%)
@mui/x-charts-pro 0B(0.00%) 0B(0.00%)
@mui/x-charts-premium 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers-pro 0B(0.00%) 0B(0.00%)
@mui/x-tree-view 0B(0.00%) 0B(0.00%)
@mui/x-tree-view-pro 0B(0.00%) 0B(0.00%)

Details of bundle changes

Generated by 🚫 dangerJS against 5fe9c3e

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Nov 7, 2025
@github-actions
Copy link

github-actions bot commented Nov 7, 2025

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Nov 7, 2025
@flaviendelangle flaviendelangle marked this pull request as ready for review November 10, 2025 07:08
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Nov 10, 2025
@github-actions
Copy link

github-actions bot commented Nov 10, 2025

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added PR: out-of-date The pull request has merge conflicts and can't be merged. and removed PR: out-of-date The pull request has merge conflicts and can't be merged. labels Nov 10, 2025
@github-actions
Copy link

github-actions bot commented Nov 10, 2025

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@rita-codes
Copy link
Member

rita-codes commented Nov 11, 2025

I was playing with the preview and I found an error related to the readonly events that is not happening in prod:

Screen.Recording.2025-11-11.at.10.07.35.mov

@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Nov 12, 2025
@flaviendelangle
Copy link
Member Author

flaviendelangle commented Nov 12, 2025 •
edited
Loading

@rita-codes it's not related to this PR but to #20145

image

The any caused TS not to complain 😬
I'll fix it in this PR though 👍

@github-actions
Copy link

github-actions bot commented Nov 12, 2025

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Nov 12, 2025
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Nov 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rita-codes rita-codes Awaiting requested review from rita-codes

@noraleonte noraleonte Awaiting requested review from noraleonte

At least 1 approving review is required to merge this pull request.

Assignees

@flaviendelangle flaviendelangle

Labels

scope: scheduler Changes related to the scheduler. type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[scheduler] Render multi-days event the same way as all-day events in all views

3 participants

@flaviendelangle @mui-bot @rita-codes