This patch is similar to #369460: Add "No books available." message to admin book list in that it adds the "No ____ available." message to the table for contact form categories found at admin/build/contact. It also includes some cleanup for much needed whitespace and some code comments. (Again, not sure how verbose comments ought to be in core; I tend to comment liberally, as evidenced by this patch.)

CommentFileSizeAuthor
no_categories_available.patch1.99 KBrszrama

Comments

webchick’s picture

Status: Needs review » Needs work
Issue tags: -Quick fix

Actually, this should be using the new tableselect form element it looks like #42962: Disable personal themes?. Would you be able to make the switch?

rszrama’s picture

Ahh, certainly not a quick fix any more, eh? In that case, neither this one (or the book admin page) are using forms, nor do they have bulk operations. Are you saying you'd want these converted into form functions?

webchick’s picture

Status: Needs work » Fixed
Issue tags: +Quick fix

Oh, hm. I didn't notice that at first blush. Never mind! :D This is why I should not review patches while on the phone. ;)

Committed to HEAD. :) Thanks!

Status: Fixed » Closed (fixed)
Issue tags: -Quick fix, -ui, -code cleanup

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