-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Open
Labels
[Feature] Link EditingLink components (LinkControl, URLInput) and integrations (RichText link formatting)Link components (LinkControl, URLInput) and integrations (RichText link formatting)[Package] Block editor/packages/block-editor/packages/block-editor[Type] TaskIssues or PRs that have been broken down into an individual action to takeIssues or PRs that have been broken down into an individual action to take
Description
Previously: #18061 (comment)
Currently, URLInput
assigns required
attribute to its rendered input. This limits its reusability and makes unnecessary assumptions about its rendering context. It's assumed that this was the default largely based on the assumption of specific contexts in which it was rendered (i.e. RichText link insertion).
Task: Avoid assigning required
in URLInput
. It should still be supported as an optional (opt-in) prop in cases where it is appropriate. This usage and associated cautions should be sufficiently documented.
Metadata
Metadata
Assignees
Labels
[Feature] Link EditingLink components (LinkControl, URLInput) and integrations (RichText link formatting)Link components (LinkControl, URLInput) and integrations (RichText link formatting)[Package] Block editor/packages/block-editor/packages/block-editor[Type] TaskIssues or PRs that have been broken down into an individual action to takeIssues or PRs that have been broken down into an individual action to take