Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
This sentence, with former/latter anaphoric reference is confusing:
// Look at the default display and display for the view mode, and fallback to
// the former if the latter does not exist is disabled.
Replace the references with the actual nouns so a reader is not forced to parse a sentence for an already confusing concept.
// Look at the default display and display for the view mode, and fallback to
// the default display if the display for the view mode does not exist
// or is disabled.
Comment | File | Size | Author |
---|---|---|---|
#2 | docs-update-2176085-2.patch | 684 bytes | jessebeach |
#2 | interdiff.txt | 721 bytes | jessebeach |
docs-cleanup.patch | 1.16 KB | jessebeach | |
Comments
Comment #1
valthebaldCan this be shorter please? Repeating definitions make understanding hard.
Something like
Comment #2
jessebeach CreditAttribution: jessebeach commented"Look at [1] and [2], fall back to [1] when [2] is not found"
Is the original template. I changed it to this template, taking out the unnecessary preamble:
"Fall back to [1] when [2] is not found"
Comment #4
jessebeach CreditAttribution: jessebeach commented2: docs-update-2176085-2.patch queued for re-testing.
Comment #6
jessebeach CreditAttribution: jessebeach commented2: docs-update-2176085-2.patch queued for re-testing.
Comment #7
jhodgdonThanks - definitely less confusing!
This cannot get committed until after the 22nd as we're in an "only major/critical commits" pre-alpha week.
Comment #8
jhodgdonThanks again! Committed to 8.x.