Problem/Motivation
The last sentence in the text information at the top of the admin/config/services/aggregator page still reads "For each feed, the latest items block may be enabled at the blocks administration page." This was correct for D7, but for D8, you place a 'Aggregator Feed' block and select which feed it is to show. I found this confusing, and I almost filed an issue saying that the latest items blocks weren't being created. This should be corrected.
Steps to reproduce
Enable aggregator.
Go to admin/config/services/aggregator.
Read the help text.
Proposed resolution
Change
For each feed, the latest items block may be enabled at the blocks administration page.
to
For each feed, the Aggregator feed block may be enabled at the block layout page.
Remaining tasks
User interface changes
Before

After

API changes
Data model changes
Release notes snippet
| Comment | File | Size | Author |
|---|---|---|---|
| #17 | 2610520-17.patch | 1.71 KB | quietone |
| #17 | After.png | 26.23 KB | quietone |
| #17 | Before.png | 26.7 KB | quietone |
| #8 | documentation_wrong_aggregator-2610520-8.patch | 1.77 KB | hansfn |
| #6 | interdiff-2610520-3-6.txt | 1.67 KB | snehi |
Comments
Comment #2
misc commentedI will have a patch for this tomorrow, so assigning to me.
Comment #3
misc commentedProposed path attached.
Comment #4
dcam commentedTagging.
I can't do a proper review of the patch at the moment, but I do think that the wording sounds strange. I was thinking something more along the lines of "To display a feed, enable an aggregator feed block at the blocks administration page", with the same HTML added of course.
Comment #5
snehi commentedComment #6
snehi commentedChanged as suggested.
Comment #7
snehi commentedComment #8
hansfn commentedThis documentation bug has caused many problems for my students, so I decided to update the patch for 8.1.x hoping it will be included.
Comment #17
quietone commentedUpdate IS and updated the patch. No interdiff because this is small.
Comment #18
antoniya commentedWe discussed this at #3253969: Drupal Usability Meeting 2021-12-17 and agreed that this is a very valid request.
As mentioned in the meeting, one could argue about rephrasing the passive 'may be enabled', but none of the meeting attendees felt really bothered by the current suggestion. I'll mark this as RTBC as it is a rather minor change and has been reviewed by a few people already.
Comment #20
catchCommitted/pushed to 10.0.x and cherry-picked to 9.4.x, thanks!