Skip to content

Conversation

@yumnazaheed
Copy link
Contributor

@yumnazaheed yumnazaheed commented Oct 1, 2025

Description 🧾

  • Moved the clear, undo, and redo buttons () to the right side of the timetable tabs, outside the scrollable tab area, so they remain fixed and always visible.
  • Made the timetable tabs horizontally scrollable when there are too many, ensuring they do not overflow past the "Friday" column and do not push the page wider.
  • Adjusted the layout so the "Select your courses" search bar is wider and the spacing between controls is more consistent, allowing three courses to fit in a single line even on smaller screens.
  • Improved overall alignment and responsiveness of the top controls and tab bar to match the requested UI.

Testing

Manually test in browser:

  • start up client + server as usual
  • original state- only one tab
image
  • timetable with the sidebar open
image
  • timetable with ten tabs open
image

Checklist

  • 📍 You have assigned yourself to this pull request.
  • 🔗 You have linked an issue to which this pull request closes.
  • 💭 Leave any relevant specific directions to reviewers.
  • 👀 No secrets in clear text in the pull request.
  • 🎟️ To categorise release notes, the pull request should be labelled with at least one of these labels:
    • feature: for application feature improvement or new features
    • bug: fixing something that previously wasn't working
    • dev: development-side related changes
    • docker: updates to the Docker code
    • setup: changes to the setup/infrastructure of the codebase
    • test: updates to internal testing

Issue: #1074

@yumnazaheed yumnazaheed self-assigned this Oct 1, 2025
@yumnazaheed yumnazaheed requested a review from a team as a code owner October 1, 2025 08:46
@yumnazaheed yumnazaheed added the 🍀 feature 🪴 New feature or request label Oct 1, 2025
Copy link
Contributor

@mark-trann mark-trann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm! just some nitpicks

import TermSelect from './TermSelect';
const ControlsBar = styled(Box)`
display: flex;
align-items: flex-end; // Align all controls to the bottom
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove comment 🪓

min-width: 140px;
display: flex;
align-items: flex-start;
margin-bottom: 0; // Remove any margin that could misalign
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here 🪓

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🍀 feature 🪴 New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants