#31-support-for-different-styles #196

Merged
c24elipe merged 10 commits from #31-support-for-different-styles into team_2_week_1 2026-04-14 13:47:09 +00:00
Collaborator

This is not a full solution to the issue. What this branch does is implement a standard to be followed for using styles, as well as dark and light mode. It also implements a hook used for changing styles and light/dark mode (issues #17 and #18) and a "skeleton" for the style select component that utilizes this hook.

Here are the things remaining to be done:

  • Full implementation of style select component (look at #223 before merging this pull request)
  • Adding a variable to the style dataset for the logo
  • Use the right fonts for the styles
  • Make the cards look "good" with the different styles. Right now the styles are implemented for cards, but I lack an eye for esthetics
  • Add the styles for the seasons

For the reviewer, read the Wiki page to understand the thought behind it.

Also, is the file useStyle.js in the right location?

This is not a full solution to the issue. What this branch does is implement a standard to be followed for using styles, as well as dark and light mode. It also implements a hook used for changing styles and light/dark mode (issues #17 and #18) and a "skeleton" for the style select component that utilizes this hook. Here are the things remaining to be done: - Full implementation of style select component (look at #223 before merging this pull request) - Adding a variable to the style dataset for the logo - Use the right fonts for the styles - Make the cards look "good" with the different styles. Right now the styles are implemented for cards, but I lack an eye for esthetics - Add the styles for the seasons For the reviewer, read the [Wiki page](https://git.webug.se/Andras/BoundlessFlowCampus2K/wiki/Implementing-different-Styles) to understand the thought behind it. Also, is the file useStyle.js in the right location?
Collaborator

Checks performed:
Verified colors compared to wiki palette.
Checked the code for wiki standards.
Pulled the branch.

Issues:
A semicolon missing on Card.module.css line 13 color: var(--card-title-color)
No comment in styleselect.js (Wiki standard)
CSS declarations in card.module.css not in alphabetical order (Wiki standard)

Checks performed: Verified colors compared to wiki palette. Checked the code for wiki standards. Pulled the branch. Issues: A semicolon missing on Card.module.css line 13 color: var(--card-title-color) No comment in styleselect.js (Wiki standard) CSS declarations in card.module.css not in alphabetical order (Wiki standard)
Collaborator

New updates are working perfectly and didn't cause any weird behaviors.

New updates are working perfectly and didn't cause any weird behaviors.
Collaborator

I have tested the code, and it compiles and works now.

I have tested the code, and it compiles and works now.
c24elipe merged commit 0a305eee5c into team_2_week_1 2026-04-14 13:47:09 +00:00
c24elipe deleted branch #31-support-for-different-styles 2026-04-14 13:47:09 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
4 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!196
No description provided.