I have an example where my parent mlid is 'main-menu:48'. This is not a valid CSS class name and in template_preprocess_menu_block_wrapper this is put straight in.

I propose that the ':' should be replaced with a '-' to make it a valid CSS identifier.

Files: 
CommentFileSizeAuthor
#2 1977108-invalid_class_names-2.patch703 bytesothermachines

Comments

Status:Active» Needs review
StatusFileSize
new852 bytes

This patch addresses the issue by replacing any occurance of ':' with a '-' for the class name only.

Issue summary:View changes
StatusFileSize
new703 bytes

This makes sense to me. Since the ':' renders the class name invalid, I can't see how making this change would cause any conflicts. Here's a re-roll with project-relative paths.