Problem/Motivation

When a Name field is configured as ROP, the UI should:

  • Respect the Name setting “Show required markers”.
  • Paint blue indicators on the configured minimum components.
  • Avoid HTML5 blocking and red asterisks from Name’s minimum components when ROP is in effect.

Current behavior conflicts with Name’s internal required markers.

Steps to reproduce

  1. Configure a Name field with minimum components (e.g., Given + Family) and “Show required markers”.
  2. Enable ROP on that Name field.
  3. Observe incorrect/extra red markers or missing blue markers on the wrapper or non-minimum components.

Proposed resolution

This solution probably requires #3552451: Required state incorrectly marks non-minimum components as required.

  • Set #required_on_publish on .name-core-component.
  • Toggle .form-required-on-publish minimum components.

Remaining tasks

  • Ensure respects Name field's show_required_markers setting.
  • Ensure HTML5 required attribute does not block submission.
  • Test coverage

User interface changes

Blue "required on publish" markers appear on the Name wrapper and only the minimum components; Name's red asterisks suppressed when ROP applies.

CommentFileSizeAuthor
#4 require_on_publish-respect-name-field-minimum-components-3554298-4.patch30.42 KBjcandan

Issue fork require_on_publish-3554298

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

jcandan created an issue. See original summary.

jcandan opened merge request !42

jcandan’s picture

jcandan commented

[wrong issue]

jcandan’s picture

jcandan commented
StatusFileSize
new require_on_publish-respect-name-field-minimum-components-3554298-4.patch30.42 KB
jcandan’s picture

jcandan commented
Assigned: jcandan » Unassigned
Status: Active » Postponed

Marking postponed. This is being addressed as a part of #3567063: Fix missing required marker on composite fields. If that gets merged, we can consider this Fixed.

jcandan’s picture

jcandan commented
Status: Postponed » Fixed

Considered fixed as part of #3567063: Fix missing required marker on composite fields.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.