Follow-up to #2214241: Field default markup - removing the divitis

Problem/Motivation

The default field.html.twig has some documentation regressions now, likely caused by copying the docblock from a Classy or Bartik template.

  1. The first line says "Theme override" rather than "Default theme implementation…".
  2. The @ingroup themeable line is now missing which adds the template to the listing at https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Render%21....

Proposed resolution

Revert these changes, see #2214241-353: Field default markup - removing the divitis.

Remaining tasks

  • Patch
  • Review

User interface changes

n/a

API changes

n/a

Data model changes

n/a

CommentFileSizeAuthor
#3 template_documentation-2557901-3.patch713 byteshussainweb

Comments

Cottser created an issue. See original summary.

star-szr’s picture

star-szr
he/him
Primary language English
commented
Issue summary: View changes
hussainweb’s picture

hussainweb
Primary language English
commented
Status: Active » Needs review
Issue tags: +Quickfix
StatusFileSize
new template_documentation-2557901-3.patch713 bytes
star-szr’s picture

star-szr
he/him
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Looks perfect, thanks @hussainweb!

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed ced413c and pushed to 8.0.x. Thanks!

  • alexpott committed ced413c on 8.0.x 
    Issue #2557901 by hussainweb: Template documentation cleanup after field...

Status: Fixed » Closed (fixed)

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