-
Notifications
You must be signed in to change notification settings - Fork 6
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
Remove CSS grid classes completely #183
Labels
bug
Issues with bugs that need to be fixed
Comments
bruvellu
added a commit
that referenced
this issue
Jun 3, 2023
bruvellu
added a commit
that referenced
this issue
Jun 3, 2023
In addition to removing grid classes, I've also renamed the IDs for consistency and moved a rule from `style.css` to the `home.css`.
Repository owner
deleted a comment from
mohNourBoshi
Jun 3, 2023
bruvellu
added a commit
that referenced
this issue
Jun 3, 2023
After removing the span-8 classes I had to add a display:grid rule. I used the same as used in the #sitelinks for now. Also made some tweaks to the font sizes and removed text justification.
bruvellu
added a commit
that referenced
this issue
Jun 8, 2023
bruvellu
added a commit
that referenced
this issue
Jun 13, 2023
bruvellu
added a commit
that referenced
this issue
Jun 13, 2023
Tweaked tour page here 8247ab3 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We are deprecating the grid-based CSS framework (Blueprint) in favour of a responsive layout. All the associated classes should probably be removed to avoid issues and inconsistencies in the template rendering.
Grid-related classes:
span-1
tospan-24
,container
(?),column
,last
,append-1
toappend-24
,prepend-1
toprepend-24
,border
(?),colborder
,pull-1
topull-24
,push-1
topush-24
,prepend-top
,append-bottom
,box
(?).(?) marks more generic classes, perhaps don't remove
Templates that need revision:
The text was updated successfully, but these errors were encountered: