re-check codestyle some places have FALSE and TRUE uppercase style
also theme version 1.0 but in php classes sometimes @SInCE 1.1, can be confused in future
in many places mixed YODA condition comparison
if ( get_theme_mod( 'vc_header_sandwich_style', '#333333 ' ) == '#FFFFFF ' ) {
if ( 'div' === $args['style'] ) {
better to consider same style for comparison
always use === instead of ==
missed space in 'footer-'.$i and other operators sometimes have missed space (please use PHPCS)
Follow code style https://codex.wordpress.org/WordPress_Coding_Standards
Right after activation I'm seeing this error on frontend: Strict Standards: Non-static method VC_Fonts::vc_theme_google_font_variants() should not be called statically in /Users/mmihey/Sites/test/wp-content/themes/visual-composer-theme/inc/customizer/vc-fonts.class.php on line 113
Please Scope all JavaScript files to localscope
inc/customizer/js/control-toggle-switch.js
inc/customizer/js/control-image-select.js
(function($){
// Local
})(window.jQuery)
Or refactored as single object if this code can be used in other js files
Sandwich style: Light and Dark must also affect sandwich background color, with dark as default (see design). This control affects sandwich icon (menu and close and background)
Let's set social icon color by default regular color and upon hover it becomes active color. As for example I have active color yellow, but transparent yellow looks really bad on blue background. See image:
\VC_Fonts::vc_theme_google_font_variants looks wrong. Why it is hardcoded? if user selected 100 why not to add it?
// Only add "700" if it exists
if ( in_array( '700', $variants ) ) {
$chosen_variants[] = '700';
}
I added featured image to the post and activated Featured Image in Customizer. In post view I see featured image, then I disable feature image in Customizer, but image on top of the content is still there and not gone.
There are images in image folder of the theme design which we do not use (no option to import dummy data). So there is no reason to keep these images there.
Now there are 2 png images dark and white for close button, we need to replace them with one SVG which get color applied to it. Otherwise retina display will not be happy about some png images and we will have problems later on if we want to introduce more sandwich menu options/colors.
I have attached svg file to use - the size should be ok, just let me know. close-icon.zip
When I select a number of columns for Footer widget and go to Widgets are I still don't see column option. I need to refresh Customizer by Shift+Cmd+R to see the changes.
For regular end-user, it will look like a clear bug.