API page: https://api.drupal.org/api/drupal/core%21modules%21language%21src%21Elem...

 * Provides language element configuration.

This doesn't make much sense.

It should say that this class defines an element for language configuration for a single field.

CommentFileSizeAuthor
#2 2859502-2.patch532 byteschiranjeeb2410
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joachim created an issue. See original summary.

chiranjeeb2410’s picture

chiranjeeb2410 CreditAttribution: chiranjeeb2410 at Google Summer of Code commented
Status: Active » Needs review
FileSize
2859502-2.patch532 bytes

@joachim,

Changed the required comment. Uploading fresh patch.
Please review.

joachim’s picture

joachim CreditAttribution: joachim as a volunteer commented
Status: Needs review » Reviewed & tested by the community

Looks good! Thanks!

  • xjm committed ae3c5b5 on 8.4.x 
    Issue #2859502 by chiranjeeb2410, joachim: class docblock is badly...

  • xjm committed 5ea4d35 on 8.3.x 
    Issue #2859502 by chiranjeeb2410, joachim: class docblock is badly...
xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Version: 8.4.x-dev » 8.3.x-dev
Status: Reviewed & tested by the community » Fixed
Issue tags: +rc eligible

Thanks @joachim and @chiranjeeb2410.

As a documentation improvement, this issue is RC eligible, so I've committed it to 8.4.x and cherry-picked it to 8.3.x.

Status: Fixed » Closed (fixed)

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