[wp-trac] [WordPress Trac] #40940: Twenty Twelve uses invalid linear-gradient() syntax
WordPress Trac
noreply at wordpress.org
Tue Aug 17 10:02:16 UTC 2021
#40940: Twenty Twelve uses invalid linear-gradient() syntax
---------------------------+------------------------------
Reporter: vrubleg | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Bundled Theme | Version: 4.7.5
Severity: normal | Resolution:
Keywords: has-patch | Focuses:
---------------------------+------------------------------
Changes (by sabernhardt):
* keywords: => has-patch
Comment:
Thanks for the report, and sorry for the delayed response.
Since opening this ticket, the gradient syntax was corrected in
changeset:45124.
Changeset r22899 applied the gradient to `button` element in its
hover/focus/active states, though not to the `button` element itself. For
consideration, [attachment:"40940.patch"] adds that. This would alter the
styling for at least the search block's submit button and custom buttons
(in Custom HTML or Classic blocks, or within a child theme template).
I did not include changes for the following, but of course they still
might be worth editing.
1. Removing vendor prefixes: Yes, these are probably all unnecessary at
this point. For comparison, Twenty Thirteen only has the `-webkit` prefix,
and Twenty Seventeen includes others again. Twenty Twenty and Twenty
Twenty-One do not include prefixes for gradients.
2. Removing the `.comments-area article header h4` style: I cannot find an
`h4` heading in the comments section, even in the
[https://themes.trac.wordpress.org/browser/twentytwelve/1.0/functions.php#L275
original functions.php file].
3. Editing `.bypostauthor cite span`: I think this part would be better to
leave as it is. The second set overrides ''almost'' all properties from
the first, but it maintains the font-size and line-height. (Adding those
three lines could negate the benefit of removing the three that reset.)
--
Ticket URL: <https://core.trac.wordpress.org/ticket/40940#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list