Conversation
|
Preview URLs Flaws (205)URL:
(comment last updated: 2026-01-09 11:17:05) |
chrisdavidmills
left a comment
There was a problem hiding this comment.
@estelle looking mostly pretty good; just a few bits for you to sift through.
| - : See {{Cssxref("background-image")}}. Default: `none`. | ||
| - `<bg-position>` | ||
| - : See {{cssxref("background-position")}}. Default: 0% 0%. | ||
| - : See {{cssxref("background-position")}}. Default: `0% 0%`. |
There was a problem hiding this comment.
Just above here, <'background-color'> should be changed to <background-color> for consistency and correctness, surely? There are other instances further down too.
There was a problem hiding this comment.
I originally thought the same thing, but it's the only value that is an actual property, and the formal syntax has it this way.
There was a problem hiding this comment.
Ah, OK. Weird, but fair enough ;-)
|
|
||
| ### Property components | ||
|
|
||
| The `background` shorthand property enables you to declare all CSS background properties in a single declaration. The background lies underneath the content of an element; the content of an element is painted on top of it's background. When you have multiple, comma separated background values, each is background layer that is painted on top of the previous layers. |
There was a problem hiding this comment.
The Description content seems correct, but weirdly structured. It should start with this bit, which explains what the shorthand does at a high level, and the bit you've got at the top should be put further down.
Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>
| - : See {{Cssxref("background-image")}}. Default: `none`. | ||
| - `<bg-position>` | ||
| - : See {{cssxref("background-position")}}. Default: 0% 0%. | ||
| - : See {{cssxref("background-position")}}. Default: `0% 0%`. |
There was a problem hiding this comment.
I originally thought the same thing, but it's the only value that is an actual property, and the formal syntax has it this way.
chrisdavidmills
left a comment
There was a problem hiding this comment.
@estelle Yessss, looking great! I've just got one more nitpick for you, but I'll approve it. I don't think I need to see this again.
Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>
* 'main' of https://github.com/Anurella/content: (7201 commits) Update InterfaceData based on WebRef (mdn#42718) fix typo in `PeroformanceResourceTiming.requestStart` (mdn#42724) fix: auto-cleanup by bot (mdn#42717) fix typo in RTCTransportStats (mdn#42704) Fix typo in queuingStrategy variable name (mdn#42706) Update guidance on <hr> styling (mdn#42708) Fx147: Add relnote for iterator sequencing support (mdn#42713) Fix order of arguments in SpeechRecognitionPhrase ctor (mdn#42711) Update warnings about event handler attributes (mdn#42347) chore(deps-dev): bump fs-extra from 11.3.2 to 11.3.3 (mdn#42468) ci(deps): bump actions/download-artifact from 6.0.0 to 7.0.0 (mdn#42404) ci(deps): bump actions/upload-artifact from 5.0.0 to 6.0.0 (mdn#42402) ci(deps): bump peter-evans/create-pull-request from 7.0.11 to 8.0.0 (mdn#42403) Explain value repetition when fewer background values are specified (mdn#42627) Add description to background shorthand property (mdn#42695) chore(deps): bump @mdn/fred from 1.9.7 to 2.0.0 (mdn#42702) chore(deps): bump qs from 6.14.0 to 6.14.1 (mdn#42598) Bump node-html-parser from 7.0.1 to 7.0.2 (mdn#42659) update documentation to note that Firefox for Android requires CT (mdn#42672) TrustedTypes: Function() constructor + eval() (mdn#42462) ...
Creates the property description section.
move stuff that should be in description from other sections into the new section.
Add information of layer order.
add information about body bg being added to HTML bg
add containment exception
add links.
fixes #42643