Jump to content

[Resolved] Foundation Skin, supress change skin message


keat

Recommended Posts

On the foundation skin, pops up a lower banner giving the option to change the skin (we offer Kurouto for a V3 legacy look)

However, if I click the cross to dismiss the banner, it comes back again when I change categories.

Is there a way to supress this, it's annoying.

Link to comment
Share on other sites

This would then disable the option to choose Kurouto.

The site in question was V3 for many years,  regular customer know thier way around the V3 site, hence the reason we offer Kurouto.

Link to comment
Share on other sites

5 hours ago, keat said:

However, if I click the cross to dismiss the banner, it comes back again when I change categories.

Is there a way to supress this, it's annoying.

Agreed !  I did report this as a github issue a long time ago but cannot find the original issue now - it may have been an original V5 issue

Link to comment
Share on other sites

I have never offered skin changes to customers, so I'd never seen this issue before. I just played with your store, and I see what you mean. This definitely needs fixing. Maybe something session cookie related, like the site cookie acceptance message?

@keat Please make a GitHub request on this.

Link to comment
Share on other sites

  • 3 weeks later...

Yes someone pointed out that CubeCart looks badly managed/full of bugs on github due to the sheer volume of open issues. Most of these are in fact features request. The idea is to close them without milestone for future consideration. Once we start integrating we will assign a milestone and open. 

Link to comment
Share on other sites

  • 2 weeks later...

You should be able to move the  {include file='templates/box.skins.php'} to this div in main.php Foundation templates:

<div class="large-3 columns show-for-large-up" id="sidebar_left">
                  {include file='templates/box.featured.php'}
                  {include file='templates/box.popular.php'}
                  {include file='templates/box.sale_items.php'}
               </div>

You will probably have to look at the code in the box.featured, etc., to see how to get the skin change box to show up properly.

Link to comment
Share on other sites

This seems to work for box.skins.php:

{if $SKINS}
<div class="panel" id="box-sale_items">
<form action="{$VAL_SELF}" method="post" class="autosubmit" id="box-skin">
      <h4>Change Skin:</h4>
         <select name="select_skin" class="auto_submit">
         {foreach from=$SKINS item=skin}
         {if isset($skin.styles)}
         {foreach from=$skin.styles item=style}
         <option value="{$skin.name}|{$style.directory}" {$style.selected}>{$skin.display} - {$style.name}</option>
         {/foreach}
         {else}
         <option value="{$skin.name}" {$skin.selected}>{$skin.display}</option>
         {/if}
         {/foreach}
         </select>
</form>
</div>
{/if}

Move the call for the skins box in main.php here (on right or change to make on left - depending on how your columns are setup):

<div class="large-3 columns show-for-large-up" id="sidebar_right">
                  {include file='templates/box.featured.php'}
                  {include file='templates/box.popular.php'}
                  {include file='templates/box.sale_items.php'}
				  {include file='templates/box.skins.php'}
               </div>

 

Link to comment
Share on other sites

Now I understand.

 

In the foundation, templates folder

 

I changed box.skins.php to the code above and then added the string

{include file='templates/box.skins.php'}

to main.php.

 

Although, this string already appears firther down the file, so I did initially end up with two 'change skin' boxes.

This is much tidier. 

 

Thanks

 

 

 

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...