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

Reference: https://www.drupal.org/core/beta-changes
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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kattekrab’s picture

Issue summary: View changes
kattekrab’s picture

kattekrab’s picture

And here's a patch.

kattekrab’s picture

Status: Active » Needs review
FileSize
14.71 KB
13.23 KB

And here's screengrabs of manual before and after on simplytest.me

Before

After

kattekrab’s picture

Issue summary: View changes
kattekrab’s picture

Issue summary: View changes
YesCT’s picture

Category: Task » Bug report
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

#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.

  • webchick committed dac288e on 8.0.x
    Issue #2409359 by kattekrab: Remove "posts per page" from Site...
webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed, and, pushed, to, 8.0.x, thanks! ;)

kattekrab’s picture

W00t!

On a plane right now waiting for take off. mel - lax - pdx for da board meeting.

Yay!!! thanks @yesct and @webchick!!!!

D.

Status: Fixed » Closed (fixed)

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