-
Notifications
You must be signed in to change notification settings - Fork 5
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 feature banner to Front Page. #37
Conversation
70284fa
to
ade8629
Compare
@media (min-width: 782px) { | ||
.wporg-hero-details { | ||
padding-top: 0; | ||
margin-top: -64px !important; |
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.
This is a bit sensitive. Can't think of a better way.
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 and looks good! 👍
Not so sure about this, any potential problems you found that might happen by using it? |
72d04a7
to
c45a0d7
Compare
* @return array The enhanced allowed mime types. | ||
*/ | ||
function wporg_add_svg_to_upload_mimes( $upload_mimes ) { | ||
if ( current_user_can( 'manage_options' ) ) { |
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.
@dd32 Does this make any sense? To allow SVG uploads but lock it down to admins?
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.
In all honesty, this doesn't belong in the theme, and I'm not sure the context of adding it.
There's also something to say that this should be using unfiltered_upload
, but that isn't available to anyone on WordPress.org.
I think I'd rather enable a plugin such as https://wordpress.org/plugins/safe-svg/ (or one of the others) on the site instead if SVG upload is required.
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.
Yeah fair, I'll avoid adding SVG support.
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 we'll need SVG support eventually, so I've added a low-priority issue over in mu-plugins: WordPress/wporg-mu-plugins#300
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.
Thanks!
Fixes: #26
This PR uses a
query
block withperPage=1
andsticky posts = only
to retrieve a post to display as the featured site on the homepage.Notes
query_loop_block_query_vars
.27
characters or less.h1
is added with "showcase" andclass=screen-reader-text
Screenshots
How to test
/
SVG