[Closed] FEEDBACK ON BETA of v1.5.5

[mc12345678]

This having to do with store_manager.php

admin/store_manager.php:

Store Manager: Update Hit Counter
to to a new value

(Anyone notice the double to?)define at: admin/includes/languages/english/store_manager.php line 23

admin/store_manager.php
Set next order number
NOTE: You cannot set the order number to a value lower than any existing order already in the database.

Shouldn't the existing order number show up as part of this statement, even if it could be constantly changing while on the screen?

Store Manager: Cleanup Debug Log Files: Indicates that will address debug logs associated with PHP errors and payment modules, in the /logs/ folder, but what about install logs? The code appears to support the delete, but the statement does not. Line 66 of the same language file as above.

[mc12345678]

Came back to revisit this after installing the software, but in tpl_modules_attributes, there is a check/variable called $options_html_id on line 30 that does not appear to be defined in includes/modules/attributes.php nor anywhere else in the code...

Anyone have any ideas why it is there? Won't there be a problem identified that referencing an array that is never created, nor assigned? Maybe not so much an issue with recalling data that hasn't been assigned, but still seems like it could end up being random.

Okay, going to take a look at the instructions/documentation this time. :) Enough playing and commenting for now. :)

[mc12345678]

Okay.. Sorta tough to read through the documentation when there doesn't appear to be any?

the install.txt file in the root says (areas related to the documentation highlighted in red):

Zen Cart(R) - The Art of E-Commerce

Welcome to Zen Cart(R) E-Commerce

THE FOLLOWING IS A VERY SIMPLIFIED INSTALLATION DOCUMENT.
WE SUGGEST YOU USE THE HTML VERSIONS IN THE /docs FOLDER FOR MORE DETAILED INSTRUCTIONS.

THE README DOCUMENTS ARE IN THE /docs FOLDER OF THIS DISTRIBUTION.


You can find online documentation at: https://tutorials.zen-cart.com


Zen Cart has a built-in automatic installation system. However to use this you must first unpack and upload the code to a compatible web server:

1. Extract the Zen Cart ZIP file (and tell the unzip utility to *retain* folder structures)
2. Upload the Zen Cart files and folders to your server using your FTP program
3. You must make some folders writable. The details are in the HTML-formatted documentation in the /docs/ folder
4. In your browser enter the URL to your website where you uploaded the files, specifically to the /zc_install/index.php file.
5. Follow the instructions. You must create your MySQL database credentials before installation, and provide those details when prompted.

For more detailed installation instructions, please see the /docs/1.readme_installation.html document in this distribution.






(Document Revision: $Id: install.txt 19798 2011-10-12 05:52:54Z drbyte $)
((c)Copyright 2003-2011, Zen Cart(R). All rights reserved.)

[balihr]

The new template is responsive out of the box.

Thanks,

Anne

Anne, sorry, but I strongly disagree with this. A truly responsive template works on ALL screen sizes no matter what device is used. This template is "broken" when screen is reduced to less than than 768px - there's no menu and no navigation at all, thus making it seem broken. I mentioned this before...

I understand using mobile detect class, but IMHO, this should not be used on the mobile menu. You're using the same menu as I always do for mobile (mmenu). I usually put the mobile navigation div in the footer (it's using absolute position anyways) and just hide it with css on larger screens. If anyone thinks this would have some major impact on site speed or performance because of the few extra queries or the few extra bytes loaded, well, I won't go into that discussion... But, that one MINOR change would make the template truly responsive. This way, it just looks unprofessional.

[picaflor-azul]

Within the classic_responsive/jscript directory, there are a bunch of minimized javascript files

1. jscript_matchHeight.min.js
2. jquery.mmenu.min.all.js
3. jquery.mmenu.fixedelements.min.js

Having the unminimized versions, too, will be a big help ... just in case anything "goes funky".

If you do a search for the jscripts you will find the links to their sites which have un minimized versions of the scripts.

Thanks,

Anne

[lat9]

If you do a search for the jscripts you will find the links to their sites which have un minimized versions of the scripts.

Thanks,

Anne

That's all well-and-good, but other than the jquery.mmenu.min.all.js, there's no version number in the minimized script. It's certainly going to make life easier to debug in the future if the unminimized versions were simply part of the distribution.

[picaflor-azul]

Anne, sorry, but I strongly disagree with this. A truly responsive template works on ALL screen sizes no matter what device is used.

The template meets this definition of responsive design from the Wikipedia:

https://en.wikipedia.org/wiki/Responsive_web_design

Responsive web design (RWD) is an approach to web design aimed at crafting sites to provide an optimal viewing and interaction experience—easy reading and navigation with a minimum of resizing, panning, and scrolling—across a wide range of devices (from desktop computer monitors to mobile phones)

If you feel that a significant number of your web site's clients will be viewing the website with their desktop window minimized below 768 pixels wide then you can do one or more of these:

* un hide the categories sidebox
* un hide the top categories navigation tabs
* add the menu code to desktops as you have stated in your post

Thanks,

Anne

[balihr]

The template meets this definition of responsive design from the Wikipedia:

https://en.wikipedia.org/wiki/Responsive_web_design



If you feel that a significant number of your web site's clients will be viewing the website with their desktop window minimized below 768 pixels wide then you can do one or more of these:

* un hide the categories sidebox
* un hide the top categories navigation tabs
* add the menu code to desktops as you have stated in your post

Thanks,

Anne

Well, Anne, if you're gonna go quoting wikipedia and highlighting it for me, then you might also consider highlighting the other, maybe most important part (the one I'm talking about):

Responsive web design (RWD) is an approach to web design aimed at crafting sites to provide an optimal viewing and interaction experience—easy reading and navigation with a minimum of resizing, panning, and scrolling—across a wide range of devices (from desktop computer monitors to mobile phones)

Furthermore, the same article on wikipedia clearly states how that's done and does NOT mention MobileDetect class as a part of RWD, but instead speaks of using CSS3 media queries to decide what content is to be shown. Therefore, this is NOT a responsive template and should not be advertised as such. This is a mobile-ready template, that's all. There IS a difference.

I'm not trying to be rude here, nor am I trying to tell you how to do your business, but I'm trying to think like an average user here. What's the best way to test if a site is responsive or not? Well, just try resizing your browser window and see what happens. Right? What do you think, how many average users out there will know how to override user agent in their browsers to see how their site looks on mobile devices?

I'm kindly asking you to make this minor change in the code and use CSS3 media queries to show/hide the mobile menu, instead of triggering it based on user agent. I'm sure it will help a lot of average users (or at least make them look professional enough)... Don't think about me and what I can do to change it - personally, I won't be using that template at all - I have my own template as starting point based on Bootstrap, I have never been a fan of Foundation... Think about all the other users who have no idea how to do it...

[picaflor-azul]

That's all well-and-good, but other than the jquery.mmenu.min.all.js, there's no version number in the minimized script. It's certainly going to make life easier to debug in the future if the unminimized versions were simply part of the distribution.

I have just submitted the un minimized versions of the scripts to be included in the distribution ;)

Thanks,

Anne

[ibuttons]

I got the beta installed ok and am poking around. So far, so good, but I would also like to:

1. add a home page slide show
2. change the look of the menu items
3. create tabs on the product info page to display product "description", "details", and "reviews
4. etc

Is this beta just the responsive design features? That's great, but I need more. For some reason, I had thought it would include the above stuff and more.

Is all this stuff in the RESPONSIVESHEFFIELD BLUE template? And if so, can I use that template on this beta right now or should I wait til the dust settles on the beta then add?

I feel like this is such a basic (aka stupid) question. If so, my apologies. I have looked, but not been able to find these answers.

If there is documentation with an overview of this beta and what it does specifically, please send me a link.