Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| if (browser) { | ||
| const saved = localStorage.getItem(TIME_MARKS_KEY); | ||
| if (saved !== null) { | ||
| $searchSettings.style["Show Time Marks"] = saved === "true"; | ||
| } |
There was a problem hiding this comment.
Avoid hydrating calendar with mismatched time-mark state
The calendar now reads localStorage and immediately sets $searchSettings.style["Show Time Marks"] during module evaluation. When a user has previously enabled time marks, the client’s initial render includes the extra column while the SSR HTML was generated with the default false value, which causes a hydration mismatch/flash on first load. To keep SSR and client output in sync, load the saved preference in onMount (or delay rendering until after mount) before updating the store.
Useful? React with 👍 / 👎.
Improve the look of the advanced search modal. Make the button more obvious when the search input is focused. Put the time mark control directly on the calendar and save to localStorage. Change icon for export. Add option to sort by capacity and to show capacity on the CourseCards