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 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 commented2: docs-update-2176085-2.patch queued for re-testing.
Comment #6
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.