Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[UX] Oddities with the "Number of posts on home page" setting in admin/config/system/site-information #5432

Open
klonos opened this issue Jan 3, 2022 · 7 comments · May be fixed by backdrop/backdrop#3897

Comments

@klonos
Copy link
Member

klonos commented Jan 3, 2022

Steps To Reproduce

  1. Go to admin/config/system/site-information
  2. Change the home page path to node -> notice that no additional option is shown.
  3. Save the settings -> the "Number of posts on home page" option is shown in the "Home page" fieldset 👎🏼 (should have shown up in the previous step)
  4. Change the home page path to home (or blank, or something else) -> notice that the "Number of posts on home page" option is still shown 👎🏼
  5. Save the form -> the "Number of posts on home page" option is gone 👎🏼 (should have been hidden the moment the path was such that it was not applicable any longer)

Actual behavior

You need to save the form for the "Number of posts on home page" option to be shown/hidden.

Expected behavior

The "Number of posts on home page" option is shown/hidden using #states.

Bonus

The help text for the "Number of posts on home page" option is:

The maximum number of posts displayed on overview pages such as the home page.

That text reads odd, and has a few inaccuracies:

  • the content types displayed may be other than just posts
  • this setting actually controls the number of pieces of content shown per page (in the Views pager that is created)
  • "such as the home page"??? ...but this setting here IS for the home page
@klonos klonos self-assigned this Jan 3, 2022
@klonos
Copy link
Member Author

klonos commented Jan 3, 2022

This is D7 carry-over that we cannot remove yet, until terms are also provided by Views (see #145), since this accounts for that as well (the words "overview pages" in the help text seem to be there because of that).

We've contemplated on this with @docwilmot and @jenlampton during the sprint for the 1.21.0 release last night, and decided that we'd be better off if this entire setting was removed from this place, and was instead exposed in the settings for the "Promoted content" block (because we now have #2433).

@indigoxela
Copy link
Member

Isn't that a duplicate of #1628?

@klonos
Copy link
Member Author

klonos commented Jan 3, 2022

Related or a "partial alternative" perhaps, but not duplicate @indigoxela. Lots have changed since 2016, and certain of the things are less problematic or non-issues now. So I would have to revisit/revise #1628 (perhaps even close off?).

@indigoxela
Copy link
Member

So I would have to revisit/revise #1628 (perhaps even close off?).

I'd suggest to close the old one, as it's for sure pretty outdated.

@klonos
Copy link
Member Author

klonos commented Jan 3, 2022

I'd suggest to close the old one...

Thanks. I've done that + added a comment to explain why.

@ghost
Copy link

ghost commented Jan 4, 2022

I previously brought this up here: #3343 (comment)

@klonos
Copy link
Member Author

klonos commented Jan 4, 2022

I just confirmed that the "Number of posts on home page" setting worked in D7 the same way as it does in Backdrop: if the actual number of promoted nodes is bigger than that setting, the it adds a listing with a pager, which allows you to navigate to the rest of the promoted nodes.

PR: backdrop/backdrop#3897

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment