Skip to content

Conversation

dletorey
Copy link
Contributor

@dletorey dletorey commented Aug 27, 2025

Description

  • Added page for text-autospace
  • Added link on CSS Text page to text-autospace
  • Updated See also for text-spacing-trim

Motivation

Related issues and pull requests

@dletorey dletorey requested a review from a team as a code owner August 27, 2025 15:44
@dletorey dletorey requested review from estelle and removed request for a team August 27, 2025 15:44
@github-actions github-actions bot added Content:CSS Cascading Style Sheets docs size/m [PR only] 51-500 LoC changed labels Aug 27, 2025
dletorey and others added 4 commits August 27, 2025 16:45
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

github-actions bot commented Aug 27, 2025

Preview URLs

Flaws (6)

URL: /en-US/docs/Web/CSS/CSS_text
Title: CSS text
Flaw count: 2

  • macros:
    • Macro produces link /en-US/docs/Learn_web_development/Core/Styling_basics/Fundamental_CSS_comprehension which is a redirect
    • Macro produces link /en-US/docs/Learn_web_development/Core/Styling_basics/Cool-looking_box which is a redirect

URL: /en-US/docs/Web/CSS/text-autospace
Title: text-autospace
Flaw count: 2

  • macros:
    • Macro produces link /en-US/docs/Learn_web_development/Core/Styling_basics/Fundamental_CSS_comprehension which is a redirect
    • Macro produces link /en-US/docs/Learn_web_development/Core/Styling_basics/Cool-looking_box which is a redirect

URL: /en-US/docs/Web/CSS/text-spacing-trim
Title: text-spacing-trim
Flaw count: 2

  • macros:
    • Macro produces link /en-US/docs/Learn_web_development/Core/Styling_basics/Fundamental_CSS_comprehension which is a redirect
    • Macro produces link /en-US/docs/Learn_web_development/Core/Styling_basics/Cool-looking_box which is a redirect

(comment last updated: 2025-09-03 12:47:29)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@dletorey dletorey requested review from dipikabh and bsmth and removed request for estelle August 27, 2025 15:48
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 the page, @dletorey.

I'm not sure about the feature support, I don't see any visual changes in the spacing in the text with different values (in Nightly behind the pref). The spacing changes are more apparent in Chrome when switching between no-autospace and normal.

I've also included some updates required for the BCD for this property.

- `insert`
- : If there are no spaces between the _ideograph_ and _non-ideographic_ scripts the user agent automatically inserts the correct spacing.
- `replace`
- : If there are spaces between the _ideograph_ and _non-ideographic_ scripts, such as [U+0020](/en-US/docs/Glossary/Whitespace), the user agent automatically replaces it with the correct spacing.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- : If there are spaces between the _ideograph_ and _non-ideographic_ scripts, such as [U+0020](/en-US/docs/Glossary/Whitespace), the user agent automatically replaces it with the correct spacing.
- : Replaces existing spacing (such as [U+0020](/en-US/docs/Glossary/Whitespace)) between ideographic and non-ideographic characters with the specified spacing.

Copy link
Contributor

Choose a reason for hiding this comment

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

For both insert and replace, we can add that if not specified, the default value is insert.

Copy link
Contributor

Choose a reason for hiding this comment

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

This got missed perhaps

@bsmth bsmth removed their request for review August 29, 2025 10:34
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 a lot, @dletorey for all the fixes!
I have "unresolved" some comments - looks like they got missed. Could you take another look (descriptions of insert and replace)?

- : Provides more control over spacing behaviors. It accepts:
- the keyword `no-autospace`, or
- a combination of one or more of `ideograph-alpha`, `ideograph-numeric`, and `punctuation`, optionally followed by `insert` or `replace`.
- `no-autospace`
Copy link
Contributor

Choose a reason for hiding this comment

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

This and other values of <autospace> will need one more level of indentation. The current formatting is broken https://pr40912.review.mdn.allizom.net/en-US/docs/Web/CSS/text-autospace#values.

Comment on lines +21 to +26
text-autospace: ideograph-alpha insert;
text-autospace: ideograph-alpha replace;
text-autospace: ideograph-numeric insert;
text-autospace: ideograph-numeric replace;
text-autospace: punctuation insert;
text-autospace: punctuation replace;
Copy link
Contributor

Choose a reason for hiding this comment

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

My previous comment was more around showing more than one value of <autospace> like ideograph-alpha ideograph-numeric or ideograph-alpha ideograph-numeric punctuation.

text-autospace: ideograph-alpha ideograph-numeric punctuation;
text-autospace: ideograph-alpha ideograph-numeric;
text-autospace: ideograph-alpha ideograph-numeric insert;

- `insert`
- : If there are no spaces between the _ideograph_ and _non-ideographic_ scripts the user agent automatically inserts the correct spacing.
- `replace`
- : If there are spaces between the _ideograph_ and _non-ideographic_ scripts, such as [U+0020](/en-US/docs/Glossary/Whitespace), the user agent automatically replaces it with the correct spacing.
Copy link
Contributor

Choose a reason for hiding this comment

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

This got missed perhaps

Comment on lines +53 to +56
- `insert`
- : If there are no spaces between the _ideograph_ and _non-ideographic_ scripts the user agent automatically inserts the correct spacing.
- `replace`
- : If there are spaces between the _ideograph_ and _non-ideographic_ scripts, such as [U+0020](/en-US/docs/Glossary/Whitespace), the user agent automatically replaces it with the correct spacing.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- `insert`
- : If there are no spaces between the _ideograph_ and _non-ideographic_ scripts the user agent automatically inserts the correct spacing.
- `replace`
- : If there are spaces between the _ideograph_ and _non-ideographic_ scripts, such as [U+0020](/en-US/docs/Glossary/Whitespace), the user agent automatically replaces it with the correct spacing.
- `insert`
- : Adds the specified spacing only if there are no existing spaces between the ideographic and non-ideographic scripts.
- `replace`
- : Replaces existing spacing (such as [U+0020](/en-US/docs/Glossary/Whitespace)) between ideographic and non-ideographic characters with the specified spacing.

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.

Leaving a +1 to unblock you since I'll be away next week. You can verify in the preview link if the formatting of the "Values" section looks good.

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 size/m [PR only] 51-500 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants