This goes in pair with #1066830: Change /node path to /latest and allow admins to disable. and #1210366: Per-bundle node listing pages, blocks, feeds..
I realize that novice users coming to the views admin interface can easily spot a view labeled 'Frontpage' and they surely grasp what it's meant for, but once people change either their default home page setting or the path of that view, then it simply makes no sense to still call it 'Frontpage' and can in fact cause confusion (#987242: The "Promoted to front page" checkbox doesn't do anything if the /node front page listing isn't used).
In short: It makes a lot more sense to have a 'Latest content' or 'Featured content' view and set it as the site's home page.
Comments
Comment #1
klonos...also, please see my proposal in #1201592-57: Front page settings cleanup
Comment #2
klonos...a better alternative to "Latest content" would be "Featured content".
Comment #2.0
klonos...adding #987242
Comment #3
dawehnerNoone will find issues, especially if they are not tagged as VDC.
Comment #3.0
dawehner...adding alternative 'featured' instead of 'latest'.
Comment #4
justinchev commentedIn the Beta 1 release I see that this sort of content is now referred to as 'Promoted'. Since that is what it is now called I suggest that we use plain 'Promoted' for the view name instead of 'Frontpage' (as it's currently set).
This will also need to be set in a couple of other places. An example is when editing the view and choosing the type of content to show - it's still using the label 'Promoted to front page status'.
* edit: I started going through the code to see what would need to be changed out. Found several of the 'UI' bits of text as seen from the admin pages, but having done a search for 'frontpage' through all files I see that there are quite a few uses of it within the site code in a view machine name/function name type of scenarios. In an ideal world all references to the Frontpage view should be changed.
Is this something to progress or something which could have an impact on things such as contrib mods?
Comment #5
justinchev commentedI went ahead and changed all instances I could find where it should be referred to as 'Promoted' rather than 'Frontpage'. This mainly involved UI labelling and descriptions as well as renaming the 'frontpage' view along with it's the machine name and test references.
I did leave references to 'Front page' when it was actually referring to the actual Frontpage.
Below are some screenshots of some of the UI changes...
Comment #7
justinchev commentedAhhh forgot to change the version this patch goes against...
Comment #8
justinchev commentedI'm really making a mess of this... once more with status set back to 'needs review' :s
Comment #13
justinchev commentedAnd again, hopefully with a fix for the one thing failing testing...
Comment #14
justinchev commentedI've updated the patch so that it applies successfully to Drupal 8.0.0 beta 2.
Comment #18
justinchev commentedSeems the latest patch is failing on one of the Language tests ('NodeLanguageTest.php').
I don't really understand why but it seems the test is trying to override the config for the Promoted view to test language filters. What I don't understand is that it appears to be targeting a view named promoted ( $config = \Drupal::config('views.view.promoted');)
This confuses me as the patch I'm providing is changing the view from it's current name of 'frontpage' to 'promoted' so wouldn't have thought there would be a test targeting it.
My dev knowledge is fairly limited so I could be completely missing something obvious. Any ideas as I'd like to get this patch passing again?
Comment #19
justinchev commentedPatch updated so it applies to Drupal 8-beta 3.
Comment #21
justinchev commentedPatch rerolled to pass tests - it should apply to Drupal 8-beta 3.
See screenshots in #5 for some of the changes.
Comment #23
justinchev commentedPatch rerolled to pass tests - it should apply to Drupal 8-beta 3.
See screenshots in #5 for some of the changes.
Comment #25
dawehnerLet's use the right version number here.
Comment #26
tadityar commentedre-rolled, fixed some capitalisation and used
-Cinstead the usual diff.Comment #41
lendudeWith the coming of Drupal CMS we should just nuke this View I recon. Along with some other Views that ship with core.