Skip to content

feat(ui5-textarea): introduce new character counter property and logic#13695

Open
aleksandar-terziev wants to merge 1 commit into
mainfrom
text-area-counter-poc
Open

feat(ui5-textarea): introduce new character counter property and logic#13695
aleksandar-terziev wants to merge 1 commit into
mainfrom
text-area-counter-poc

Conversation

@aleksandar-terziev

@aleksandar-terziev aleksandar-terziev commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

JIRA: BGSOFUIRILA-4331

  • New property showExceededTextFocus
  • Visibility logic updated:
    • shows character counter when:
      • focused
      • exceeded
      • valueState is "Critical" or "Negative"
  • if the new property showExceededTextFocus and the existing property showExceededText are both set, the new showExceededTextFocus property logic takes precedence.
  • For space reservation visibility: hidden is used
  • ValueState-based counter coloring - Critical = warning color, Negative = error color

@sap-ui5-webcomponents-release

sap-ui5-webcomponents-release Bot commented Jun 15, 2026

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown

👋 Heads-up: dev close is in effect

Thanks for the contribution! This repository is currently in dev close ahead of release next (scheduled 2026-07-09, UTC). See the release schedule for the full timeline.

This PR appears to introduce public-API changes (detected by diffing the Custom Elements Manifest against the latest published version on npm):

@ui5/webcomponents

  • ➕ added property: showExceededTextFocus
  • ➕ added attribute: show-exceeded-text-focus
  • 🔄 changed attribute value-state (type)

Could you please hold off on merging into main until the release ships? Public-API changes are best landed in the next dev cycle so they don't slip into the release at the last minute. Once the release is out, this PR is good to go.

If this change must ship in the current release, please request a review from one or two members of @UI5/ui5-team-webc so the team can sign off explicitly.

💬 False positive? If you believe this PR doesn't actually change the public API (e.g. only internal refactoring, or an entry the detector mis-attributed), please reply on this thread — your feedback helps us improve the detection during this trial run.

Posted automatically by the Dev Close Notice workflow.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant