X-Cart forums

[xgarb]

The banner module as far as I can see serves the same size images whatever the screen size.

This means a lot of waste bandwidth on small devices.

Foundation has a tool that requests content based on screen size: http://foundation.zurb.com/sites/docs/interchange.html

This is different from just using classes to show or hide content as the content isn't requested at all with Interchange.

Is there any plans to introduce something like this in the module?

[qualiteam]

Do you mean this banner module?
https://market.x-cart.com/addons/banner-system.html

[xgarb]

Yes.

[Dmitry]

If there is no other idea then why not try to dev a simple module for loading a custom css file on home page only and load @media css for this container items...

Please let me know your feedback

[qualiteam]

Thank you for your feedback!

I've notified the module developer and she has this improvement in her to-do list for the module.
However, it is in her future plans, not something that will be implemented in the near future.

[xgarb]

I'm wondering if the mobile detection scripts could be used somehow.

maybe class AMobile_Detect in xcart/lib/

or something with

\XLite\Core\MobileDetect::getInstance()->isMobile()

to do something like

{% if this.getBannerBoxes() && !isMobilePhone() %}
<div class="bannerBox">
......
{% endif %}

?

[xgarb]

I just discovered after some investigation that the IsVisible method for this module always returns true. Possible due to the negation of the isMobileDevice not giving the expected result.

    /**
     * Check if widget is visible
     *
     * @return boolean
     */
    protected function isVisible()
    {   
        if (!\XLite\Core\Database::getRepo("XLite\Model\Module")->findOneBy(array("name"=>"Mobile", "author"=>"XC", "enabled"=> true)) || !\XLite\Core\Request::isMobileDevice()) {
          $result = true;
        } else {
          $result = false;
        }
        
        return parent::isVisible() && $result;

    }

Dropping the negation and swapping the true and false like this works..

    protected function isVisible()
    {   
        if (\XLite\Core\Database::getRepo("XLite\Model\Module")->findOneBy(array("name"=>"Mobile", "author"=>"XC", "enabled"=> true)) || \XLite\Core\Request::isMobileDevice()) {
          $result = false;
        } else {
          $result = true;
        }
        
        return parent::isVisible() && $result;

    }

So now it doesn't show the banner on mobile which is what I wanted.