-
Notifications
You must be signed in to change notification settings - Fork 19
Schedule Section #129
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
Schedule Section #129
Conversation
✅ Deploy Preview for inpycon2025 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
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.
The design of the schedule in this PR is different from the one approved. Is there a reason for doing so?
@palnabarun I've made the changes. Tried to make it as close to the design as possible, except for the horizontal duration ruler on the left for large devices. It might take some time to implement this. Can we pick this up as an improvement for later? |
2322a98
to
0a9ece4
Compare
Design diffs that we can still notice:
|
Yeah. This is something we can take care of as a follow-up. |
Color scheme issues:
|
5f5accb
to
800088a
Compare
@palnabarun @bhavin192 Made the reviewed changes. Do take a look. |
8816f3c
to
10078e5
Compare
This PR covers Schedule Section #114
Created this as an initial draft so that there's visibility. Still in progress.