807-fix-button-layout #839

Merged
a24julot merged 5 commits from 807-fix-button-layout into team_1_week_5 2026-05-08 10:28:44 +00:00
Collaborator

New layout for toolbar (desktop), toolbar is not rounded, buttons are placed on the right-side and header on the left.

Added a function so that when the toolbar transitions up/down the icon for the button changes with the same transition time, may need some tuning but the addition was out-of-scope of this issue so we may do a new one.

New layout for toolbar (desktop), toolbar is not rounded, buttons are placed on the right-side and header on the left. Added a function so that when the toolbar transitions up/down the icon for the button changes with the same transition time, may need some tuning but the addition was out-of-scope of this issue so we may do a new one.
removed hardcoded position of toolbar for mobile, centered toolbar to the toggle div. Changes transition value, new css.
removed padding/margin from toggle button, made toolbar item way to big.
a function so that the icon of toolbar is changed at the same speed as toolbar transition, one direction does not swap at exact time but it was somewhat out of scope of this issue.
Collaborator

Review on #807

Whats done

  • Checked so buttons have new positions
  • Toolbar not rounded

Tested on Windows using Chrome

  • App starts
    • Works as intended

Inspected following files

  • frontend/src/components/ToggleButton/ToggleButton.module.css
  • frontend/src/pages/dashboard/Dashboard.js
  • frontend/src/pages/dashboard/Dashboard.module.css
    • Indentation follows standard
    • Good code quality

What needs to be done

  • Nothing

Conclusion

  • Ready to be merged
# Review on https://git.webug.se/Andras/BoundlessFlowCampus2K/issues/807 ## Whats done - Checked so buttons have new positions - Toolbar not rounded ### Tested on Windows using Chrome - App starts - Works as intended ### Inspected following files - frontend/src/components/ToggleButton/ToggleButton.module.css - frontend/src/pages/dashboard/Dashboard.js - frontend/src/pages/dashboard/Dashboard.module.css - Indentation follows standard - Good code quality ## What needs to be done - Nothing ## Conclusion - Ready to be merged
a24julot merged commit a59e1d1bed into team_1_week_5 2026-05-08 10:28:44 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Andras/BoundlessFlowCampus2K!839
No description provided.