Quote Originally Posted by torvista View Post
As per the previous poster, I tried this on a local install xampp 181, php 5.4.7 but not using the default configurations for php.ini nor apache httpd.conf due to previous problem solving attempts. Also using virtual hosts rather than alias.

I subscribe to the design theory: "Don't make me think" (or rather "...assume I cannot think").
It looks great and very important for first impressions.

Some nitpicking comments:

1) I note tabbing between the input fields requires an extra click every time.

2) Database setup screen: why not check for the existence of the mysql server when entering this page and flag it immediately?

3) Creating database:
I got multiple repeat error messages:
Warning: sha1_file(DIR_FS_CATALOGDIR_WS_INCLUDESconfigure.php): failed to open stream: No such file or directory in D:\My Documents\GitHub\zc-v1-series\zc_install\includes\functions\password_funcs.php on line 118

The read.md says you have to rename the configures, why? They get created and the store works without doing that.

4) I think the labels stating that you must make a note of your password and the new name of the admin folder should be more in your face.

5) There are some minor validation errors (using Total Validator) on each page.

6) Shop: alt-nav looks like the page is broken, no css is loaded.

What about detecting its a localhost install and creating local/configure.php?

Steve
Thanks for the detailed feedback Steve.

I'll address the tab order asap :)

2) Database setup screen: why not check for the existence of the mysql server when entering this page and flag it immediately?
Not sure what you mean here. The system checking code should have already tried to see if there is a live mysql server, and if not informed you on the initial screen.

3) Creating database:
Noted: Will check code.

The read.md says you have to rename the configures, why? They get created and the store works without doing that.
Apologies the read.md is a little out of date, and based on old installer/v1.51

4) I think the labels stating that you must make a note of your password and the new name of the admin folder should be more in your face.
Fair enough :)

5) There are some minor validation errors (using Total Validator) on each page.
Double fair enough :)

6) Shop: alt-nav looks like the page is broken, no css is loaded.
Assume you mean the admin menu alt-nav. You haven't signed an NDA so I can't tell you that there will be changes coming to the admin menu infrastructure.

What about detecting its a localhost install and creating local/configure.php?
Nice idea. will ponder. :)