Download & Extend

Remove the "More" area from the bottom of handler configuration

Project:Drupal core
Version:8.x-dev
Component:views_ui.module
Category:task
Priority:normal
Assigned:Unassigned
Status:needs work
Issue tags:BADCamp2012UX, Novice, Usability, VDC

Issue Summary

When configuring a handler like a field, filter, or sort, the bottom of every form has a "More" section.

90% of the time, the only thing in there is "Administrative title", which is anticlimactic, but also seems more important than it is.

Original proposed resolution:
We should move everything out of there, and have a #states-based checkbox to toggle the "Administrative title" textfield.

Current proposed resolution:
(needs described) Moves other things out of more, just leaves admin title there.

Next steps:
next step is to rename that fieldset to say Admin title. (See comment #24)

UI Changes:
Yes, see screenshots in the review in #23

Comments

#1

Screenshot of current behavior.

more-section.png

AttachmentSizeStatusTest resultOperations
more-section.png36.77 KBIgnored: Check issue status.NoneNone

#2

Issue tags:+BADCamp2012UX

#3

Here is a list of places what the more fieldset is used for:

  • General: Administrative title
  • Node edit link field: Checkbox to add the current view as destination after editing the node.
  • Fieldapi field: Select field to choose on which internal value to clicksort.
  • Has new content field: Checkbox to use new comments additional to new nodes.
  • User link field: Advanced settings how to display the anonymous user
  • Contextual filters: Allow multiple values and some more advanced edge case settings
  • Contextual filters for numbers: Exclude the items instead of include
  • Contextual filters for strings: Glossary options
  • Filters: $Key used in the query, for example ?$key=Value

#4

Not sure on what's the action item on this though?
While original description moved to remove the 'more' fieldset @dawehner shows that it is being used in other places.

#5

So ~80% of the time there's one, maybe 2 items in the More section. I still think it's disruptive more than useful.

#6

Agreed, and we can just keep it for the few cases there are many?

#7

The only place in which we currently multiple items are contextual filters. It seems to be that these settings are all pretty much advanced so putting
them in there might make sense. In the issue queue there have been quite some users which haven't been able to find out "how to configure a glossary" so we
might should move these options out into a new fieldset?

I personally would suggest to put in a total different fieldset, to make it clear what can be done there.

For now the admin label is still part of the more fieldset (not sure what to do with it), though if we expose the admin label more visible to the users
we might don't have to add [hidden] to the fields? (just an idea)

AttachmentSizeStatusTest resultOperations
drupal-1831080-7.patch5.23 KBIdlePASSED: [[SimpleTest]]: [MySQL] 50,166 pass(es).View details | Re-test

#8

Yhea, I am +1 on putting it in a total different fieldset - labels that represent the content of fieldsets tend to be much better than "more" or "advanced".

#9

Moved some of the settings into a new fieldset

AttachmentSizeStatusTest resultOperations
need_better_title.png22.94 KBIgnored: Check issue status.NoneNone
drupal-1831080-9.patch8.67 KBIdleFAILED: [[SimpleTest]]: [MySQL] 49,980 pass(es), 1 fail(s), and 0 exception(s).View details | Re-test

#10

Status:active» needs review

bot?

#11

Status:needs review» needs work

The last submitted patch, drupal-1831080-9.patch, failed testing.

#12

Status:needs work» needs review

Rerolled.

AttachmentSizeStatusTest resultOperations
drupal-1831080-12.patch8.67 KBIdlePASSED: [[SimpleTest]]: [MySQL] 55,296 pass(es).View details | Re-test

#13

#12: drupal-1831080-12.patch queued for re-testing.

#14

Status:needs review» reviewed & tested by the community

Lets do this!

#15

#12: drupal-1831080-12.patch queued for re-testing.

#16

Issue tags:+Needs screenshot

What's the "after" screenshot?

#17

Status:reviewed & tested by the community» needs review

I tried to test this on simplytest.me but it didn't behave as I expected, is this bugged or is it simplytest.me?

#18

Status:needs review» needs work

I applied the patch, but still get the 'More' label and content inside that area (see image).

AttachmentSizeStatusTest resultOperations
more-link.png116.16 KBIgnored: Check issue status.NoneNone

#19

If we really want to move out the adminstrative label, then please let's fix #1838010: Remove redundant admin labeling on AreaPluginBase first.

#20

The issue #1838010-38: Remove redundant admin labeling on AreaPluginBase was committed a few days ago.
Given that what new direction should this go in?

#21

Status:needs work» needs review

Yup, @dawehner does this need a reroll?

#22

#12: drupal-1831080-12.patch queued for re-testing.

#23

Status:needs review» needs work

To find out what the patch does, I did the following.
I installed drupal 8 without the patch, went to admin/structure/views and added a view
In the view, I went to Filter Criteria and chose "add"
I chose "Content: Author uid" and clicked Apply
The resulting window had "MORE" as a choice at the bottom; when I clicked on it, the only thing under it was "Administrative Title." This is illustrated in the following screenshot:

MORE_issue_without_patch.png

I then went back to the view window and selected ADVANCED
I went to Contextual Filters and clicked "add"
I chose "Content: body" and clicked Apply
The resulting window had "MORE" at the bottom. When I clicked MORE, "Administrative Title" came up, and so did several other items. These are illustrated in the following screenshot:

more_under_MORE_without_patch.png

I then applied the patch and reinstalled drupal 8.
I repeated the steps to apply the filter criterion "Content: Author uid"
The resulting window still had "MORE" as a choice at the bottom, and when I clicked on it, the only thing under it was still "Administrative Title." This is illustrated in the following screenshot:

with_patch_admin_title_under_more.png

I then repeated the steps to apply the Contextual Filter "Content: body"
The resulting window still had "MORE" as a choice at the bottom. When I clicked on it, the only thing under it was "Administrative Title." The additional items that had been there before the patch was applied were no longer there.

I then went back and saw that a dropdown choice called "STRING SETTINGS" was present. When I clicked on it, the items that had been present under "MORE" before the patch was applied were displayed. This is illustrated in the following screenshot:

with_patch_string_settings_more_issue.png

So as far as I can tell what the patch did was to remove everything except for "Administrative Title" from "MORE" and put them under a more informative heading, but the issue of having the "MORE" dropdown choice with nothing under it except for "Administrative Title" is still present with the patch.

AttachmentSizeStatusTest resultOperations
MORE_issue_without_patch.png114.13 KBIgnored: Check issue status.NoneNone
more_under_MORE_without_patch.png89.61 KBIgnored: Check issue status.NoneNone
with_patch_admin_title_under_more.png71.63 KBIgnored: Check issue status.NoneNone
with_patch_string_settings_more_issue.png81.17 KBIgnored: Check issue status.NoneNone

#24

I'd say the next step is to rename that fieldset to say Admin title?

#25

Issue tags:-Needs screenshot

Thanks @bannockree that write up is excellent. It says what you did in a way that someone else can repeat the steps later if they need to. And those embedded screenshots very well illustrate what is happening before and after the patch.

I took off the needs screenshot tag. We can add it back later if this needs more screenshots.

If the only thing now in that fieldset is going to be admin title, then @Bojhan's suggestion in #24 sounds like enough to get this in.

I'll update the issue summary with the next step.

#26

Is 'Admin title' informative enough? 'Admin' has so many meanings. How about 'Alternative title' instead?

#27

@lisarex Nope, the fieldset should be "Administrative title". There is nothing alternative, it is the title we use for administrative purposes. It is also the convention to label it like this.

#28

Is this all we wanted: to change the name of the fieldset (see interdiff)? or did we want to remove the fieldset altogether?

Screenshot with this patch applied:
Admin Title with patch

AttachmentSizeStatusTest resultOperations
drupal-1831080-28.patch6.04 KBIdleFAILED: [[SimpleTest]]: [MySQL] 56,955 pass(es), 1 fail(s), and 0 exception(s).View details | Re-test
interdiff.txt985 bytesIgnored: Check issue status.NoneNone
admin-title.png92.2 KBIgnored: Check issue status.NoneNone

#29

Status:needs work» needs review

#30

Status:needs review» needs work

The last submitted patch, drupal-1831080-28.patch, failed testing.

#31

We do not want to remove the fieldset, its often not relevant - having it in a collapsed fieldset for that purpose is fine.

#32

Issue tags:+sprint

tagging

#33

Issue tags:-sprint

Having a single element hidden in a fieldset is pointless IMO. Bad UX and waste of space if you ask me.

#34

Issue tags:+sprint

@klonos Its not, this is the exact reason why collapsible fieldsets exist. Its to hide something that is not always relevant, but does offer useful functionality and exposing it would lead to distraction. Could we make it better and find a different location (e.g. redesign the whole admin title interaction)? We could but I think this is a great intermediate step, if you want to push to a great fix please open a new/followup issue - as I think thats more appropriate given the time its taken to get this as far as it is.

#35

...don't know why that tag was removed. Sorry :/

As for my comment, I was speaking in general - it was not meant for this specific use case only. Fieldsets AFAIK are used to group things together. That is their main function. There's no point in grouping a single item by itself. The ability that fieldsets have to be collapsed can be used to hide distracting/confusing things away, but that is a secondary function IMO.

If we want to reduce importance we could simply have a label read "Optional administrative title" or "Administrative title (optional)"

#36

Status:needs work» needs review
Issue tags:-sprint

Let's do it properly.

AttachmentSizeStatusTest resultOperations
drupal-1831080-32.patch7.84 KBIdleFAILED: [[SimpleTest]]: [MySQL] 57,469 pass(es), 77 fail(s), and 57 exception(s).View details | Re-test
interdiff.txt2.58 KBIgnored: Check issue status.NoneNone

#37

Status:needs review» needs work

The last submitted patch, drupal-1831080-32.patch, failed testing.

#38

@klonos I agree its a secondary purpose. We shouldn't put fieldsets around single things, but in this case its the only pattern to make it less important.

#39

Yep, I see that. I was just brainstorming over this. I'm currently taking a look at all the places this issue effects (these mentioned in #3). I'll post here if I come up with something useful/worthwhile.

#40

From the original testing we did, we observed people clicking the More fieldset open, puzzling, then moving on. They were disappointed that it wasn't anything more exciting :) The More fieldset was a detractor from completing the task... collapsed fieldset in and of itself didn't reduce importance because the label was so mysterious.

However, a collapsed fieldset with a clear label will at least mean people can skim past without worrying they're missing out, so that's good.

What about the other options in #3? What fieldset label will they be under? I'm not clear from reading these comments over and over again.

#41

I think most of them are moved under a specialised fieldset called "Filter identifier"