Winchester Responsive

rbarbour:

The reason it was not working is because it was looking for a $_SESSION on page load which did not exist, I fixed that!

Now the php-detect class renders the proper responsive files on page load and YES, sets the $_SESSION after a link is clicked.

My fix will be added to the update of the DIY Responsive Template Default for v1.5.x package, Anne may use or choose to go your route or any other as she sees fit. Just looking at what you provided, I'm sure it works but have not tested.

Got it. Just wanted to mention the fix in case anyone was developing a site and depended on the switches to test the layouts. Just did not want anyone to waste their development time to only find out later they were not modifying the correct files.

A note to Anne is that the fix you implemented has not solved the switches. The demo site looks like it is using the updated version you submitted, but if you were to use the switches on the homepage, it will not work. The files being pulled are static. If you don't have the fix from rbarour, you can use the workaround I posted above by including the session is blank and it will now allow the switches to change display. I have tested it this and works fine.

And your templates are top notch that is very user friendly for customers and shop owners.

The code below from post #115 is correct, works as intended and should be the ONLY code used. Using alternate code will only make future template upgrades that more difficult.

I personally updated the php-mobile-detect code within the html_header.php file of this DEMO first time around and did not inform Anne it should also be replaced in any file she used the php-mobile-detect code to call device specific code, my apologies.

I have just updated the rest of this DEMOS php-mobile-detect code and tested. I am confident that as soon as time permits, Anne will resubmit!

Again, my apologies.

Anne, my hat is off to you, this template is truly a masterpiece and most definitely the next generation of Zen Cart responsive templates.

if ($detect->isMobile() && !$detect->isTablet() or $detect->isMobile() && !$detect->isTablet() && $_SESSION['display_mode']=='isMobile' or $detect->isTablet() && $_SESSION['display_mode']=='isMobile' or $_SESSION['display_mode']=='isMobile') {

    echo $responsive_mobile;

} else if ($detect->isTablet() or $detect->isMobile() && $_SESSION['display_mode']=='isTablet' or $detect->isTablet() && $_SESSION['display_mode']=='isTablet' or $_SESSION['display_mode']=='isTablet'){

    echo $responsive_tablet;

} else if ($detect->isMobile() && !$detect->isTablet() && $_SESSION['display_mode']=='isDesktop' or $detect->isTablet() && $_SESSION['display_mode']=='isDesktop' or $_SESSION['display_mode']=='isNonResponsive'){

    echo '';

} else {

    echo $responsive_default;

  }  

rbarbour:

The code below from post #115 is correct, works as intended and should be the ONLY code used. Using alternate code will only make future template upgrades that more difficult.

I personally updated the php-mobile-detect code within the html_header.php file of this DEMO first time around and did not inform Anne it should also be replaced in any file she used the php-mobile-detect code to call device specific code, my apologies.

I have just updated the rest of this DEMOS php-mobile-detect code and tested. I am confident that as soon as time permits, Anne will resubmit!

Again, my apologies.

Anne, my hat is off to you, this template is truly a masterpiece and most definitely the next generation of Zen Cart responsive templates.

if ($detect->isMobile() && !$detect->isTablet() or $detect->isMobile() && !$detect->isTablet() && $_SESSION['display_mode']=='isMobile' or $detect->isTablet() && $_SESSION['display_mode']=='isMobile' or $_SESSION['display_mode']=='isMobile') {

echo $responsive_mobile;

} else if ($detect->isTablet() or $detect->isMobile() && $_SESSION['display_mode']=='isTablet' or $detect->isTablet() && $_SESSION['display_mode']=='isTablet' or $_SESSION['display_mode']=='isTablet'){

echo $responsive_tablet;

} else if ($detect->isMobile() && !$detect->isTablet() && $_SESSION['display_mode']=='isDesktop' or $detect->isTablet() && $_SESSION['display_mode']=='isDesktop' or $_SESSION['display_mode']=='isNonResponsive'){

echo '';

} else {

echo $responsive_default;

}

rbarbour, thank you so much for your help with this!  The php mobile detect should be working 100% now thanks to the edits made by rbarbour :smile:

I am in the process of packaging up all of the changes and other found bugs  and will be submitting an update later today.


Thanks,

Anne

picaflor-azul:

I am in the process of packaging up all of the changes and other found bugs and will be submitting an update later today.
If it's not too late, may I suggest a small addition.. in the "includes/templates/winchester_responsive/templates/tpl_modules_mobile_categories_tabs.php" file add an admin control for turning the EZ Pages menu item on or off.. Not adding pages to the menu is not enough to stop the dropdown from appearing..

       <?php if (SHOW_EZ_PAGES_MENU == 'true') { ?>
    <li><a href="#"><?php echo TITLE_EZ_PAGES; ?></a>
    <ul>
  <?php require(DIR_WS_MODULES . 'sideboxes/' . $template_dir . '/' . 'ezpages_drop_menu.php'); ?>
    </ul>
    </li>
      <?php } ?>

DivaVocals:

If it's not too late, may I suggest a small addition.. in the "includes/templates/winchester_responsive/templates/tpl_modules_mobile_categories_tabs.php" file add an admin control for turning the EZ Pages menu item on or off.. Not adding pages to the menu is not enough to stop the dropdown from appearing..

   <?php if (SHOW_EZ_PAGES_MENU == 'true') { ?>
<li><a href="#"><?php echo TITLE_EZ_PAGES; ?></a>
<ul>

</ul>
</li>
  <?php } ?>

Ok, I will take a look at this, thank you for the suggestion.  I will also be fixing the mobile menu (I hope ;) before submitting the update.

Thanks,

Anne

DivaVocals:

If it's not too late, may I suggest a small addition.. in the "includes/templates/winchester_responsive/templates/tpl_modules_mobile_categories_tabs.php" file add an admin control for turning the EZ Pages menu item on or off.. Not adding pages to the menu is not enough to stop the dropdown from appearing..

   <?php if (SHOW_EZ_PAGES_MENU == 'true') { ?>
<li><a href="#"><?php echo TITLE_EZ_PAGES; ?></a>
<ul>

</ul>
</li>
  <?php } ?>

I tried to incorporate this switch, adding the constant to the database and it does turn on and off the ezpages in the header menu, but when turned on it is not showing the ezpages marked as on in the header with a sort order.

Thanks,

Anne

picaflor-azul:

I tried to incorporate this switch, adding the constant to the database and it does turn on and off the ezpages in the header menu, but when turned on it is not showing the ezpages marked as on in the header with a sort order.

Thanks,

Anne

Disregard this post, I think I have it working now ;)

Thanks,

Anne

picaflor-azul:

Disregard this post, I think I have it working now ;)

Thanks,

Anne
Cool!!!:clap:

I just submitted an update with these changes:

05/27/2014------Added admin control for turning the EZ Pages menu item on or off in includes/templates/winchester_responsive/templates/tpl_modules_mobile_categories_tabs.php

05/27/2014------Added includes/templates/winchester_responsive/css/responsive.css div#mobile-nav1.hidenav1 {display:none;}

05/27/2014------Fixed class "onerow-fluids" includes/templates/winchester_responsive/common/tpl_footer.php, includes/templates/winchester_responsive/common/tpl_header.php, includes/templates/winchester_responsive/common/tpl_header_mobile.php, includes/templates/winchester_responsive/common/tpl_header_tablet.php, includes/templates/winchester_responsive/common/tpl_main_page.php

05/27/2014------Added code to exclude main image and additional image popups from being responsive /includes/templates/winchester_responsive/common/html_header.php

05/27/2014------Fixed php mobile detect script includes/templates/winchester_responsive/templates/tpl_modules_mobile_categories_tabs.php

05/27/2014------Fixed php mobile detect script includes/templates/winchester_responsive/common/tpl_main_page.php

Thanks,

Anne

hi... i want copy only header from (westmister_new) to this template please help .. i want add this header to this template (this header From westmister_new template )

jennydutch:

hi... i want copy only header from (westmister_new) to this template please help .. i want add this header to this template (this header From westmister_new template )

in the common/ folder you will see 3 header files. Find the block of code that you want added to this template for all 3 files. But also remember to pull out each css instance in the 4 stylesheets that reference the div and elements you are moving over.

Is it the top navigation links you want?

yaritai:

in the common/ folder you will see 3 header files. Find the block of code that you want added to this template for all 3 files. But also remember to pull out each css instance in the 4 stylesheets that reference the div and elements you are moving over.

Is it the top navigation links you want?

Yes, this is exactly what you would need to do. You will also need to add the code for the currencies and languages header boxes if you want this functionality.

Thanks,

Anne

I have searched the code and forums all night, so I decided to ask. Questions 1, 2, & 3 are all about the header menu. Question 4 is about the shopping cart link at the top.

1. I need the categories under the Categories Tab (in the header) to be wider. Which file do I need to find to edit the width of that drop-down categories menu?

2. I need to get rid of the other drop-down tab, "Information," in the header menu and make a new link without a drop-down menu to go to just one page. Where do I find that file?

3. I need to add "Home" to the header menu. Where do I find that file?

4. Is there a way to get rid of the cart at the top of the screen? I tried to do so with the Tools, Layout Boxes Controller, but evidently that is not where I need to look, so which file do I need to edit and what do I need to remove?

Thanks! :-)

CookieMonster:

I have searched the code and forums all night, so I decided to ask. Questions 1, 2, & 3 are all about the header menu. Question 4 is about the shopping cart link at the top.

1. I need the categories under the Categories Tab (in the header) to be wider. Which file do I need to find to edit the width of that drop-down categories menu?

2. I need to get rid of the other drop-down tab, "Information," in the header menu and make a new link without a drop-down menu to go to just one page. Where do I find that file?

3. I need to add "Home" to the header menu. Where do I find that file?

4. Is there a way to get rid of the cart at the top of the screen? I tried to do so with the Tools, Layout Boxes Controller, but evidently that is not where I need to look, so which file do I need to edit and what do I need to remove?

Thanks! :-)

If you post a link to your site I can take a look.

1 and 4 are css questions. If you will be making css changes to the template I strongly recommend using the chrome or firefox web developer tools. They will help you find the exact line of the exact file that you need to edit and you can test your edits in real time without reloading the page.

2 and 3 you will edit includes/templates/winchester_responsive/templates/tpl_modules_mobile_categories_tabs.php

Thanks,

Anne

picaflor-azul:

rbarbour, thank you so much for your help with this! The php mobile detect should be working 100% now thanks to the edits made by rbarbour :smile:

I am in the process of packaging up all of the changes and other found bugs and will be submitting an update later today.

Thanks,

Anne

So this needs to be added to ALL your Responsive Templates? Thanks!

uswebworx:

So this needs to be added to ALL your Responsive Templates? Thanks!

Only the templates using the new code that includes the $_SESSION for switching between devices. 3.0-Beta completely eliminates the need to include the php-mobile-detect if statements in individual files all together!

rbarbour:

Only the templates using the new code that includes the $_SESSION for switching between devices. 3.0-Beta completely eliminates the need to include the php-mobile-detect if statements in individual files all together!

Thanks, I believe I am exempt for now, or the client is until we figure out what we are going to do! We are using responsive_apparel_boutique

Thanks, Anne! You are awesome! All I needed to know was where to look, and I modified the files from there to get all four issues corrected. Thanks, again! :-)

rbarbour:

Only the templates using the new code that includes the $_SESSION for switching between devices. 3.0-Beta completely eliminates the need to include the php-mobile-detect if statements in individual files all together!

That is such a cool improvement! Thank you so much rbarbour for your work on the responsive code :cool:

Thanks,

Anne

CookieMonster:

Thanks, Anne! You are awesome! All I needed to know was where to look, and I modified the files from there to get all four issues corrected. Thanks, again! :-)

I am happy that you figured it out ;)

Thanks,

Anne