The views row classes are not lower-cased, which can be a problem if you are using field values like Node: Published, which only displays pre-defined values.

Attached is a simple patch to just strtolower class names, and gives users the option to toggle it on or off.

CommentFileSizeAuthor
#4 views_row_class_lower-1571352-4.patch1.59 KBAnonymous (not verified)
views_strtolower_row_class.patch1.59 KBAnonymous (not verified)

Comments

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Needs review

Changing to needs review.

Status: Needs review » Needs work

The last submitted patch, views_strtolower_row_class.patch, failed testing.

dawehner’s picture

dawehner
Primary language German
commented 
+++ b/plugins/views_plugin_style.incundefined
@@ -171,6 +174,7 @@ class views_plugin_style extends views_plugin {
+      $options['row_class'] = array('default' => TRUE);

this shold probably be row_class_lower => array('default' => FALSE, 'bool' => TRUE);

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs work » Needs review
StatusFileSize
new views_row_class_lower-1571352-4.patch1.59 KB

@dawehner right, I also changed default to FALSE so it doesn't change default behavior for users who already use row classes in their views.

Status: Needs review » Needs work

The last submitted patch, views_row_class_lower-1571352-4.patch, failed testing.