Westminster New

[simon1066]

Out of the two I've decided to go with this template, the home page slider better suits my needs.

I've noticed in the readme for both templates that - includes/init_includes/config.display_mode.php - is stated to be in the package, although it's not.

No doubt a typo.

Thank you for providing this template Anne, I'm implementing it in a heavily modded and customised site, it'll be a challenge but is seriously needed as 40% of this sites' traffic is mobile/tablet based, as no doubt is the case of most sites these days.

[picaflor-azul]

Out of the two I've decided to go with this template, the home page slider better suits my needs.

I've noticed in the readme for both templates that - includes/init_includes/config.display_mode.php - is stated to be in the package, although it's not.

No doubt a typo.

Thank you for providing this template Anne, I'm implementing it in a heavily modded and customised site, it'll be a challenge but is seriously needed as 40% of this sites' traffic is mobile/tablet based, as no doubt is the case of most sites these days.

I am happy that you like the template ;)

The file is in files/includes/init_includes/init_display_mode.php in the zip file that I have uploaded.


Thanks,

Anne

[simon1066]

The file is in files/includes/init_includes/init_display_mode.php in the zip file that I have uploaded.

Anne

Yes, I thought that was the case.

[RixStix]

Slight alignment issue. I don't think size matters but this was a 7" touch device. I will try using a real phone later.

I stopped testing using Android tablets due to the menu issue.

FWIW: 11% of our site traffic stats is from Android devices.

==========================================
This is a phone.
Your UA is Mozilla/5.0 (Linux; Android 4.2.2; DX752 Build/JDQ39) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/33.0.1750.170 Mobile Safari/537.36

Attachment 14080

[picaflor-azul]

Slight alignment issue. I don't think size matters but this was a 7" touch device. I will try using a real phone later.

I stopped testing using Android tablets due to the menu issue.

FWIW: 11% of our site traffic stats is from Android devices.

==========================================
This is a phone.
Your UA is Mozilla/5.0 (Linux; Android 4.2.2; DX752 Build/JDQ39) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/33.0.1750.170 Mobile Safari/537.36

Attachment 14080

I just corrected the php mobile detect script on the demo. Give it another try and let me know if it looks different.



Thanks,

Anne

[RixStix]

Here it is. Yes, there is some difference. Screengrab from phone.

The 7" has now decided to be difficult with screengrabs.

Attachment 14081Attachment 14082

[yaritai]

Here it is. Yes, there is some difference. Screengrab from phone.

The 7" has now decided to be difficult with screengrabs.

Attachment 14081Attachment 14082

I suggest you stop editing the css files and layout until you get the full fix for the mobile detect. There are 2 issues which 1 was partially fixed in the winchester template by rbarour. Basically, the fix was to include the 2.0 workaround to call the correct css files. However, the code fix did NOT attempt to change ALL instances where specific device files are called. And this is where your issue comes in as the tpl header file is the same one called for mobile, tablet and desktop. But if you look in the common folder, you will see that the 2.1 code has specific headers for each device.

So the same logic has to be implemented, BUT it depends on whether you are using the switches on the templates. If you do NOT use the switches to change views, then you can use this code:

In tpl_main_page.php, find:

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


    require($template->get_template_dir('tpl_header_mobile.php',DIR_WS_TEMPLATE, $current_page_base,'common'). '/tpl_header_mobile.php');

      } else { 

    require($template->get_template_dir('tpl_header.php',DIR_WS_TEMPLATE, $current_page_base,'common'). '/tpl_header.php'); 

change to:

      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'){ 


    require($template->get_template_dir('tpl_header_mobile.php',DIR_WS_TEMPLATE, $current_page_base,'common'). '/tpl_header_mobile.php');

      } else { 

    require($template->get_template_dir('tpl_header.php',DIR_WS_TEMPLATE, $current_page_base,'common'). '/tpl_header.php'); 

A note on this is that is the tpl_header_tablet is not being called in the code. I don't know whether it was done on purpose by Anne. But if the tpl_header_tablet is suppose to be called for tablets then you can use this code:

      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'){ 

    require($template->get_template_dir('tpl_header_mobile.php',DIR_WS_TEMPLATE, $current_page_base,'common'). '/tpl_header_mobile.php');

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

require($template->get_template_dir('tpl_header_tablet.php',DIR_WS_TEMPLATE, $current_page_base,'common'). '/tpl_header_tablet.php');

     } else {
    require($template->get_template_dir('tpl_header.php',DIR_WS_TEMPLATE, $current_page_base,'common'). '/tpl_header.php');

      } ?> 

If you are using the switches, then follow the posts I wrote in the winchester as you will have to add the session to each if statement. Also, you may have to modify other files if you are using the mobile detect code to call certain files.