Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor CSS to SCSS #56

Merged
merged 26 commits into from
Jun 23, 2021
Merged

Refactor CSS to SCSS #56

merged 26 commits into from
Jun 23, 2021

Conversation

Gum-Joe
Copy link
Owner

@Gum-Joe Gum-Joe commented Mar 19, 2021

Refactor CSS to SCSS so it is more manageable.
Fixes #44.

TODO:

Gum-Joe added 2 commits March 16, 2021 14:59
Transfer:
- Buttons
- Base styles
- Add some mixins
- Add header styles to SCSS
@Gum-Joe Gum-Joe added this to the Service Pack 1 (MS1.5) milestone Mar 19, 2021
@Gum-Joe Gum-Joe requested a review from ILikeTeaALot March 19, 2021 10:53
@Gum-Joe Gum-Joe marked this pull request as draft March 19, 2021 10:53
@Gum-Joe
Copy link
Owner Author

Gum-Joe commented Mar 19, 2021

(BTW the REPL.IT user is me)

@Gum-Joe
Copy link
Owner Author

Gum-Joe commented Mar 30, 2021

Got to line 490

Create tabs.scss
Mobile headers
@Gum-Joe
Copy link
Owner Author

Gum-Joe commented Mar 30, 2021

Got to line 553

@Gum-Joe Gum-Joe linked an issue May 5, 2021 that may be closed by this pull request
Gum-Joe added 9 commits June 18, 2021 18:13
Add new mixin for fully centered flex items & migrate SCSS to it
Thank you @Asametric for your suggested fix!
The fix of `position: absolute` on the tab row was applied for small screen sizes
(on desktop things became a bit too spaced out)
@Gum-Joe Gum-Joe marked this pull request as ready for review June 19, 2021 14:38
@Gum-Joe
Copy link
Owner Author

Gum-Joe commented Jun 19, 2021

Only thing outstanding is #60

Copy link
Collaborator

@ILikeTeaALot ILikeTeaALot left a comment

Choose a reason for hiding this comment

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

other than typos (which can either be fixed with a further commit, or in some future commit to another branch), all looks good 👍

@Gum-Joe Gum-Joe merged commit 505554f into main Jun 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

R-Tab overlaps 'next week' text on mobile CSS refactors
2 participants