Closed Pastedko closed 8 months ago
I saw multiple interfaces used across the application. Please, make sure to provide them in the models folder. @Pastedko
@g-slavov In total, i just worked on nearly every page that exists. Found another minor issue with text centering in sports, minor adjustments to navigation, resolved your comment, fixed some bugs, did 90% of the contact us page. I think we can just merge it so we don't have to deal with conflicts later.
Great work! Contact Us Page looks in my opinion! 🚀
I will leave here a few comments that we need to address at some point, but I will merge the pull requests since there were some conflicts that I had to resolve.
[ ] The header of the Contact Us page could be bigger.
[ ] The Contact Us page is not very responsive.
[ ] The popup of the navigation looks a little bit off:
[ ] We could add some styling to the text boxes on the Contact Us page so that there is some padding to the user's input, the cursor is visible, and the font is different from the generic font of the browser.
[ ] Same comment for the dropdowns is also valid for the Theme button (adjust the min-width property @g-slavov ).
[ ] The width of the sponsors' widget could be increased on various devices since it looks a little off in my opinion: We might consider creating a component for the sponsors' widget(in the components folder) reused across the application (best-case scenario if time allows).
[ ] Initial loading on a smaller device also looks slightly strange - perhaps this has something to do with the KeenSlider component, since on a reload everything looks good:
Changed all colors to css variables Made sponsors responsive Updated landing page Added minor adjustments to other pages Implement theme-changing feature Fix bugs