[Closed] FEEDBACK ON BETA of v1.5.5

[DrByte]

Don't know if relevant - I run my test site 1.5.4 and 1.5.5 earlier git release on WAMP/Windows 10 - I had to make everything http - no https - to avoid error.

Unless you've got a fully-certified SSL certificate installed on your local WAMP, it's completely expected that you would get SSL errors. That's not a Zen Cart issue.

[mc12345678]

ZC 1.5.5, downloaded "today" (headers reflect full version non-beta).
Created a new database, pointed my configure.php files to the new database including updating the prefix.
Imported ZC 1.3.8 database into the blank database (although it had the applicable drop/truncate commands)
Admin table already had some issues as some admin_level fields had a zero, one had a one.
Username/password was definitely expired.
Used the How do I change or reset my Admin Password? (I lost it) to establish a username to support the database upgrade.
Upgrade only on ZC 1.3.8 database.

PHP 5.6 (Zend 2.6.0)
Server OS: Linux 3.2.61-grsec-modsign
PHP Memory Limit: 128M

Database: MySQL 5.6.25-log



No issues to report with the upgrade.
Database upgrade was performed in one "step", there are a few log errors from the install but wouldn't have been known if I hadn't gone to look for/at them out of curiosity:

[13-Mar-2016 00:54:42 America/New_York] MySQL error 1091 encountered during zc_install:
Can't DROP 'page_accessed'; check that column/key exists
ALTER TABLE admin_activity_log DROP INDEX page_accessed;
---------------


[13-Mar-2016 00:54:42 America/New_York] MySQL error 1091 encountered during zc_install:
Can't DROP 'access_date'; check that column/key exists
ALTER TABLE admin_activity_log DROP INDEX access_date;
---------------


[13-Mar-2016 00:54:42 America/New_York] MySQL error 1061 encountered during zc_install:
Duplicate key name 'idx_page_accessed_zen'
ALTER TABLE admin_activity_log ADD INDEX idx_page_accessed_zen (page_accessed);
---------------


[13-Mar-2016 00:54:42 America/New_York] MySQL error 1061 encountered during zc_install:
Duplicate key name 'idx_access_date_zen'
ALTER TABLE admin_activity_log ADD INDEX idx_access_date_zen (access_date);
---------------

[mc12345678]

Logged into admin with mobile device.

Nearly the entire lower half of the screen is taken up by logo, copyright, etc... Be nice if that could be turned off in some way.

Menu button on upper right corner doesn't work.

Only options are the log viewer and currently the gift certificate queue because the database did have a gift certificate to get validated from back when it was backed up. Once I clicked on one of these, the half screen problem section disappeared, so at least it does go away eventually, but if I go back to my admin directory /index.php the "popup" appears again. Further, pretty much have to know all of the commands (or use alt_nav.php) to navigate...

Have seen conversations about the grid framework, etc... Don't really care as long as it works.. I do think that having one on one side and the other on the other side offers an opportunity and method to see/choose what it would take for one or the other. So, how does one turn off this admin template thing or what is needed to "correct" the issue?

[lat9]

Is there a reason that the admin load of the jQuery library (from /admin/includes/header.php) is done with the defer attribute:

<script defer src="//ajax.googleapis.com/ajax/libs/jquery/1.11.3/jquery.min.js"></script>
<script defer src="//maxcdn.bootstrapcdn.com/bootstrap/3.3.6/js/bootstrap.min.js"></script>

It's currently wrecking havoc on an admin tool that I'm developing, making it "very difficult" to get my jQuery helper and its initialization loaded without error. If I simply remove the defer attributes, all is well.

[DrByte]

Logged into admin with mobile device.

Nearly the entire lower half of the screen is taken up by logo, copyright, etc... Be nice if that could be turned off in some way.

There was a pending PR that hadn't been merged yet ... to deal with that footer issue. Should be okay now.

Menu button on upper right corner doesn't work.

Which mobile device are you using? Screen resolution dimensions?

Is there a reason that the admin load of the jQuery library (from /admin/includes/header.php) is done with the defer attribute:

<script defer src="//ajax.googleapis.com/ajax/libs/jquery/1.11.3/jquery.min.js"></script>
<script defer src="//maxcdn.bootstrapcdn.com/bootstrap/3.3.6/js/bootstrap.min.js"></script>

It's currently wrecking havoc on an admin tool that I'm developing, making it "very difficult" to get my jQuery helper and its initialization loaded without error. If I simply remove the defer attributes, all is well.

I have no preference. I think the defer has it place, but may not be necessary in this context.

[mc12345678]

There was a pending PR that hadn't been merged yet ... to deal with that footer issue. Should be okay now.

Will take a look-see about merging the PR for that.

Which mobile device are you using? Screen resolution dimensions?

Blackberry 9930, 640 x 480 pixels... While it is touch screen, it does have single mouse button click capability like a macintosh basically.

Button responded as if moused over, correction, but when clicked no change, not in screen not in response; no action occurred..

[lat9]

There was a pending PR that hadn't been merged yet ... to deal with that footer issue. Should be okay now.

Which mobile device are you using? Screen resolution dimensions?


I have no preference. I think the defer has it place, but may not be necessary in this context.

If possible, could it be removed? I haven't gone back to my other jQuery-using admin plugins, but I know that removing the defer made my under-development plugin go from not-working to working.

[DrByte]

If possible, could it be removed? I haven't gone back to my other jQuery-using admin plugins, but I know that removing the defer made my under-development plugin go from not-working to working.

I have a new pending PR which includes the removal of `defer` along with other fixes for the admin nav area.

[DrByte]

Will take a look-see about merging the PR for that.

Blackberry 9930, 640 x 480 pixels... While it is touch screen, it does have single mouse button click capability like a macintosh basically.

Button responded as if moused over, correction, but when clicked no change, not in screen not in response; no action occurred..

I hope when today's commits are merged (tomorrow) those issues will be resolved.

[mc12345678]

I hope when today's commits are merged (tomorrow) those issues will be resolved.

The four commits up to commit: 7145bb6061c2b6cc3863c3ef71fd201aed14f3e1

Seemed to fix things.. I saw that there were a few PRs in the queue. I didn't try to apply them to see what would change, but the work done Saturday to Sunday did resolve the mobile in admin navigation issue.

I did notice one thing that still looked a little off, the button to accept related to gift certificate queue, overlaps the ZenCart logo slightly. There isn't much room to move. To the right, because the button for the activity log is adjacent. That second button does have some room to move right, but I find it difficult to provide clear coding direction.

With regards to the overlap of the first button, the lower left part of the button itself begins half way up from the a n the logo. The text description beneath it is somewhat lost at the beginning within the logo. The C in Certificate is position over the lower part of the t in ZenCart. Two fixes come to mind, either move over the Gift Certificate Queue button and hopefully the second button also would move or reduce the size of the logo in this setup...