System module template form-element should provide class form-type-<type>

The Needs Review Queue Bot → tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Re-roll against D10.

Reviewed patch #20 getting "form-type". Please refer screen shot

This looks like an oversight in the original classes, but tagging for front end framework manager review since I'm not sure how it sits with classy/stable deprecations etc.

The problem we might have here is that technically stable templates are supposed to be stable, i.e. have no changes.

We discussed this with @bnjmnm and we agree this extra class would be useful. But as @larowlan says this shouldn't be a change to Stable, only to the module itself. So moving to Needs work to remove this from the last patch.

Confirmed patch #26 is only applying to module. With such a small change wonder if it needs a change record?

The last submitted patch, 26: 3051692-26.patch, failed testing. View results →

Patch still applies cleanly
random test failure

The last submitted patch, 26: 3051692-26.patch, failed testing. View results →

Checked starterkit_theme and it already has this class too. Created an MR, hiding patches.

Seems straight forward then.

nod_ → closed merge request !12578

Committed faec142 and pushed to 11.x. Thanks!