Problem/Motivation
The description under Site Information on the Configuration page ( /admin/config) is no longer accurate.
It says:
"Change site name, email address, slogan, default front page, and number of posts per page, error pages."
The number of posts per page setting has been removed as that is now provided by a view.
Note: This is a regression, it was previously fixed in #2095249: Change description text for admin/config/system/site-information to remove "number of posts per page"
Beta phase evaluation
Issue category | Bug because the text was wrong. |
---|---|
Issue priority | Normal because it effects only one isolated part. |
Unfrozen changes | Unfrozen because it only changes text. | Disruption | NOT Disruptive for core/contributed and custom modules/themes because it will require a BC break/deprecation/internal refactoring/widespread changes... |
Since it is a unfrozen change only, is allowed in the beta.
Proposed resolution
Update the text to simply remove "number of posts per page" from the site info config description.
Remaining tasks
Write the patch.Done.- (done) Review the patch.
Possibly needs a change notice, as people looking to change this setting may not realise this is now handled by views.
User interface changes
Yes, text change.
API changes
None.
Comment | File | Size | Author |
---|---|---|---|
#4 | 2409359-afterpatch.png | 13.23 KB | kattekrab |
#4 | 2409359-beforepatch.png | 14.71 KB | kattekrab |
#3 | 2409359-remove-posts-per-page-3.patch | 679 bytes | kattekrab |
d8-config-siteinfo-noposts.png | 90.76 KB | kattekrab |
Comments
Comment #1
kattekrab CreditAttribution: kattekrab commentedComment #2
kattekrab CreditAttribution: kattekrab commentedComment #3
kattekrab CreditAttribution: kattekrab commentedAnd here's a patch.
Comment #4
kattekrab CreditAttribution: kattekrab commentedAnd here's screengrabs of manual before and after on simplytest.me
Before
After
Comment #5
kattekrab CreditAttribution: kattekrab commentedComment #6
kattekrab CreditAttribution: kattekrab commentedComment #7
YesCT CreditAttribution: YesCT commented#2226903: Step 1: Move static menu links to yml files accidentally put it back in.
Anyway... luckily, you have the comma correctly before the "and" this time.
Comment #9
webchickCommitted, and, pushed, to, 8.0.x, thanks! ;)
Comment #10
kattekrab CreditAttribution: kattekrab commentedW00t!
On a plane right now waiting for take off. mel - lax - pdx for da board meeting.
Yay!!! thanks @yesct and @webchick!!!!
D.