Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
node system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
11 Mar 2009 at 04:38 UTC
Updated:
25 May 2010 at 00:52 UTC
Jump to comment: Most recent file
Comments
Comment #1
mitchell commentedtagging for usability team
Comment #2
dmitrig01 commentedComment #3
karschsp commentedHere's a patch for this. I changed the text to "Publishing settings" to keep consistent with the other fieldsets on that form (Submission form settings, display settings, etc).
Comment #4
karschsp commentedChanging status to CNR
Comment #5
moshe weitzman commentedOh yes - good one.
Comment #6
mitchell commentedAgree, RTBC.
(This idea came from the Rules BoF discussion group @ DrupalCON.)
Comment #7
webchickSince on the create content form this is called "Publishing options," should we should probably make the fieldset on the content types page consistent.
Comment #8
karschsp commentedgood point. here's a patch (398068.002.patch) with the text on the "Content Types" page set to "Publishing Options"
although now that I look at the node/add form, maybe it should be "Publishing settings" there as well? We have Revision Information, Comment Settings, URL path settings, Author Information and Publishing Options. Options is the only one that doesn't fit...
So I'm also attaching another patch (398068.003.patch) that changes it to Publishing Settings everywhere. I feel like that makes more sense, but I don't know really know...
Comment #9
karschsp commentedchanging status to needs review.
Comment #10
cburschkaI like the second idea better due to the consistency argument. But they both look good.
Comment #11
Bojhan commentedSounds good, indeed RTBC.
Comment #12
Bojhan commentedtag
Comment #13
sunTo get this committed, please create a separate issue requesting that $form['options'] needs to be renamed.
'options' was a weird form array key before already. Now it's completely off-the-wall, because not even the fieldset title matches 'options'.
Comment #14
dries commentedCommitted 398068.002.patch from #8. Thanks!
Marking this 'code needs work' so we can continue our conversation about naming.
Comment #15
sun