Skip to content

Visual redesign & style refactoring - #219

Merged
pixup1 merged 23 commits into
devfrom
redesign
Aug 26, 2026
Merged

pixup1 merged 23 commits into
devfrom
redesign

Conversation

@pixup1

@pixup1 pixup1 commented Dec 7, 2025

Copy link
Copy Markdown
Member

Description

This PR refactors all the style code from Tailwind to pure CSS classes, and redesigns the whole frontend using the new graphic charter. I also fixed like 10 other design-related issues.

Checklist

  • I have tested the changes locally and they work as expected.
  • I have tested the responsiveness of the changes and they work as expected.
  • I have assigned the pull request to the appropriate reviewer(s).
  • I have added labels to the pull request, if necessary.

Refactored files (zero Tailwind classes) :

  • src/App.vue
  • src/assets/XLogo.vue
  • src/components/AdminPizza/PizzaModal.vue
  • src/components/Content.vue
  • src/components/Footer.vue
  • src/components/FormField.vue
  • src/components/Hero.vue
  • src/components/LoginForm.vue
  • src/components/Map.vue
  • src/components/Modal.vue
  • src/components/NameConfirmationModal.vue
  • src/components/Navigation.vue
  • src/components/Notification.vue
  • src/components/PartnerCard.vue
  • src/components/Partners.vue
  • src/components/PasswordInput.vue
  • src/components/PasswordResetForm.vue
  • src/components/PaymenStatusIcon.vue
  • src/components/PaymentStatusIcon.vue
  • src/components/PizzaCard.vue
  • src/components/Planning.vue
  • src/components/ProgressBar.vue
  • src/components/RegisterForm.vue
  • src/components/StringListInput.vue
  • src/components/Tournament/Admin/AdminBracket.vue (318 lines)
  • src/components/Tournament/Admin/AdminGroups.vue (340 lines)
  • src/components/Tournament/Admin/AdminGroupsMatchs.vue (231 lines)
  • src/components/Tournament/Admin/AdminSwiss.vue (383 lines)
  • src/components/Tournament/Admin/TournamentAdminStages.vue (950 lines)
  • src/components/Tournament/Admin/TournamentAdminTeams.vue (149 lines)
  • src/components/Tournament/Bracket.vue (215 lines)
  • src/components/Tournament/GroupDetail.vue (51 lines)
  • src/components/Tournament/GroupTable.vue (361 lines)
  • src/components/Tournament/MatchCard.vue (367 lines)
  • src/components/Tournament/PrivateTournamentCard.vue
  • src/components/Tournament/SeatingPlan.vue (325 lines)
  • src/components/Tournament/SwissRoundTable.vue (88 lines)
  • src/components/Tournament/TeamCard.vue
  • src/components/Tournament/TournamentBrackets.vue (236 lines)
  • src/components/Tournament/TournamentCard.vue
  • src/components/Tournament/TournamentGroups.vue (104 lines)
  • src/components/Tournament/TournamentInfo.vue (216 lines)
  • src/components/Tournament/TournamentPlanning.vue
  • src/components/Tournament/TournamentRules.vue
  • src/components/Tournament/TournamentSeating.vue
  • src/components/Tournament/TournamentStages.vue (161 lines)
  • src/components/Tournament/TournamentSwiss.vue (20 lines)
  • src/components/Tournament/TournamentTeams.vue (131 lines)
  • src/components/Tournament/TravelFormModal.vue
  • src/components/TournamentMeCard.vue
  • src/views/AdminPizza/AdminPizza.vue (781 lines)
  • src/views/AdminPizza/AdminPizzaExportList.vue (262 lines)
  • src/views/AdminPizza/AdminPizzaList.vue (212 lines)
  • src/views/AdminPizza/PizzaListEl.vue
  • src/views/AdminPizza/PizzaModal.vue
  • src/views/Archives.vue
  • src/views/Association.vue
  • src/views/Cart.vue
  • src/views/Eat.vue
  • src/views/EmailVerif.vue
  • src/views/Home.vue
  • src/views/Informations.vue
  • src/views/Login.vue
  • src/views/Logout.vue
  • src/views/Me.vue
  • src/views/NotFound.vue
  • src/views/Register.vue
  • src/views/ResetPassword.vue
  • src/views/ReturnPayment.vue
  • src/views/ScanQrCode.vue
  • src/views/Schedule.vue
  • src/views/TeamDetail.vue
  • src/views/Tournament/Tournament.vue
  • src/views/Tournament/TournamentDetail.vue
  • src/views/Tournament/TournamentRegister.vue
  • src/stores/content.store.ts

Final checks:

  • Removed all instances of layer="override" (used to redefine tailwind classes)
  • Checked that everything looks fine without Tailwind
  • Removed Tailwind dependency
  • Made sure all icon buttons are accessible (wrapped in <button type="button"> tag with a title attribute) and made all clickable elements buttons (check all c-btn-* instances)
  • [Kinda mostly] Animated modals closing

CMS changes

Here are the changes that will need to happen to the content on the production website after this is merged :

  • Replace all the colored text <span style="color:#48cad9">text</span> with bold **text**;
  • Adjust the content of the home page to have just the schedule in the left column and the rest of the text in the right column;
  • Remove all the random <h1> tags from the home page;
  • Unify and check title levels (amount of #)
  • Add the Adresse content, that will appear above the map;
  • Rename parti_public to partie_public;
  • Add the lan file (File, not Content) which is a photo that will be in the background of the "Association" page (in webp format if possible);

Related Issues

Fix #200
Fix #124
Fix #224
Fix #225
Fix #228
Fix #226
Fix #189
Fix #147
Fix #202
Fix #49
Fix #137
Fix #220

Screenshots

image image image image image image image image image image image image image image image NB: The "Input Text" was replaced, just couldn't be bothered to make a new screenshot image image image image image image image image image image image image image image

@pixup1 pixup1 changed the title Redesign Visual redesign & refactoring Dec 7, 2025
@pixup1 pixup1 changed the title Visual redesign & refactoring Visual redesign & style refactoring Dec 7, 2025
@pixup1
pixup1 requested a review from floflo0 December 10, 2025 16:15
@awing-ding

Copy link
Copy Markdown

Whats the difference between style in the style.css and style directly in the file ? Like can't we put it all in the same place ?

@pixup1

pixup1 commented Dec 10, 2025

Copy link
Copy Markdown
Member Author

Whats the difference between style in the style.css and style directly in the file ? Like can't we put it all in the same place ?

Style directly in the files is scoped, so it doesn't affect other files. It's more logical to do this unless the class is susceptible to be used in at least a few files. So for instance, for the video player, I just want the bottom corners to be rounded, but I'm never going to use that class anywhere else. It would clutter style.css if I added that in there, and I would have to come up with more unique names and follow a stricter naming convention. So I declare it as scoped in Hero.vue.

@pixup1

pixup1 commented Dec 10, 2025

Copy link
Copy Markdown
Member Author

But I do wonder if I shouldn't just use the style html field for those classes. I know it's advised to not use it but idk why.

@awing-ding

Copy link
Copy Markdown

But I do wonder if I shouldn't just use the style html field for those classes. I know it's advised to not use it but idk why.

AFAIK it isn't recommanded because you put style at a lot of different places and it gets hard to maintain

@pixup1

pixup1 commented Dec 10, 2025

Copy link
Copy Markdown
Member Author

Yeah but is it really better to make classes I only use once

@floflo0

floflo0 commented Dec 10, 2025

Copy link
Copy Markdown
Member

If a class is only use once, it's okay to put it in the file it will be used in

@pixup1
pixup1 force-pushed the redesign branch 2 times, most recently from 8a56464 to 7535dbe Compare February 7, 2026 19:53
@pixup1 pixup1 self-assigned this Feb 8, 2026
@pixup1 pixup1 mentioned this pull request Mar 2, 2026
3 tasks
@pixup1
pixup1 requested a review from cocanard June 17, 2026 23:13
@pixup1
pixup1 marked this pull request as ready for review June 17, 2026 23:20
Comment thread src/components/AdminPizza/PizzaModal.vue Outdated
Comment thread src/components/Modal.vue Outdated
Comment thread src/views/AdminPizza/AdminPizza.vue Outdated
Comment thread src/views/AdminPizza/AdminPizza.vue
Comment thread src/views/AdminPizza/AdminPizza.vue
Comment thread src/components/ProgressBar.vue Outdated
Comment thread src/components/Tournament/SeatingPlan.vue Outdated
Comment thread src/components/Tournament/TournamentInfo.vue Outdated
Comment thread src/views/Archives.vue Outdated
Comment thread src/views/Association.vue Outdated
@floflo0

floflo0 commented Jun 24, 2026

Copy link
Copy Markdown
Member

Also, there's several places where the text isn't really readable due to the contrasts :

  • The progress bar :
2026-06-24-212248_hyprshot
  • The alert link :
2026-06-24-212223_hyprshot

I don't know if we can do anything to improve those contrasts.

@pixup1

pixup1 commented Jun 25, 2026

Copy link
Copy Markdown
Member Author

Also, there's several places where the text isn't really readable due to the contrasts :

* The progress bar :
2026-06-24-212248_hyprshot
* The alert link :
2026-06-24-212223_hyprshot

I don't know if we can do anything to improve those contrasts.

I fixed the links but yeah for the progress bars I don't know. I looked into fixing this back when I created the ProgressBar component, but as far as I can tell there's no way to have alternate "blend modes" with just CSS (which would give us an inverted effect that would guarantee readability). I made #240 if someone wants to look at this in the future.

Comment thread src/components/Tournament/TournamentStages.vue
@cocanard
cocanard self-requested a review August 6, 2026 14:10
Comment thread src/components/Tournament/SeatingPlan.vue
Comment thread src/views/Tournament/TournamentRegister.vue Outdated
Comment thread src/views/Me.vue
Comment thread src/views/Me.vue
Comment thread src/components/Tournament/Bracket.vue Outdated
Comment thread src/components/Tournament/Bracket.vue Outdated
Comment thread src/components/Tournament/GroupTable.vue Outdated
Comment thread src/components/Tournament/SwissRoundTable.vue Outdated
Comment thread src/components/Tournament/TournamentCard.vue
Comment thread src/components/Navigation.vue Outdated
Comment thread src/views/Me.vue
@awing-ding awing-ding mentioned this pull request Aug 9, 2026
60 tasks
@pixup1
pixup1 merged commit 45a4e0e into dev Aug 26, 2026
3 checks passed
@pixup1
pixup1 deleted the redesign branch August 26, 2026 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

4 participants