Closed (outdated)
Project:
Drupal core
Version:
9.4.x-dev
Component:
node system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
18 Aug 2009 at 17:22 UTC
Updated:
9 Dec 2022 at 01:19 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
rickvug commentedMoving to field_ui component now that CCK UI is in core.
Comment #2
lisarex commentedWill have a look today.
Comment #3
dman commentedIs this related to #560330: Make Field UI description more user-friendly ?
Comment #4
lisarex commentedThere are several updates in this patch that tidy, clarify and correct some of the UI text in here. Screenshots are attached for all changes apart from the first two which are simply capitalizing the word 'Add'.
dman: Thanks for checking. There's no overlap with #560330. This patch is for the UI text.
I just noticed the field settings page could use some love so will make a separate issue for that.
Comment #5
heather commentedMew fields! I think your cat wrote this patch. ;)
I didn't install and run this patch. Let's see the test-bot try it out.
I think you might have better luck submitting individual patches for each section? I'm not sure. I tried something like this before and it sort of languished.
This review is powered by Dreditor.
Comment #6
lisarex commentedSpotted a typo (mew should not appear!) so use this patch instead (which matches the screenshot), thanks.
Comment #7
lisarex commentedAh, you got me :) I can do indiv. patches if that'll help --- will they need to be in separate issues though?
Comment #8
heather commentedI don't know, I do know webchick told me about "killing kittens" when you have a patch that affects many different components.
And I know how much you love your cat, and I do too. So... not sure if this is a kills kittens thing. I do know there may be existing issues out there which could be duplicated in this uber-patch.
#202533: Improve help texts in "add content type" is related maybe? is it a duplicate?
{edit: Bojhan confirms, this is too much to review at once, so best to break up into component issues, then link to each from here for posterity}
Comment #9
lisarex commentedCool, I have broken this down into smaller issues/patches:
#614316: improvements to body field label and submission guidelines
#614324: Node UI text improvements to author display settings
#614358: UI text improvements to 'Delete' content type confirmation message
#614396: Clarify UI text on page for adding new content types
Comment #10
yched commentedNone of the changes here are related to Field UI.
Also, note that the 'Body label' field will be gone from the 'edit content type' screen when we're done with #553306: Make nodes have no body field by default. Remove deprecated APIs for body field.
Comment #11
yched commentedComment #12
lisarex commentedThis can probably be made inactive now (see comment 9) unless there's further work to be done around this issue.
Comment #14
Bojhan commentedPushing this to Drupal 8, we might need to rewrite parts given on how the UI will evolve then and any testing we do on Drupal 7 but given the text-freeze it doesn't seem this meta issue should stay for 7.
Comment #15
xjm(Merging "node system" and "node.module" components for 8.x; disregard.)
Comment #27
quietone commentedLooking at the patch some of the changes are now in core. I tracked one to #2528340: Improve description text for 'Content types' item on the Structure page.
I think it is time to close this.