Skip to content

Fix block nav padding and margin#3610

Merged
swissspidy merged 2 commits intodevelopfrom
fix/3598-block-nav-indent
Oct 23, 2019
Merged

Fix block nav padding and margin#3610
swissspidy merged 2 commits intodevelopfrom
fix/3598-block-nav-indent

Conversation

@swissspidy
Copy link
Copy Markdown
Collaborator

@swissspidy swissspidy commented Oct 23, 2019

Summary

See #3598.

Checklist

  • My pull request is addressing an open issue (please create one otherwise).
  • My code is tested and passes existing tests.
  • My code follows the Engineering Guidelines (updates are often made to the guidelines, check it out periodically).

@swissspidy swissspidy added this to the v1.4 milestone Oct 23, 2019
@googlebot
Copy link
Copy Markdown

googlebot commented Oct 23, 2019

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@googlebot googlebot added the cla: no Has not signed the Google CLA label Oct 23, 2019
@swissspidy swissspidy requested a review from barklund October 23, 2019 13:17
@barklund
Copy link
Copy Markdown
Contributor

barklund commented Oct 23, 2019

@googlebot I consent.

@googlebot
Copy link
Copy Markdown

googlebot commented Oct 23, 2019

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

@googlebot googlebot added cla: yes Signed the Google CLA and removed cla: no Has not signed the Google CLA labels Oct 23, 2019
@swissspidy swissspidy merged commit f6b30e3 into develop Oct 23, 2019
@swissspidy swissspidy deleted the fix/3598-block-nav-indent branch October 23, 2019 15:26
westonruter added a commit that referenced this pull request Oct 25, 2019
@westonruter
Merge branch 'develop' of github.com:ampproject/amp-wp into add/query…
03a2395 
…-args-debugging

* 'develop' of github.com:ampproject/amp-wp: (163 commits)
  Remove entries array test after completed.
  Update dependency core-js to v3.3.4 (#3624)
  Quick UX fixes (#3611)
  Fix most pressing RTL issues (#3558)
  Add test case for broken parent relationship
  Adapt broken test
  Throw a _doing_it_wrong() when an expected parent is not found
  Change template hierarchy query arguments to public
  Reuse data-set-focus to reliably match twentytwenty keyboard focus management
  Add tabindex attribute to modals
  Guess the role of a modal based on its classes
  Disable Code Editor (#3608)
  Update dependency postcss to v7.0.20 (#3613)
  Update dependency autoprefixer to v9.7.0 (#3614)
  Update dependency browserslist to v4.7.2 (#3612)
  Add rel=preconnect link for AMP CDN
  Fix block nav padding and margin (#3610)
  Adjust Gutenberg / WordPress requirement (#3609)
  Don't allow multiple CTA / attachment block to be pasted. (#3601)
  Fix page crashing when duplicating immovable blocks (#3593)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@barklund barklund barklund approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cla: yes Signed the Google CLA

Projects

None yet

Milestone

v1.4

Development

Successfully merging this pull request may close these issues.

3 participants

@swissspidy @googlebot @barklund