-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add the new site-header template for the DotOrg design #316
Conversation
Add some changes in the generation of the social tags
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested locally and works nicely 👍
echo do_blocks( '<!-- wp:wporg/global-header /-->' ); | ||
|
||
gp_enqueue_styles( array( 'gp-jquery-webui-popover', 'driver-js' ) ); | ||
gp_enqueue_scripts( array( 'gp-tour' ) ); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it make sense to use wp_enqueue_styles()
and wp_enqueue_scripts()
instead? Also, I'm wondering if we need the tour in the Translation Events (I'm assuming jquery-webui-popover
and driver-js
are for the tour).
|
||
?> | ||
|
||
<div class="event-list-top-bar"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it could make sense to remove this from this PR for now, as it doesn't currently adhere to the new design, and implement it in a later PR with the new design.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving since we we can improve it other PRs, if needed.
Merging, we can iterate in future PRs. |
This PR implements the new design for the site-header at translate.wordpress.org, using only blocks. This PR:
site-header
template in the new design.global-header
block.gp_tmpl_header();
method.Currently, this PR only works in My events, because we don't have another main templates in the new design.
New design with the current template system
New design with blocks
Currently, we have a problem with the breadcrumb at translate.wordpress.org, because it is not working. We should address this problem in a future PR.