Closed (fixed)
Project:
Mollom
Version:
7.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
29 Jan 2011 at 03:17 UTC
Updated:
24 Apr 2014 at 17:13 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
dries commentedThanks Chris. This makes sense to me.
Comment #2
dave reidHrm, 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.
Comment #3
sunComment #4
dries commentedThe 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.
Comment #5
sunRevised proposal attached.
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.
Comment #6
gábor hojtsyI 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.
Comment #7
sunNot sure whether we want to replace "Manage" with "Configure" though.
Powered by Dreditor.
Comment #8
cbrookins commentedI 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.
Comment #10
cbrookins commented#8: mollom-HEAD.description.6.patch queued for re-testing.
Comment #12
effulgentsia commented#8: mollom-HEAD.description.6.patch queued for re-testing.
Comment #13
JacobSingh commentedI like it. The description uses the word content redundantly, so it sounds awkward, but I don't have a better suggestion. Certainly an improvement.
Comment #14
gábor hojtsyTitle 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?
Comment #15
cbrookins commentedHere is a new patch with a shorter title and a description that removes the redundant text as per #13 and #14
Comment #16
Noyz commented"Mollom spam protection and automated content moderation" sounds like this blocks spam of type Mollom.
Maybe "Mollom: Spam and profanity filtering"
Comment #17
cbrookins commentedNoyz, not sure I agree. Is Terminex Pest Prevention only about Terminex Pests? I think people will get it.
Comment #18
horncologne commentedPersonally, I am down with this as of #15. It's as clear and concise as I can think of right now. Marking rtbc.
Comment #19
dries commentedWow, 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!
Comment #20
sunSorry, 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:
Comment #21
cbrookins commentedSun, 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 .
Comment #22
dries commentedLet's keep the conversation going then -- maybe we can come up with something better. I'll give it some thought too.
Comment #23
gábor hojtsyLooks like needs works then as per @Dries.
Comment #24
sunAttached patch simplifies and shortens to "Mollom content moderation", which resolves the issue.
Comment #25
sunI think this is ready to fly.
Comment #26
dries commentedWorks for me. Feel free to commit.
Comment #27
dries commentedCommitted 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.
Comment #28
sunCommitted 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.