Skip to content
This repository has been archived by the owner on Jan 15, 2019. It is now read-only.

[In Progress] Add a new 600px breakpoint to help align front-end margins with the editor #439

Closed
wants to merge 8 commits into from

Conversation

kjellr
Copy link
Collaborator

@kjellr kjellr commented Oct 31, 2018

❗️ Note: This PR contains bits of #262 and needs some cleanup before merge. 😞 ❗️


Renames existing breakpoint widths to relative sizes, and adds a new "Small" (600px) breakpoint. An effort to match the front-end margins up with those in the Gutenberg editor (#236).

Breakpoint size name updates:

  • Mobile => Extra Small
  • NEW => Small
  • Tablet => Medium
  • Desktop => Large
  • Wide => Extra Large

The margins on many blocks/page elements are adjusted to support this new breakpoint.

Screenshots

artboard

A followup to #22.

This PR removes our secondary `style-editor-frame.css` file, and consolidates all of our editor styles into `style-editor.css`. Initially, the standard editor stylesheet was limited in scope to individual blocks, which meant we were unable to style the general alignment of the page, as well as items like the page title and default appender.

The scope of `style-editor.css` is being expanded in WordPress/gutenberg#10956 , so we're now able to style those elements in a single stylesheet.

Some minor style tweaks were necessary to make our original styles work in this context, but for the most part this is identical to #22.
Use of [data-align] selectors appears to be here to stay, so no need to point this out in the header.
Since it's no longer used for the editor frame styles.
So that it builds in case we need to edit it later on.
Renames existing breakpoint widths to relative sizes, and adds a new "Small" breakpoint.

Mobile = Extra Small
*NEW* = Small
Tablet = Medium
Desktop = Large
Wide = Extra Large

This also commits changes to lots of blocks and pages so they support these new variables, and so that they support the new Small breakpoint.

(apologies for the giant commit, I had to squash these together from another, broken branch)
@kjellr kjellr added the enhancement New feature or request label Oct 31, 2018
@kjellr
Copy link
Collaborator Author

kjellr commented Nov 8, 2018

Closing, as this PR is stale. This was addressed via another method in #502 and #515.

@kjellr kjellr closed this Nov 8, 2018
@kjellr kjellr deleted the add/new-breakpoint branch November 8, 2018 20:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant