Changed the mollom configuration page to

Spam and profanity filters
Automatically manage undesirable visitor submissions using Mollom.

See attached patch

Comments

dries’s picture

Thanks Chris. This makes sense to me.

dave reid’s picture

Hrm, there aren't any other modules that could provide spam or profanity filters? If so, using an ambiguous title like that may be a problem. I kinda disagree about not using 'Mollom' as the title.

sun’s picture

Status: Needs review » Needs work
  1. The proposed change is too tailored for a specific distribution, in which most users won't ever consider or install a different module for this kind of functionality.
  2. The proposed wording is too limited with regard to Mollom's current and upcoming features.
  3. We need to keep "Mollom" in the title. We can optionally append descriptive terms; e.g., "Mollom content moderation".
  4. The direction of the revised description makes sense, but I think the wording can be improved.
dries’s picture

The current title, which is just 'Mollom' is not very descriptive. Most end-users might not know what Mollom is. The proposed title is better with end-users in mind. What is the rationale for keeping 'Mollom' in the title?

If we insist on keeping Mollom in the title, maybe we can go with 'Mollom spam and profanity filters' instead of 'Spam and profanity filters'? Just a thought.

sun’s picture

Title: Improved module description text and link » Improve module description text and link
Category: bug » task
Status: Needs work » Needs review
StatusFileSize
new1.35 KB

Revised proposal attached.

What is the rationale for keeping 'Mollom' in the title?

We're simply not there yet. Removing module names from the administration UI might work for certain Drupal distributions that are intended for a certain audience, in which site administrators won't even attempt to download and install other/third-party modules, so the distribution can tweak the entire administration UI in a way that assumes there can only be one module for a particular functionality.

However, regular Drupal users do install Spam module + CAPTCHA module + Mollom module + others at the same time. To try them, and to perhaps keep them. Removing the project/module name means that you have multiple links that state Spam protection, Spam prevention, Spam and profanity filtering, etc.

To some extent, regular Drupal users also expect modules to expose themselves with their module name, because the users know that they are working with a modular system.

gábor hojtsy’s picture

I think there are two competing things here:

(a) many sites are built and handed over to clients - clear menu items help if you do not hand over a complete manual too, so people know what Mollom is
(b) Mollom does not yet have the name recognition like Google Analytics does for example, so people recognize what it is just by its name

I think the new proposed patch is a good middle ground to try and support those running multiple modules as well as those who just get their site.

sun’s picture

+++ mollom.module	29 Jan 2011 23:17:41 -0000
@@ -191,8 +191,8 @@ function mollom_menu() {
+    'description' => 'Manage automated moderation of user-submitted content.',

Not sure whether we want to replace "Manage" with "Configure" though.

Powered by Dreditor.

cbrookins’s picture

StatusFileSize
new1.09 KB

I agree with some of the points made, but the solution in #5 is so generic that it obfuscates the 80% use-case of the Mollom module. This is an attempt to add the Mollom brand, explain Mollom's larger purpose AND make it clear what Mollom's 80% use case is so new users aren't left scratching their head about WTF content moderation means.

Status: Needs review » Needs work

The last submitted patch, mollom-HEAD.description.6.patch, failed testing.

cbrookins’s picture

Status: Needs work » Needs review

#8: mollom-HEAD.description.6.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, mollom-HEAD.description.6.patch, failed testing.

effulgentsia’s picture

Status: Needs work » Needs review

#8: mollom-HEAD.description.6.patch queued for re-testing.

JacobSingh’s picture

I like it. The description uses the word content redundantly, so it sounds awkward, but I don't have a better suggestion. Certainly an improvement.

gábor hojtsy’s picture

Title will look a bit unusual (just look at a screenshot with the other admin items), given other modules rarely use so long titles. The description is supposed to elaborate. But otherwise looks good. Can we somehow shorten the title?

cbrookins’s picture

StatusFileSize
new1.05 KB

Here is a new patch with a shorter title and a description that removes the redundant text as per #13 and #14

Noyz’s picture

"Mollom spam protection and automated content moderation" sounds like this blocks spam of type Mollom.

Maybe "Mollom: Spam and profanity filtering"

cbrookins’s picture

Noyz, not sure I agree. Is Terminex Pest Prevention only about Terminex Pests? I think people will get it.

horncologne’s picture

Status: Needs review » Reviewed & tested by the community

Personally, I am down with this as of #15. It's as clear and concise as I can think of right now. Marking rtbc.

dries’s picture

Status: Reviewed & tested by the community » Fixed

Wow, lot's of activity for a small patch. :-)

I'm happy with #15 - it is an improvement over the current so I committed it to HEAD.

I don't think we need to backport the changes.

Thanks!

sun’s picture

Status: Fixed » Needs work
StatusFileSize
new9.84 KB
new47.39 KB

Sorry, but I'm not happy with this. The link title is too long/wordy. It looks like this change wasn't actually tested and visually reviewed:

mollom-admin-link.png

mollom-admin-link2.png

cbrookins’s picture

Status: Needs work » Fixed

Sun, seems like an issue for the Admin menu's issue queue that they shouldn't wrap module names so poorly. Can you open an issue there? This looks fine in core as per your mollom-admin-link2.png .

dries’s picture

Let's keep the conversation going then -- maybe we can come up with something better. I'll give it some thought too.

gábor hojtsy’s picture

Status: Fixed » Needs work

Looks like needs works then as per @Dries.

sun’s picture

Status: Needs work » Needs review
StatusFileSize
new548 bytes

Attached patch simplifies and shortens to "Mollom content moderation", which resolves the issue.

sun’s picture

Status: Needs review » Reviewed & tested by the community

I think this is ready to fly.

dries’s picture

Works for me. Feel free to commit.

dries’s picture

Status: Reviewed & tested by the community » Patch (to be ported)

Committed to 7.x. Not sure if this needs to be committed to other branches -- 6.x, 7.x working branches or 8.x. Moving to 'to be ported' so we can double check what needs to happen.

sun’s picture

Status: Patch (to be ported) » Fixed

Committed a cumulative backport to 6.x-1.x.

Note: The last patch was committed to 7.x-rest only, not to master (7.x-1.x). I've committed it to master, too.

Status: Fixed » Closed (fixed)

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

  • Commit 102e552 on master, fai6, 8.x-2.x, fbajs, actions by sun:
    - Patch #1044262 by sun: one more tweak to the Mollom module description...
  • Commit eb5cec0 on master, fai6, 8.x-2.x, fbajs, actions by Dries:
    - Patch #1044262 by cbrookins, sun: improve module description text and...

  • Commit 102e552 on master, fai6, 8.x-2.x, fbajs, actions by sun:
    - Patch #1044262 by sun: one more tweak to the Mollom module description...
  • Commit eb5cec0 on master, fai6, 8.x-2.x, fbajs, actions by Dries:
    - Patch #1044262 by cbrookins, sun: improve module description text and...