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

add hyphenate-limit-chars CSS property #22747

Merged
merged 9 commits into from
Dec 6, 2022

Conversation

chrisdavidmills
Copy link
Contributor

Description

Chrome is adding support for the hyphenate-limit-chars property in version 109, so this PR adds content for it to MDN. See my research document for more information.

Notes for reviewers:

  • At the moment, the iframe under the "Try it" heading will 404, because I have not yet added an interactive example for this property. That will be done in a separate PR
  • Also, the "Formal definition" section will show an error "Value not found in DB!", because the formal definitions have not been added to the relevant repo. This will be done in a separate PR too.

Motivation

Additional details

Related issues and pull requests

@chrisdavidmills chrisdavidmills requested a review from a team as a code owner December 5, 2022 14:46
@chrisdavidmills chrisdavidmills requested review from dipikabh and removed request for a team December 5, 2022 14:46
@github-actions github-actions bot added the Content:CSS Cascading Style Sheets docs label Dec 5, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Dec 5, 2022

Preview URLs

(comment last updated: 2022-12-06 06:55:01)

@kojiishi
Copy link

kojiishi commented Dec 5, 2022

lgtm, thank you.

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 for adding this page, @chrisdavidmills. I've added a few comments for your consideration.

files/en-us/web/css/hyphenate-limit-chars/index.md Outdated Show resolved Hide resolved
files/en-us/web/css/hyphenate-limit-chars/index.md Outdated Show resolved Hide resolved
files/en-us/web/css/hyphenate-limit-chars/index.md Outdated Show resolved Hide resolved
files/en-us/web/css/hyphenate-limit-chars/index.md Outdated Show resolved Hide resolved
files/en-us/web/css/hyphenate-limit-chars/index.md Outdated Show resolved Hide resolved
chrisdavidmills and others added 6 commits December 6, 2022 06:50
Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
@chrisdavidmills
Copy link
Contributor Author

Great stuff:

  • Thank you for the tech review, @kojiishi !
  • And thank you for the editorial review, @dipikabh !

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 for the updates, @chrisdavidmills! 👍

@dipikabh dipikabh merged commit d26ee02 into mdn:main Dec 6, 2022
@chrisdavidmills chrisdavidmills deleted the add-hyphenate-limit-chars branch December 7, 2022 08:40
@chrisdavidmills
Copy link
Contributor Author

Wonderful, thanks @dipikabh. Since you are here, could I also trouble you for a review of the associated BCD update and interactive example? ;-)

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
* add hyphenate-limit-chars CSS property

* adding short explanation of interactive example

* fix typo

* Update files/en-us/web/css/hyphenate-limit-chars/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>

* Update files/en-us/web/css/hyphenate-limit-chars/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>

* Update files/en-us/web/css/hyphenate-limit-chars/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>

* Update files/en-us/web/css/hyphenate-limit-chars/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>

* Update files/en-us/web/css/hyphenate-limit-chars/index.md

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>

Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants