Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
This is similar to #806232: Site Information Improvements which is marked as fixed, but the issues still remain.
It would be nice if we could find a true home for these items - removing them from the site information page. I couldn't find another issue that moved these things back on to the site information page, but I do like the idea of adding a "Front page settings" option under structure and would love to revisit the idea in D8.
Comment | File | Size | Author |
---|---|---|---|
#21 | move_front_page_error-1201580-21.patch | 16.02 KB | delzhand |
#20 | move_front_page_error-1201580-20.patch | 15.96 KB | delzhand |
#16 | move_front_page_error-1201580-16.patch | 12.49 KB | delzhand |
#15 | drupal-move_error_page_settings-1201580-15.patch | 9.63 KB | delzhand |
#6 | front-error-pages-settings.1201580.006.patch | 6.39 KB | karschsp |
Comments
Comment #1
karschsp CreditAttribution: karschsp commentedShould we change "Front page" to "Homepage"? "Front page" is so, I don't know, Microsoft circa 1999? I think people would maybe understand "Homepage" better, but I could be wrong.
I could see admin/structure/homepage as a possibility, but would that adequately describe the 403/404 settings?
Comment #2
jenlamptonNo, I think 403/404 don't belong in site information or home page settings. Perhaps a place of their own?
Comment #3
karschsp CreditAttribution: karschsp commentedHow about a new section called Home and Error Pages.
Or Front and Error Pages?
Comment #4
karschsp CreditAttribution: karschsp commentedOK, renaming "Front page" to "Home page" should probably be a separate issue. Here's a patch that creates a "Front and Error pages" menu item under admin/config/system.
Comment #5
karschsp CreditAttribution: karschsp commentedYeah, that's gonna fail. /me scrambles to write some tests.
Comment #6
karschsp CreditAttribution: karschsp commentedHere's a patch with updated tests.
Comment #7
lambic CreditAttribution: lambic commentedShould there also be something on the node edit page? Something like "Set this node as the home page"?
Comment #8
karschsp CreditAttribution: karschsp commented@lambic, I think that would just confuse things. What if you marked 2 (or more) nodes as "Set this node as the home page"? Which one wins? And then how do you track down the others that you no longer want to be marked as the home page?
Comment #9
lambic CreditAttribution: lambic commented@karschsp - When I said "node edit page" I meant when editing a single node. I think setting the node currently being edited/created as the home page would immediate override whatever node was set to the home page before (probably with some confirmation page saying "Node X is already set as the home page, are you sure you want to replace it with node Y?")
However, my suggestion is just an added convenience for the average user. We would still need the settings page for situations where the front page isn't actually a node.
Comment #10
karschsp CreditAttribution: karschsp commented#6: front-error-pages-settings.1201580.006.patch queued for re-testing.
Comment #12
jenlamptonAlso see #1201592: Front page settings cleanup for more cleanup being done on front page settings.
Comment #13
klonosGiving this a friendly bump. We seemed to have Dries' consent in #806232: Site Information Improvements about this needing to happen.
Comment #14
delzhand CreditAttribution: delzhand commentedComment #15
delzhand CreditAttribution: delzhand commentedPatch created for this. Creates a new router item and form class. I suspect there are tests to be updated as well.
Comment #16
delzhand CreditAttribution: delzhand commentedSorry, missed the front page setting. I also searched for AccessDeniedTest and couldn't find it - it's probably been renamed since. If the tests fail it should point me in the right direction, though.
Comment #17
delzhand CreditAttribution: delzhand commentedComment #20
delzhand CreditAttribution: delzhand commentedUpdated with changes to tests.
Comment #21
delzhand CreditAttribution: delzhand commentedRan the tests locally before testbot got around to it and found another failure point.
Comment #23
delzhand CreditAttribution: delzhand commented21: move_front_page_error-1201580-21.patch queued for re-testing.
Comment #25
dawehner