Setting checkbox should be under "Comment settings"

arhak - September 14, 2008 - 15:28
Project:Talk
Version:6.x-1.5
Component:User interface
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed
Description

setting checkbox should be under "Comment settings" instead of "Workflow settings" group

#1

KiamLaLuno - January 13, 2009 - 20:07
Title:setting checkbox should be under "Comment settings" instead of "Workflow settings" group» Setting checkbox should be under "Comment settings"
Version:6.x-1.4» 6.x-1.5

I am updating the version to the last available one, that still doesn't place the settings in the "Comment settings" fieldset.

#2

cwgordon7 - April 10, 2009 - 16:04
Status:active» needs review

Here's a patch, please let me know if it works for you.

#3

cwgordon7 - April 10, 2009 - 16:04

Er. Real patch this time.

AttachmentSize
talk_move_settings.patch 1.67 KB

#4

GregoryHeller - May 21, 2009 - 16:52

This would be great to have in the next release, the option is pretty well hidden in the workflow settings section

#5

Todd Nienkerk - June 30, 2009 - 21:40

+1. Also, the weight should be adjusted to ensure that option is placed at the top of that fieldset.

Question regarding the comment form: Perhaps this module should automatically move the commenting form to a separate page and leave a note in the content type editing form explaining this? Just a thought.

#6

greggles - July 6, 2009 - 01:52
Status:needs review» reviewed & tested by the community

I'll give this a RTBC.

I disagree on pushing it to the top of the form, though. It's not the most important thing and pushing down other items which people are used to seeing in that area could be jarring. New things tend to show at the bottom.

#7

greggles - July 14, 2009 - 17:10
Status:reviewed & tested by the community» fixed

Committed to 6.x: http://drupal.org/cvs?commit=237338

If someone wants to roll this for 5.x I would be happy to commit.

#8

System Message - July 28, 2009 - 17:20
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.