Hi,

I’ve spent a few hours trying to get the theme to validate on W3C and had to fix the following two bugs in your core php:

• setting multiple IDs instead of class name in vc_templates/vc_row.php (line 296) :

faulty: echo ‘<div ‘.( (!empty($class))?’id=”‘.$class.'”‘:”).’ style=”position: absolute;top: 0;”></div>’;

corrected:  echo ‘<div ‘.( (!empty($class))?’class=”‘.$class.'”‘:”).’ style=”position: absolute;top: 0;”></div>’;

• setting padding even if no custom padding was set resulting in a “padding:” validation error in vc_templates/vc_column.php (lines 71-74)

faulty: $extra_style .= ‘padding:’.$column_padding;

corrected: if($column_padding != ”) $extra_style .= ‘padding:’.$column_padding.”;”;

Unfortunately, there is one bug I can’t fix – as replacing the faulty code does not work in this case

• setting letter-spacing value without metrics in css/dynamic_styles.css

faulty: <span style=”color: #000000; font-family: monospace; font-size: medium; white-space: pre-wrap;”>.woocommerce-page ul.products li.product h6{text-transform:uppercase;letter-spacing:1.5;</span>

Which is weird because we do not use any woocommerce, and since it is dynamically generated I can’t fix this. Maybe you can. Or add the option to unload bulks of unnecessary css in the theme options, e.g. woocommerce, animation stuff, etc… for basic implementations of your theme.

• This topic was modified 5 years, 1 month ago by Monika.