New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Browser JS: horrible page reflows in tbl_structure.php #13881

Closed
Alexander-- opened this Issue Dec 22, 2017 · 1 comment

Comments

Projects
None yet
2 participants
@Alexander--

Alexander-- commented Dec 22, 2017

Steps to reproduce

  1. Install affected browser, preferably Firefox (I am using Extended Support release aka 53)
  2. Open "Table Structure" page for any table with large number of columns (> 50)
  3. The browser/page becomes unusable as countless menus for individual rows are generated

Also reproducible in Chrome, but freezes are less severe.

Expected behaviour

No reflows, the page draws fine.

Actual behaviour

The browser rendering engine trashes, as phpmyadmin "cleverly" folds many-many-many row menus into popups with Javascript.

Server configuration

Arch Linux

Web server: Nginx (irrelevant)

Database: irrelevant (this is a client-side issue)

PHP version: irrelevant (this is a client-side issue)

phpMyAdmin version: phpmyadmin 4.7.6-1 from Arch Linux official repositories

Client configuration

Browser: Firefox Extended Support and Chrome appear to be affected

Operating system: Arch Linux

Preferred workaround

In my opinion, there are two equally acceptable ways to mitigate this issue:

  1. Remove the "clever" JS popup transform completely, use the old layout (which is currently shown when JS is disabled)
  2. Re-implement the popup thing in pure CSS, without JS (there are lots of ways to do so)
@mauriciofauth

This comment has been minimized.

Show comment
Hide comment
@mauriciofauth

mauriciofauth Dec 22, 2017

Member

Duplicate of #13694

Member

mauriciofauth commented Dec 22, 2017

Duplicate of #13694

@mauriciofauth mauriciofauth self-assigned this Dec 22, 2017

@mauriciofauth mauriciofauth marked this as a duplicate of #13694 Dec 22, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment