
Originally Posted by
dbltoe
Changing core files can be a problem during future upgrades if not meticulously documented. There are so many other options that don't require the modification of a core file.
The first that comes to mind in this instance is the use of a stylesheet to do a display:none where you do not want them to appear. Conversely, when needed again, simply change the stylesheet.
There are specific methods to do this and they are laid out in the CSS_read_me.txt of either the template_default or responsive_classic CSS folder.
dbltoe your comments are always appreciated however your attitude to 'display:none' seems to be a bit fickle, earlier on in this thread you were decrying it's usage and now you are advocating it. https://www.zen-cart.com/showthread....ne#post1359475
Your suggested method of using CSS does require the creation of more CSS files, this has an impact on page rendering (albeit small) and should be avoided.
I do not regard the file 'includes/templates/fluorspar/common/tpl_main_page.php' as a core file as it is part of the Fluorspar template and is therefore an 'override' file.
Modifying these files is part of the 'override system' and is the esscence of the Zen Cart open source project. It would be impossible to design a new template if you were unable to change these files.
Zen Cart v1.5.6c file :- includes/templates/template_default/common/tpl_main_page.php
contains this code :-
Code:
// the following IF statement can be duplicated/modified as needed to set additional flags
if (in_array($current_page_base,explode(",",'list_pages_to_skip_all_right_sideboxes_on_here,separated_by_commas,and_no_spaces')) ) {
$flag_disable_right = true;
}
This must endorse my method as being acceptable and probably better than creating more CSS files.
Bookmarks