Prefix is shown before description

Issue Fixed:

SS:

I've Tested MR 8695 on Drupal 10.3.x-dev
MR is applied Successfully...

Testing steps:

1. Create a Webform with the Webform module.
2. Create a 'Number' element.
3. Fill in the description.
4. Fill in the prefix.
5. Set 'Description display' to 'Before'.
6. Save the element and view the form.

The Prefix is shown before the Number field instead of before the description.

Attaching SS for reference
RTBC+1

Keeping "Needs review" for Code verification

MR should be against 11.x branch.

Also probably will need test coverage and definitely a change record.

Recommend steps are updated using just core without a contrib to make sure the bug isn't from contrib.

Sourav_Paul → closed merge request !8695

• web/core/profiles/demo_umami/themes/umami/templates/classy/form/form-element.html.twig
• web/core/themes/stable9/templates/form/form-element.html.twig
• web/core/themes/starterkit_theme/templates/form/form-element.html.twig

All these core templates follow the same structure for prefix

  {% if prefix is not empty %}
    <span class="field-prefix">{{ prefix }}</span>
  {% endif %}
  {% if description_display == 'before' and description.content %}
    <div{{ description.attributes }}>
      {{ description.content }}
    </div>
  {% endif %}

Shouldn't we update all of them. Or is it something that should be handled by the 'webform' module itself?

I think we should. I see no valid reason or argument to put the prefix before the description, so it seems like a mistake or bug. The prefix is mostly used to indicate what value the user needs to enter, such as a currency.

Was previously tagged for tests.

And if a template needs changed then all of them would have to be updated with a change record.

Still tagged for tests.

Also has anyone gone through the git history for when this was added to see if done on purpose?

From a non-developer perspective, it's a bug as it feels no purpose to put the prefix before the description but, if we put the prefix below the description, it looks quite meaningful as the prefix is mostly used to indicate what value the user needs to enter.

I do appreciate with #13

sahana _n → made their first commit to this issue’s fork.

I Uptaed MR Please review.

I am happy to take suggestions for improvements. Please let me know.
Thank you!!

Pipeline has errors.

sriharsha.uppuluri → made their first commit to this issue’s fork.

Could anyone build proper tests & fix pipeline errors?