NO BUG: Issues with {echo:wp_is_mobile()}

[cromar]

Hi,

so in header we have two different navigations, desktop and mobile and based users device it will show correct one, never had issues before but on this website it randomly shows sometimes desktop on mobile and vice versa, for example on 10 pages it shows desktop but on one mobile and vice versa if on mobile.

Did any of you had any issues with this? I am running newest bricks. version with all codes signed and put to functions.php based on bricks instructions. I don’t want to share website globally here so please help without website if possible, but if not possible I will send it.

[HeroRox]

Doesn’t work for me either (19.7.1 + 1.9.8 beta).

[simon]

It’s working OK for me using 1.9.7.1 but I’m not using it to control the header menu, instead using it to control visibility of two elements, one for desktop and one for mobile.

I have this set in functions.php:

add_filter( ‘bricks/code/echo_function_names’, function() {
return [
‘wp_is_mobile’,
‘get_days_difference’,
];
} );

Then I have this set as the condition to show say the element for desktop devices:

dynamic data
{echo:wp_is_mobile}
==
0

So no () at the end of wp_is_mobile

What I did find when I initially set this up is that caching has a major effect. If I cleared the cache and then on desktop viewed the page with the elements in, the desktop element is correctly shown and therefore cached. This meant it was also shown on mobile devices. Vice versa if I cleared the cache and then viewed the page using a mobile device, that element got cached and was shown on desktops too. It drove me mad for a few hours until I realised it was caching and therefore set the page to be not cached.

Hopefully this might help in some way.

[HeroRox]

Thanks @simon I had not used a cache plugin. Now it works - I previously had the code in a snippet plugin and have now added it to the function.php.

[timmse]

Hi Cromar,
Thanks so much for your report!

Unfortunately, I cannot replicate the issue using these conditions:

[jins8]

@simon

wp_is_mobile

Is wp_is_mobile a built-in function already in the theme?

[wplit]

it’s a native WP function. It’s unreliable tbh, as soon as caching is added it becomes useless. (this is mentioned in the WP docs)

[simon]

Yes, I have taken the pages I am using the function on out of caching to avoid this.

[cromar]

Yes, actually had huge issues on all websites using the function recently because of caching. Do any of you know maybe some other better option if using cache because if using only CSS media queries and if you want the website to pass Google PSI it isn’t good that the same elements load on both?