Closed (fixed)
Project:
Re: Comment subjects
Version:
6.x-2.0-beta1
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
18 Sep 2010 at 08:27 UTC
Updated:
7 Dec 2010 at 00:10 UTC
This is useful for flat (not threaded) lists of comments.
Note: The comments modules offers an option to disable or enable submission comment titles? However, if that option is set to "Disabled", titles are automatically extracted from new comments and next still displayed. :-(
Comments
Comment #1
arhak commentedbut if you use the 6.x-2.x branch of this module you have the chance to get rid of subjects
- edit your content type
- change the "Default subject pattern" to be empty/blank (it is located under Comment settings fieldset)
- disable the "Comment subject field"
and you're done
Comment #2
physiotek commentedusing this strategy within 6.x.2 is ok to hide the comment title but the problem is that the title is not just hidden but nonexistant.
this cause problems using a block showing titles. the ideal would be to still have the re: title but with the option to hide it in the node template. yes i could edit the template but actually i have a need to hide it from a core template in a multisite drupal install. so editing the core template is not an option for me... any ideas? thanks,
pht3k
Comment #3
arhak commentedmodify your theme's style.css
this will work for Garland (might not work for another themes, e.g: Atrium, because it depends on the markup used in the template)
Comment #4
not_Dries_Buytaert commentedHiding elements using CSS is always possible, though it:
a) causes unnecessary overhead and
b) is less secure (however small the consequences of this maybe in certain use-cases).
Agree that the solution should also work for multisites (without the need to alter files, like css-files).
So, we need following (additional) optional values for the "Default subject pattern"-field:
1) "do NOT allow inputing titles and do NOT output titles"
3) "do NOT allow inputing titles, DO output"
2) "DO allow inputing titles and do NOT output titles"
4) "DO allow inputing titles, DO output"
Comment #5
arhak commented@#4 it seems too much..
how to display such controls without overwhelm?
note that the 90+% of the cases won't need that complexity,
majority shouldn't be affected due to minorities' needs
I'll have to consider some elegant alternative,
like some not-so-obvious advanced controls, to avoid confusing average users
Comment #6
arhak commentedBTW, what do you have in mind for this?
it has to be hid via CSS, but themes are different (for instance Open Atrium themes)
so @#3 will work for Garland and others, but can't ensure it would work with every theme
then
1) is answered @#1
2) CSS in your theme (not in this module)
3) similar to #1 but don't let the pattern empty
4) usual case with subject enabled
so? anything else?