Closed (fixed)
Project:
Drupal core
Version:
8.3.x-dev
Component:
views.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
25 Nov 2016 at 10:22 UTC
Updated:
7 Dec 2016 at 11:10 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #2
hardikpandya commentedComment #3
hardikpandya commentedComment #4
rosinegrean commentedComment #5
rosinegrean commentedIs something else required except for the update of the .yml file for the view ?
Comment #6
ifrikThanks, this works as intended.

The new configuration is only used when the Drupal site is installed. It doesn't change existing sites. I've just checked with dawehner and that's fine because it's easy to change manually on existing sites.
Therefore just the yaml file is fine.
Comment #7
xjmThis seems like a sensible UX improvement -- the illustration in the summary says it all.
I agree that it is okay to just fix this for new installations -- after all, existing installations may have already customized the view.
I noticed
admin/peoplehas a similar problem. Can we fix that view here too?admin/content/filesseems to be in the correct order already.Comment #8
ifrikprics,
Do you want to change that second view as well?
Comment #9
rosinegrean commented@ifrik, yes, i will update the patch tomorrow.
Comment #10
rosinegrean commentedHere we go, patch and the interdiff for it.
Comment #12
rosinegrean commentedI'm not sure why, it passes all tests, but the issue is set in needs work.
Any clue?
I will just add it in needs review again.
Comment #13
ifrikThanks,
both the list of content and the list of users now have the filters in the same order as the columns underneath on a fresh install.
Comment #15
xjmCommitted 28ecd0c and pushed to 8.3.x. Thanks!
Comment #16
xjmComment #17
ifrikThanks for getting this fixed and committed so fast.
It's one of those small things that often don't seem to be worth the bother, but then we can just fix it for lots of users.
Comment #18
tkoleary commentedComment #19
gábor hojtsy