[Closed] FEEDBACK ON BETA of v1.5.5

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.

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. :)

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.)

picaflor-azul:

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.

lat9:

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

2. jscript_matchHeight.min.js

3. jquery.mmenu.min.all.js

4. 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

picaflor-azul:

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.

balihr:

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

picaflor-azul:

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... :wink:

lat9:

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

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.

ibuttons:

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

2. add a home page slide show

3. change the look of the menu items

4. create tabs on the product info page to display product "description", "details", and "reviews

5. 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.

This default template does not come with all of the bells and whistles like a slide show or tabs, etc. ;) They (and any other features you need) can be added. You can make css changes using the stylesheets. If you get stuck on a specific change you can post and I or someone can help you out ;)

You can certainly use the Responsive Sheffield Blue or any other template with this version of zen cart. If you are building a live store, you might want to wait until the final version of 1.5.5 is released to use the code.

Thanks,

Anne

picaflor-azul:

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

Thanks,

Anne
Much appreciated, Anne!

Thanks, Anne.
I am working on a test version and want to see what it will do, so I'll install the RESPONSIVE SHEFFIELD BLUE and play around with it for a bit to get some experience.

picaflor-azul:

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

I don't know if the pixel width is as much of a "thing" as I think think the percentage of screen. Allow me to explain what I just came across and how I browse... (Remember, obviously I'm not your "typical" internet user otherwise I wouldn't be where I am with ZC. :))

Anyways, I went to my page on the beta version, didn't really matter what, though let's say I went to a product, I then had the screen width reduced to about 50% of my screen display so that I could compare what I had on screen with notes on a different file or an email to make sure that I got the right product and had the settings needed. I'm a guy, I get things wrong so I try to not do that where I can avoid it... But, my options were to go back to the home page from where I could navigate to the "new", "specials", etc.. product, navigate up one or more categories (bread crumbs or the previous-return to product list-next buttons), etc...

I haven't gone to look at how that turn off occurs, but do wonder if the set value is perhaps a little too small for turnoff of the menu which may include other options besides just categories...

So, I don't know a magic number or setting, but did find it surprising to not have any navigation menu when not yet thinned down to almost nothing.

Vanilla install, responsive classic template, index.php?main_page=index&cPath=53

screen width of #mainWrapper at: 757px - 799px.

At this point, the Add Selected To Cart button has shifted downward and now is off the right side of the background grey box, part of the white text: Displaying 1 to 10 (of 12 products) has wrapped around to the white background (non-visible text).
The Add Selected To Cart button first shifted down at 1060px.

Also, I don't recall changing a setting, but I would have thought that perhaps the product list would be columnar rather than by row or at least available with this template without additional plugin. I guess though if trying to keep the # of modified files to a minimum in each basic template, I could see how that could affect all templates not just the desired.

So is there now to be a sort of responsive "default" template? (Similar to the template_default folder but for all responsive templates?) I'm thinking from the perspective of overrides and file loading if that has or is changing in this version...

Had one problem with install - Running zc_install it says...
"We did not change your admin directory automatically as it already seems to have been changed from the default." but it DID automatically change the directory and does not allow it to be edited on that page.

So, perhaps it should say...."We changed your admin directory for security. You can change the name through FTP or FileManager when install is complete."

When clicking on the link in this box - Admin Directory, the popup says:

We try to rename your admin folder for you automatically, to offer a degree of security-by-obscurity. While we understand that this doesn't make it foolproof, it does discourage unauthorized visitors from attacking your site. You may still consider changing the foldername yourself (just rename the folder to whatever you wish it to be, by using your FTP program or your hosting company's File Manager tool in your hosting control panel).

This directly conflicts with what is in that box that I referenced above.

Also... this is just a goofy OCD thing, but I thought I would throw it out there just in case... In all these years I have been building zen cart websites and templates for people, the one thing that annoys the heck out of me is that the layout boxes under the tools menu is not in alphabetical order! lol It would be SO MUCH easier to blow through the list of what you want to turn on or leave off if they were listed alphabetically. Oh MY .... OCD to the max!! giggle

mc12345678:

Vanilla install, responsive classic template, index.php?main_page=index&cPath=53

screen width of #mainWrapper at: 757px - 799px.

At this point, the Add Selected To Cart button has shifted downward and now is off the right side of the background grey box, part of the white text: Displaying 1 to 10 (of 12 products) has wrapped around to the white background (non-visible text).
The Add Selected To Cart button first shifted down at 1060px.

Also, I don't recall changing a setting, but I would have thought that perhaps the product list would be columnar rather than by row or at least available with this template without additional plugin. I guess though if trying to keep the # of modified files to a minimum in each basic template, I could see how that could affect all templates not just the desired.

So is there now to be a sort of responsive "default" template? (Similar to the template_default folder but for all responsive templates?) I'm thinking from the perspective of overrides and file loading if that has or is changing in this version...

The widths you are working at (757 - 799 pixels wide) are not typically laptop or desktop widths. If you want to view the tablet layout while working on a desktop, switch the user agent using the links on the home page or:

for a tablet:

yourdomain.com/index.php?main_page=index&layoutType=tablet

for a mobile:

yourdomain.com/index.php?main_page=index&layoutType=mobile

Thanks,

Anne

the one thing that annoys the heck out of me is that the layout boxes under the tools menu is not in alphabetical order! lol It would be SO MUCH easier to blow through the list of what you want to turn on or leave off if they were listed alphabetically. Oh MY .... OCD to the max!! giggle
...and it would annoy me no end if they layout boxes were not listed in the order I have set for their sort order, which mirror how they display on the site. layout boxes I am not using I set to right column and a really high sort order number to move them to the bottom of the listing, so the only ones I have turned on are listed first in the order I want them in.

barco57:

...and it would annoy me no end if they layout boxes were not listed in the order I have set for their sort order, which mirror how they display on the site. layout boxes I am not using I set to right column and a really high sort order number to move them to the bottom of the listing, so the only ones I have turned on are listed first in the order I want them in.

I would agree that "losing" the current sort order would be a problem, but perhaps adding the ability to sort by clicking on a header field would be a good mix?

balihr:

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?

The average user wont even consider resizing their browser, 1024 x 768 since at least March 2009 is the average desktop screen size only increasing in width and I can't recall flipping my desktop monitor from portrait to landscape. The desktop version of this template is indeed responsive out of box.

For development purposes and "the average user" their are pretty obvious and strategically placed links on the home page to switch views and most modern browsers now provide UA switching.

balihr:

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 personally think instead of changing the code from UA to CSS media queries a admin switch might be more sufficient to allow one to choose whether or not they want the mobile menu to display in desktop view.

balihr:

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... :wink:

Just an FYI, this template isn't based on Foundation at all.

As always, just my #2cents