Skip to content
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

feat(time-preferences): add time grid for users to select study time … #6

Merged
merged 4 commits into from
Oct 16, 2024

Conversation

WelldoneM
Copy link
Contributor

@WelldoneM WelldoneM commented Oct 16, 2024

…preferences

Please include a summary of the changes and the related issue.

Type of change

  • New feature
Screenshot 2024-10-15 at 7 07 17 PM

Checklist

  • I have performed a self-review of my code
  • Single file within 200 lines of codes
  • Comments has added
  • My changes generate no new warnings

@github-actions github-actions bot added size/M Medium PR (50-199 lines changed) profile labels Oct 16, 2024
@WelldoneM WelldoneM assigned ZL-Asica and unassigned ZL-Asica Oct 16, 2024
Copy link

github-actions bot commented Oct 16, 2024

🚀 Preview for this PR is available at: https://studybuddy-8086e--pr-6-3aw8p2so.web.app

@ZL-Asica
Copy link
Member

Could you use F12 and select here to see what it looks like on mobile?

F12

1. time only contain hour.
2. Use abbrev for dates in week.
3. Add select for "No earlier than" and "No later than".
4. Add "Include Weekends" toggle.
@github-actions github-actions bot added size/L Large PR (200-799 lines changed) and removed size/M Medium PR (50-199 lines changed) labels Oct 16, 2024
Copy link

github-actions bot commented Oct 16, 2024

🚀 Preview for this PR is available at: https://studybuddy-8086e--pr-6-3aw8p2so.web.app

@ZL-Asica
Copy link
Member

mobile example

@WelldoneM
Copy link
Contributor Author

WelldoneM commented Oct 16, 2024 via email

@WelldoneM
Copy link
Contributor Author

WelldoneM commented Oct 16, 2024 via email

@ZL-Asica ZL-Asica enabled auto-merge (squash) October 16, 2024 02:07
@ZL-Asica
Copy link
Member

@WelldoneM You can open a new PR for that after this is approved. That will make things easier.

@ZL-Asica ZL-Asica removed the request for review from chipanyanwu October 16, 2024 02:09
@ZL-Asica ZL-Asica merged commit 7fa131c into main Oct 16, 2024
4 checks passed
@ZL-Asica ZL-Asica deleted the feat/time-preferences branch October 16, 2024 02:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
profile size/L Large PR (200-799 lines changed)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants