UI text on admin/structure/types/manage/[content type]/delete could be easier to understand.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | uitext_614358_4.patch | 1.27 KB | Bojhan |
| #4 | uitext_614358_4.patch | 1.25 KB | lisarex |
| #4 | afterDeleteContentTypePlural2.png | 19.52 KB | lisarex |
| #4 | afterDeleteContentTypeSingular2.png | 19.28 KB | lisarex |
| #1 | beforeDeleteContentTypePlural.png | 20.57 KB | lisarex |
Comments
Comment #1
lisarex commentedI cleaned it up a bit, following best practices outlined at #604342: Interface text
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.
Comment #2
lisarex commentedComment #3
lisarex commentedComment #4
lisarex commentedPer 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)
Comment #5
Bojhan commentedWe 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.
Comment #6
dries commentedCommitted 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.