A: How can I view all my contents ?
me: Views "Find Content"
A: Okay. But I don't remember my content title, can I have a full content list?
me: Yeah. "Find Content" lists all your contents.
....
..

Until you click into "Find Content" page, you might think that's a search page.
content1.png
(unfortunately if you really going to search something, there hasn't any search feature on the page)

Check out other websites:
Gmail:
gmail.png

WordPress:
wordpress.png

"All contents" is more intuition.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

webchick’s picture

Also hilarious is the fact that that screen has no search box, so it's impossible to "find" anything. :)

Fixing tag.

ditcheva’s picture

I have never helped with any Drupal core fixes or patches yet, and this is me wanting to start! This seems like an appropriately small patch to get me started. :-)

I hope I did this correctly and that I patched the correct latest version of Drupal 8 dev for this issue. If I haven't, please let me know how I should have done it, so I can learn and contribute more...

For this suggestion, the text replacement had to occur both in the standard install profile and in one of the test files of the shortcut module....

Bojhan’s picture

Yup, sounds good.

droplet’s picture

Issue tags: -Usability

#2: content-list-rename-1809590.patch queued for re-testing.

Status: Needs review » Needs work
Issue tags: +Usability

The last submitted patch, content-list-rename-1809590.patch, failed testing.

ditcheva’s picture

Any idea why the patch keeps failing? I'd be happy to fix it, but can't figure out what the issue is by looking at the log. Is it because the version of Drupal 8 dev I applied this to has changed by now?

droplet’s picture

@ditcheva, the patch doesn't apply to current DEV version,

git apply content-list-rename-1809590.patch
error: patch failed: core/modules/shortcut/lib/Drupal/shortcut/Tests/ShortcutTestBase.php:59
error: core/modules/shortcut/lib/Drupal/shortcut/Tests/ShortcutTestBase.php: patch does not apply
error: patch failed: core/profiles/standard/standard.install:410
error: core/profiles/standard/standard.install: patch does not apply

A quick reroll is find. Anyway, code looks good.

webchick’s picture

A screenshot of the "after" would be useful, too. I actually am a little worried that "Add content" and "All content" right next to each other might look too similar and cause confusion. :\

ditcheva’s picture

OK, here goes!

Here's both a revised patch and some screenshots. I created this patch against the latest 8.x branch from here (http://drupal.org/project/drupal/git-instructions). I hope I got that right. If I didn't I hope one of you would be able to explain that part (mostly so I can do it correctly for future patches, because I really do want to start helping more).

Re: the screenshots. Personally, I don't think I find the new look confusing. I do still think 'All content' is more accurate since the page is a listing of the entire site's content with filters on top.

Screenshots

Content links (before)
Content links (before)

Content links (after)
Content links (after)

Content links - selected (before)
Content links (before)

Content links - selected (after)
Content links (after)

webchick’s picture

Status: Needs work » Needs review

That's perfect, ditcheva, thanks! Marking needs review so testbot will test it.

Yeah looking at it, that doesn't seem too bad. People will probably learn quickly that the one on the far left is for adding, and the one in the middle is for listing.

Bojhan’s picture

We can rename "Add content" to "Compose", hah :D

webchick’s picture

Ha. :D Or old school "Create content." ;)

Anyway, let's defer that bikeshed to a follow-up (if at all), as long as people don't feel this is too confusing.

droplet’s picture

Issue tags: -Usability

#9: content-list-rename-1809590.patch queued for re-testing.

Status: Needs review » Needs work
Issue tags: +Usability

The last submitted patch, content-list-rename-1809590.patch, failed testing.

yoroy’s picture

Issue tags: +Novice

I think this makes a lot of sense. Should be relatively easy to fix the patch?

franskuipers’s picture

Status: Needs work » Needs review
FileSize
1.14 KB

Rerolled #9

RobLoach’s picture

Status: Needs review » Reviewed & tested by the community

Grepped for "Find content". Those are the only two instances.

MustangGB’s picture

It always annoyed me that in some places there would be "add" and others "create", I don't mind which one it is, but just pick one and stick to it.

xjm’s picture

xjm’s picture

Yeah, I think this is a nice improvement. It'll help me. :)

xjm’s picture

Issue tags: +Quick fix
xjm’s picture

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 3e3f16e and pushed to 8.x. Thanks

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

tkoleary’s picture

Status: Closed (fixed) » Needs review

This needs a usability test to validate if it improved anything.

Also it seems to me that the critical user need that "find content" was intended to address (users want to be able to find content) got missed. Perhaps it would have been a better idea to actually provide a search box...

xjm’s picture

Issue summary: View changes
Status: Needs review » Closed (fixed)

@tkoleary, can you please open a dedicated followup issue for that task and link it here? The patch was already committed and reusing issues causes a lot of confusion.