[SPIKE] Update to simplified grid layout from GOV.UK Frontend #1154
+409
−47
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
See: alphagov/govuk-frontend#1090 Co-authored-by: Oliver Byford <[email protected]>
This is to be consistent with other functions which are namespaced under the govuk prefix. We keep grid-width as a deprecated alias which we can remove in the next major version. See: alphagov/govuk-frontend#1090 Co-authored-by: Oliver Byford <[email protected]>
Wrapping the row functionality up in a mixin offers little other than the ability to use a custom name, but we’re not sure there’s a need for users to do that. Simplify the code by deprecating the mixin and just using the code to create the concrete class directly. See: alphagov/govuk-frontend#1090 Co-authored-by: Oliver Byford <[email protected]>
The class names generated by the mixin are not flexible enough, as the width is just appended to the class name. In spiking how we might add an extended grid we found the generated class names a little yoda-ish, for example `govuk-grid-column--at-desktop-one-quarter` (bonus points if you actually read that in Yoda’s voice). Moving the class naming out of the mixin and doing it within the objects layer is also more consistent with the rest of Frontend, and means that users who do use the grid mixins directly don’t have to fit with the constraints of our class name structure. See: alphagov/govuk-frontend#1090 Co-authored-by: Oliver Byford <[email protected]>
See: alphagov/govuk-frontend#1090 Co-authored-by: Oliver Byford <[email protected]>
Inspired by “desktop specific grid classs” in: alphagov/govuk-frontend#1094
colinrotherham
force-pushed
the
grid-updates
branch
from
9263cdf to
416fd12
Compare
This was referenced Feb 21, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR is a spike into the simplified grid layout used by GOV.UK Frontend as part of:
I'll follow up with our Sass modules migration in #1137 to see how we can lift-and-shift various
govuk-*mixins and functions via@userather than maintain our own copyBased on work by @36degrees in the following PRs:
Checklist