Opened 5 years ago
Closed 5 years ago
#46770 closed defect (bug) (fixed)
Twenty Eleven: Background gradient argument
Reported by: | Malae | Owned by: | desrosj |
---|---|---|---|
Milestone: | 5.2 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Bundled Theme | Keywords: | has-patch commit |
Focuses: | Cc: |
Description
Line 1587
.featured-post .feature-text:after, .featured-post .feature-image.small:after
Value Error : background The first argument to the linear-gradient function should be to top, not top )
Attachments (4)
Change History (14)
#1
@
5 years ago
- Component changed from Themes to Bundled Theme
- Summary changed from Twentyeleven theme style.css to Twenty Eleven: Background gradient argument
#3
@
5 years ago
- Keywords has-patch added; needs-patch removed
- Milestone changed from Awaiting Review to Future Release
- Severity changed from minor to normal
#5
@
5 years ago
Thanks for your patch @man4toman,
In order to better track issues in each theme, could you please remove the Twenty Twelve
fix from your patch and create a new ticket just for that theme. Thanks
#6
@
5 years ago
Hi @ianbelanger
The new patch attached, I'll create a new ticket for "Twenty Twelve".
#7
@
5 years ago
- Keywords commit added
- Milestone changed from Future Release to 5.2
- Version 4.9.8 deleted
Thanks @man4toman,
The patch looks good, I am marking this for commit
#8
@
5 years ago
As pointed out by @joyously in #46786
The patch doesn't actually fix the problem, because this ticket was just written for syntax. The real problem is that the old syntax goes the other direction from the new syntax.
For example,
-webkit-linear-gradient(top, #f4f4f4, #e6e6e6);
will start at top with #f4f4f4 and go down to #e6e6e6. But
linear-gradient(to top, #f4f4f4, #e6e6e6);
will start at bottom with #f4f4f4 and go to top to #e6e6e6.
So in 46770.4.diff I have switched linear-gradient
to use to bottom
to correct the issue.
#46773 was marked as a duplicate.