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

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Cottser created an issue. See original summary.

star-szr’s picture

Issue summary: View changes
hussainweb’s picture

Status: Active » Needs review
Issue tags: +Quickfix
FileSize
713 bytes
star-szr’s picture

Status: Needs review » Reviewed & tested by the community

Looks perfect, thanks @hussainweb!

alexpott’s picture

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.