Comments

Stefan Nagtegaal’s picture

StatusFileSize
new27.97 KB

Screenshot of admin/comment...

Stefan Nagtegaal’s picture

StatusFileSize
new543.56 KB

Attached screenshot of admin/user..

You can also see in these 3 screenshots how inconsistent the UI is.. (The 'Status'-columns are on all of the screenshots on another place.)
But, that has to be another patch..

Stefan Nagtegaal’s picture

StatusFileSize
new551 bytes

Attached chackmark.png. Should be copied to 'misc/'.

Stefan Nagtegaal’s picture

StatusFileSize
new509 bytes

Attached error.png. Should be copied to 'misc/'..

Stefan Nagtegaal’s picture

StatusFileSize
new3.26 KB

And - finally - the patch... =)

Stefan Nagtegaal’s picture

Status: Active » Needs review
StatusFileSize
new3.26 KB

And - finally - the patch... =)

Stefan Nagtegaal’s picture

StatusFileSize
new40.94 KB

Reuploading screenshot of admin/user.. Some people could open the previous one, which was likely a fault on my site...

Enjoy the screenshots!!!

Kobus’s picture

I am sorry to say, but this is not very good (well done, but not effective...). For me, "Published" and "Active" is not equal to "checkmark". I see checkmark. What does it mean? Same with "Not published" and "Blocked". The red X will make users panic. What is wrong with this thing? Why this red X on my screen?

Bèr Kessels’s picture

I have to admit that I am with kobus in this. I really think teh work you did is very nice. But I think replacing information with icons is a bad thing. Icons can be used next to that information, o enhance it, not replace it.

Bèr

moshe weitzman’s picture

i like this sort of candy. if user hates images, he shuts them off and sees the alt text. no problem. also, less of a translation burden.

dries’s picture

I like these too (if done right).

Stefan Nagtegaal’s picture

Version: 4.6.2 » x.y.z

Changing version to CVS because it could maybe slip in CVS.

Dries, Moshe: What do you guys suggest on where to use the icons?

You name it, I'll patch it..

Bèr Kessels’s picture

What about moving these into the admin thelme occy is worknig on? There is load of niftyness in there, that could be considered to be ported to core.

rkerr’s picture

Do these follow the guidelines for Use of icons? :)

Instead of using x/checkmark, how about using an icon of a page.. and for published overlay a globe on it, and for unpublished overlay the red circle with a slash through it?

drumm’s picture

Status: Needs review » Needs work

Lullabot made some icons, http://www.lullabot.com/articles/free_gpl_icons_lullacons_pack_1. I'd like to see the icons in that style.

I'm not sure how much published/not published and active/blocked maps to check/x.

Personally, I think the overall goal is to make the tables smaller and less intimidating for fitting into page layouts and making them more usable. There are currently two options for each status, and one of them is usually much more common than the other. My idea was to remove the whole status icon and add '(not published)' after those posts and '(blocked)' after those users.

coreb’s picture

Version: x.y.z » 6.x-dev
Category: task » feature
Status: Needs work » Patch (to be ported)

moving from x.y.z to 6.x-dev version queue

Although I don't know the difference between a task and feature request, this appears to be a feature request to me. But there may be a fine line that I'm missing about the difference between the two.

Looking at the last patch being submitted over a year ago, it's almost safe to say that it needs to be ported to be compliant with a newer version.

coreb’s picture

Status: Patch (to be ported) » Needs work

I incorrectly used the new "patch (to be ported)" status.

catch’s picture

tasks are usually:
1. feature requests with patches attached ;)
2. usability fixes that aren't strictly bugs.

This patch is very old now, worth revisiting for D6?

pancho’s picture

Version: 6.x-dev » 7.x-dev

Too late to revisit this... moving to D7.

yoroy’s picture

Issue tags: +icons

tagging

Status: Needs work » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.