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

Quadrat: add custom logo support #3761

Closed
wants to merge 47 commits into from
Closed

Quadrat: add custom logo support #3761

wants to merge 47 commits into from

Conversation

jffng
Copy link
Contributor

@jffng jffng commented May 5, 2021

This PR adds support for a custom logo. It depends on this PR: WordPress/gutenberg#31513

Screen Shot 2021-05-05 at 12 06 11 PM

This is a part of #3661

jffng and others added 30 commits May 4, 2021 13:35
* Quadrat: Add new heading sizes

* Quadrat: Further work on Lists and Quotes

* Quadrat: Remove whitespace

* Quadrat: Add responsive Heading font-sizes
* Add the hosts block pattern.

* Fix categories.

* Update img refs.

* Add alt text to media text pattern.

* Revise block pattern names.
* override BCB's templates with php ones
#3663)

* Quadrat: Add responsive Paragraph styles and start styling the Code block

* Quadrat: Remove Code block styling

* Revise heading max vw.

Co-authored-by: Jeff Ong <[email protected]>
* added Latest episodes block pattern

* translatable strings

* staging url for media

* rebased, updated texts and adjusted paddings

* added a group with padding to the blocks

* Body line-height to 1.7.

Co-authored-by: Jeff Ong <[email protected]>
* Modified site-title styles to match comps

* Added a navigation rendering function to render the navigation block based on a classic data source, wrap the output in an HTML block to be inserted anywhere blocks are do_block()'ed

* Added a header template part that leverages the rendered navigation block markup

* Replaced usage of navigation block template with new template-part

* Added styles for header for proper layout unachievable with blocks.
pbking and others added 13 commits May 4, 2021 13:35
* Change colors used in theme.json from descreptive to semantic

* semanticizing the color

* formatting

* also changed the child theme.json
* Quadrat: Add styles for tables

* Add tbody for extra specificity

* Add a border to the top of the first row

* Remove text-align.

Co-authored-by: Jeff Ong <[email protected]>
* Rebase trunk.

* Revise footer.

* Add CSS for the next/prev links

* Continue rebase.

* remove test code

* used gutenberg_block_template_part to load template parts on php templates

* Remove testing text

* alignment tweaks

* Add separate templates for page and post.

* Restore header wrapper that was accidentally removed in rebase.

* Template tweaks.

* Add page block template.

Co-authored-by: Jeff Ong <[email protected]>
Co-authored-by: Maggie Cabrera <[email protected]>
* created block pattern

* padding rules for mobile

* added new class to all the group blocks

* Remove all padding from text content on mobile.

Co-authored-by: Jeff Ong <[email protected]>
@jffng jffng changed the base branch from trunk to make/quadrat May 5, 2021 15:49
@jffng jffng requested review from a team, kjellr and beafialho May 5, 2021 17:14
@jffng jffng added this to the Quadrat v1 milestone May 5, 2021
<!-- wp:site-title /-->' .
'<!-- wp:group {"style":{"spacing":{"padding":{"top":"35px","right":"35px","left":"35px"}}}} --><div class="wp-block-group" style="padding-top:35px;padding-right:35px;padding-left:35px">
<!-- wp:group --><div class="wp-block-group">
<!-- wp:site-logo {"width":100} /-->
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure what the default width should be.

@kjellr
Copy link
Contributor

kjellr commented May 5, 2021

We'll need to adjust the alignment + location of the site title and nav when this logo is present. @beafialho can you provide a recommendation?

@beafialho
Copy link
Collaborator

Logo's default width should be 150px

On Desktop, it should be placed like this, and the site title and nav should be middle aligned with the logo.

Captura de ecrã 2021-05-06, às 10 56 42

Captura de ecrã 2021-05-06, às 10 57 47

On Mobile, it's placed like this and the site title and nav are middle aligned below the logo.

Captura de ecrã 2021-05-06, às 10 50 18

Base automatically changed from make/quadrat to trunk May 6, 2021 15:06
@jffng
Copy link
Contributor Author

jffng commented May 7, 2021

Closing as it's a duplicate, but will still reference the design mocks here.

@jffng jffng closed this May 7, 2021
@scruffian scruffian deleted the add/quadrat-logo branch May 7, 2021 21:33
@jffng jffng mentioned this pull request May 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants