Per your comments in IRC, I've revised the text. Better?
So I deleted a content type that had nodes associated with it to see what would happen. When editing one of those nodes I see:
* Notice: Undefined index: listing_node_form in drupal_retrieve_form() (line 469 of /Users/lisa/web/d7/includes/form.inc).
* Warning: call_user_func_array(): First argument is expected to be a valid callback, 'listing_node_form' was given in drupal_retrieve_form() (line 496 of /Users/lisa/web/d7/includes/form.inc)."
So clearly they cannot delete content types unless they also want to delete or basically forget about editing the nodes also.
In IRC you suggested having an extra line break between the text and the button, which I agree would look better, but this hasn't been done in the other places I looked (such as deleting a taxonomy at admin/structure/taxonomy/2)
Committed to CVS HEAD. I committed lisa's patch in #4 because we shouldn't hardcode line breaks in the t() strings. We can follow up with a spacing patch if necessary. Thanks Lisa and Bojhan.
#1
I cleaned it up a bit, following best practices outlined at [#604342]
Forgot to get a 'before' example screenshot of the text when the content type contains a single post but it's very similar to the plural version.
#2
#3
#4
Per your comments in IRC, I've revised the text. Better?
So I deleted a content type that had nodes associated with it to see what would happen. When editing one of those nodes I see:
So clearly they cannot delete content types unless they also want to delete or basically forget about editing the nodes also.
In IRC you suggested having an extra line break between the text and the button, which I agree would look better, but this hasn't been done in the other places I looked (such as deleting a taxonomy at admin/structure/taxonomy/2)
#5
We might want to add some spacing. A between cannot be undone and the previous sentence, and between the button? Gave a try at it with a patch.
#6
Committed to CVS HEAD. I committed lisa's patch in #4 because we shouldn't hardcode line breaks in the t() strings. We can follow up with a spacing patch if necessary. Thanks Lisa and Bojhan.
#7
Automatically closed -- issue fixed for 2 weeks with no activity.