Skip to content
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

Document font-palette & @font-palette-values with its descriptors #22519

Merged
merged 84 commits into from
Dec 6, 2022

Conversation

dletorey
Copy link
Contributor

Description

While working on the [CSS] Ship COLRv1 color-font support #21537 I have created pages for:

  • font-palette CSS Property
  • @font-palette-values CSS at-rule
  • font-family CSS descriptor for @font-palette-values
  • base-palette CSS descriptor for @font-palette-values
  • override-colors CSS descriptor for @font-palette-values
  • CSS data-type

Motivation

Completing documentation for the Firefox release of COLRv1 color-font support

Additional details

Related issues and pull requests

Relates to #21537

@dletorey dletorey requested a review from a team as a code owner November 24, 2022 15:09
@dletorey dletorey requested review from estelle and removed request for a team November 24, 2022 15:09
@github-actions github-actions bot added the Content:CSS Cascading Style Sheets docs label Nov 24, 2022
@dletorey dletorey requested a review from a team as a code owner November 24, 2022 16:57
@dletorey dletorey requested review from Elchi3 and removed request for a team November 24, 2022 16:57
@github-actions github-actions bot added the Content:Other Any docs not covered by another "Content:" label label Nov 24, 2022
@dipikabh dipikabh self-assigned this Nov 29, 2022
@dipikabh dipikabh changed the title Documented font-palette & @fotn-palette-values with its descriptors Document font-palette & @font-palette-values with its descriptors Nov 29, 2022
Copy link
Contributor

@dipikabh dipikabh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @dletorey for answering my comments. Please go ahead and update the text as you've suggested.
Also for now, you can ignore my comments about adding more descriptions to Examples and Result sections. Let's do that in a new PR after this one lands.

Copy link
Contributor

@dipikabh dipikabh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @dletorey for your extensive work on this feature and all the related pages! Let's land this 🎉.

@dipikabh dipikabh merged commit 27ae7e9 into mdn:main Dec 6, 2022
wbamberg added a commit to wbamberg/content that referenced this pull request Dec 8, 2022
* upstream/main: (286 commits)
  Correct Epsilon property description (mdn#22804)
  Fix typo in subtle crypto digest (mdn#22802)
  Alt tag added - reference issue mdn#21616 (mdn#22255)
  WeakSet: Improve recursive validation example (mdn#22787)
  New page: validityState.valueMissing (mdn#22691)
  Edit: <dialog> element (mdn#22786)
  Update Paint Timing pages (mdn#22771)
  chore(deps): bump @mdn/yari from 1.27.16 to 1.27.20 (mdn#22791)
  chore(deps): bump technote-space/get-diff-action from 6.1.1 to 6.1.2 (mdn#22790)
  Remove WebGLSidebar (mdn#22782)
  fix indefinite articles usage (mdn#22789)
  Removed part of sentence that refers to code that appears to have been removed (mdn#22788)
  Split (Async)?(Generator)?Function class and ctor pages (mdn#22778)
  Fix interfaces links (mdn#22779)
  Change "hypertext link" to "hypertext links" (mdn#22780)
  add hyphenate-limit-chars CSS property (mdn#22747)
  22225: Add links to accessibility guides section (mdn#22240)
  ci(markdownlint): use GitHub API to get changed files (mdn#22773)
  Document font-palette & @font-palette-values with its descriptors (mdn#22519)
  fix typo in createImageBitmap api (mdn#22774)
  ...
hamishwillee pushed a commit to hamishwillee/content that referenced this pull request Dec 12, 2022
…n#22519)

* created the pages needed with default content

* added more tags

* added first draft of the font-palette page

* added the first draft of font-palette-values page

* created the dashed-ident paged and referenced it to/from ident and custom-ident

* fixed explanations and missing punctuation

* updated the @rules to be consistent with At-rules

* americanized the spelling an dmade @rules consistent

* updated all links to be markdown links

* updated the bcd location

* added content for font-family & override-colors descriptors

* updated the font-family descriptor with grammar issues

* added images to base-palette and override-colors and descriptions

* completed the override-colors descriptor and added references to css descriptor on each page

* added the COLRv1 color-font new features to the release notes

* fixed the broken link in the content

* Update files/en-us/mozilla/firefox/releases/107/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/font-palette/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/font-palette/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/font-palette/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/font-palette/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/font-palette/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/font-palette/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/font-palette/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/font-palette/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/font-family/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/font-family/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/font-family/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/font-family/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/font-family/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/font-family/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/font-family/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/font-family/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/font-family/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/base-palette/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/base-palette/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/base-palette/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/base-palette/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/base-palette/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/base-palette/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/base-palette/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/base-palette/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/base-palette/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/override-colors/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/override-colors/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/override-colors/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/override-colors/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/@font-palette-values/override-colors/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/dashed-ident/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/dashed-ident/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/dashed-ident/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/dashed-ident/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/dashed-ident/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/dashed-ident/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/dashed-ident/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/dashed-ident/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/dashed-ident/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/dashed-ident/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/dashed-ident/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/web/css/dashed-ident/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

* added a description of the dark palette example mdn#22519

* updated the introductory sentence about @font-palette-values mdn#22519

* updated the description of the override-colors descriptor mdn#22519

* added a description of the override-colors example mdn#22519

* added explanation of why base-palette is not always needed mdn#22519

* updated the name of the example mdn#22519

* rewritten the explanation of using the same identifier for multiple fonts mdn#22519

* rewritten the explanation of the use of the font-family name mdn#22519

* updated the name of the liveSample mdn#22519

* updated the example name mdn#22519

* updated the description of the changing default palette example mdn#22519

* added spacing to the syntax example mdn#22519

* rewqritten the explanation about color and base-palette indexes mdn#22519

* removed the example headings

* added a difference between custom and dashed idents mdn#22519

* added a note that there is no bcd for these types mdn#22519

* Update files/en-us/mozilla/firefox/releases/107/index.md

Co-authored-by: Dipika Bhattacharya <[email protected]>

Co-authored-by: Dipika Bhattacharya <[email protected]>
@dletorey dletorey deleted the 21537 branch December 21, 2022 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:CSS Cascading Style Sheets docs Content:Other Any docs not covered by another "Content:" label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants