Skip to content

Remove deprecated word-wrapping Stylelint suppressions added in #80738 #82293

Description

@ciampo

Related to #80738.

What problem does this address?

PR #80738 enables Stylelint 17's deprecated property and value rules. To keep that dependency upgrade behavior-neutral, it adds 23 local suppressions around existing word-wrapping declarations:

  • 20 suppressions for word-break: break-word under declaration-property-value-keyword-no-deprecated.
  • 3 suppressions for word-wrap: normal under property-no-deprecated.

Most are marked Pending review of overflow-wrap alternative. These should not become permanent lint debt. The logical CSS suppressions whose rule names were migrated in #80738 already existed before that PR and are not part of this issue.

Proposed solution

Review each declaration in its layout context and replace it with standards-based word wrapping that preserves the current behavior. Do not apply one mechanical replacement everywhere. In particular, check intrinsic sizing, tables, flex and grid children, long URLs or identifiers, and editor versus front-end block styles. The Button block already records that overflow-wrap alone did not handle its nested-link case, so it needs a focused solution.

Remove all 23 suppressions introduced by #80738:

word-break: break-word

word-wrap: normal

  • packages/block-editor/src/components/responsive-block-control/style.scss
  • packages/block-library/src/common.scss
  • packages/ui/src/visually-hidden/style.module.css

Completion criteria

  • All 23 suppressions introduced by Update Stylelint to v17 and related packages #80738 are removed.
  • Each replacement preserves line breaking and intrinsic sizing in its actual layout.
  • Block styles are checked in both the editor and front end where applicable.
  • Focused regression coverage is added where a plain declaration-level check cannot prove the behavior.
  • npm run lint:css passes without a global exception for these deprecated declarations.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    CSS StylingRelated to editor and front end styles, CSS-specific issues.[Type] Code QualityIssues or PRs that relate to code quality

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions