[Closed] FEEDBACK ON BETA of v1.5.5

Responsive template, no additional plugins, possibly a day or two old installation, but didn't find comment on the issue in this thread.

meta name for author in the header is not ended properly:

Code:

---

<meta name="author" content="">

---

I removed the content between the quotes, but that is properly populated, the issue is that it doesn't resolve to:

Code:

---

<meta name="author" content="" />

---

Sorry for stopping there on the research with only reporting the issue and not a fix. No other template(s) installed, no modifications loaded (other than was testing the init_canonical modifications suggested on another thread and started looking at the source code to find this issue).

Also, two "errors" reported in validation, though have also seen "on the internet" that it may not really be an issue...

required attribute "type" not specified for both areas.

Code:

---

<script> if (typeof zcJS == "undefined" || !zcJS) {  window.zcJS = { name: 'zcJS', version: '0.1.0.0' }; };  zcJS.ajax = function (options) {  options.url = options.url.replace("&amp;", "&");  var deferred = $.Deferred(function (d) {      var securityToken = 'obscured to not publicly post an SID';      var defaults = {          cache: false,          type: 'POST',          traditional: true,          dataType: 'json',          timeout: 5000,          data: $.extend(true,{            securityToken: securityToken        }, options.data)      },      settings = $.extend(true, {}, defaults, options);

---

Where the initial <script> does not have a type assigned to it.

Same issue at bannerSix for the following "advertisement?" which would be a database population issue I believe not so much a "code" issue.

Code:

---

<!--bof- banner #6 display --> <div id="bannerSix" class="banners"><script><!--//<![CDATA[    var loc = '//pan.zen-cart.com/display/group/1/';    var rd = Math.floor(Math.random()*99999999999);    document.write ("<scr"+"ipt src='"+loc);    document.write ('?rd=' + rd);    document.write ("'></scr"+"ipt>"); //]]>--></script></div> <!--eof- banner #6 display -->

---

Quote:

---

Originally Posted by mc12345678

Responsive template, no additional plugins, possibly a day or two old installation, but didn't find comment on the issue in this thread.

meta name for author in the header is not ended properly:

Code:

---

<meta name="author" content="">

---

I removed the content between the quotes, but that is properly populated, the issue is that it doesn't resolve to:

Code:

---

<meta name="author" content="" />

---

Sorry for stopping there on the research with only reporting the issue and not a fix. No other template(s) installed, no modifications loaded (other than was testing the init_canonical modifications suggested on another thread and started looking at the source code to find this issue).

Also, two "errors" reported in validation, though have also seen "on the internet" that it may not really be an issue...

required attribute "type" not specified for both areas.

Code:

---

<script> if (typeof zcJS == "undefined" || !zcJS) {  window.zcJS = { name: 'zcJS', version: '0.1.0.0' }; };  zcJS.ajax = function (options) {  options.url = options.url.replace("&amp;", "&");  var deferred = $.Deferred(function (d) {      var securityToken = 'obscured to not publicly post an SID';      var defaults = {          cache: false,          type: 'POST',          traditional: true,          dataType: 'json',          timeout: 5000,          data: $.extend(true,{            securityToken: securityToken        }, options.data)      },      settings = $.extend(true, {}, defaults, options);

---

Where the initial <script> does not have a type assigned to it.

Same issue at bannerSix for the following "advertisement?" which would be a database population issue I believe not so much a "code" issue.

Code:

---

<!--bof- banner #6 display --> <div id="bannerSix" class="banners"><script><!--//<=!=[=C=D=A=T=A=[    var loc = '//pan.zen-cart.com/display/group/1/';    var rd = Math.floor(Math.random()*99999999999);    document.write ("<scr"+"ipt src='"+loc);    document.write ('?rd=' + rd);    document.write ("'></scr"+"ipt>"); //]=]=>--></script></div> <!--eof- banner #6 display -->

---

---

Those are all just discrepancies between HTML5 vs XHTML4.
Debating whether to aggressively force the whole template to HTML5 or not ...

Quote:

---

Originally Posted by DrByte

Those are all just discrepancies between HTML5 vs XHTML4.
Debating whether to aggressively force the whole template to HTML5 or not ...

---

FWIW..

+1 vote for HTML5..

IJS.. :laugh:

Quote:

---

Originally Posted by DivaVocals

+1 vote for HTML5..

---

That makes it +2 :D

Quote:

---

Originally Posted by bn17311

the 2 boxes are slightly different sizes and change depending on screen size

Attachment 15900

Attachment 15901

bn

---

Quote:

---

Originally Posted by picaflor-azul

What pages are you seeing these on?

Thanks,

Anne

---

Quote:

---

Originally Posted by bn17311

hi,

its on the product info page

bn

---

What screen resolutions are you seeing this on?

Thanks,

Anne

Quote:

---

Originally Posted by picaflor-azul

What screen resolutions are you seeing this on?

Thanks,

Anne

---

hi

the boxes seem to change depending on screen size also out slightly on iPad

i have resized browser and done screenshot if it helps

bn

Quote:

---

Originally Posted by picaflor-azul

What screen resolutions are you seeing this on?

Thanks,

Anne

---

iPad

bn

Quote:

---

Originally Posted by bn17311

hi

the boxes seem to change depending on screen size also out slightly on iPad

i have resized browser and done screenshot if it helps

bn

---

It looks like you've turned off Admin->Configuration->Layout Settings->CSS Buttons. Is that intentional?

Quote:

---

Originally Posted by DrByte

It looks like you've turned off Admin->Configuration->Layout Settings->CSS Buttons. Is that intentional?

---

hi i was just trying this code change

Quote:

---

Originally Posted by picaflor-azul

You can add this to line 77 of the stylesheet.css:

Code:

---

height:auto;width:auto;

---

I'll get the change submitted.

Thanks,

Anne

---

but doesn't matter if on or off the boxes seem to resize

bn

Quote:

---

Originally Posted by DrByte

It looks like you've turned off Admin->Configuration->Layout Settings->CSS Buttons. Is that intentional?

---

screenshot with css buttons back on

bn