Skip to content
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

Using Command+Click to open in new tab does not work (Firefox/Safari) #15225

Closed
ftrotter opened this issue Apr 25, 2019 · 7 comments · Fixed by #17067
Closed

Using Command+Click to open in new tab does not work (Firefox/Safari) #15225

ftrotter opened this issue Apr 25, 2019 · 7 comments · Fixed by #17067
Assignees
Labels
Bug A problem or regression with an existing feature has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete ui Issues relating to the user interface
Projects
Milestone

Comments

@ftrotter
Copy link

ftrotter commented Apr 25, 2019

Describe the bug

When clicking a table in the database/table tree (on the left side of the web interface) will open that tables "browse" interface (sql.php).

Clicking the same link with the command key (Ctrl on windows) SHOULD result in this same page opening in a new tab. Instead, the table is opened in the current web-browser tab.

On OSX, this impacts both safari and firefox. But Chrome works correctly.

Given that this is low-level browser functionality, I am at a loss about how this happening, and phpmyadmin is the only web interface that seems to work this way.

Edit: I just confirmed that this problem happens when there are no extensions loaded in Firefox or in Safari.

To Reproduce

Steps to reproduce the behavior:

  1. Using Safari or Firefox on OSX, Go to https://demo.phpmyadmin.net/master-config/
  2. Click on any database to show a list of tables in that database
  3. Command+click on a table name
  4. The link will open in the current tab

Expected behavior

The link should open in a new tab, as the user has indicated.

Screenshots

No screenshots.

Server configuration

  • Operating system: My server uses Ubuntu and I am replicating the bug on the public demo
  • Web server: nginx/1.10.3 and Apache/2.4.18 (Ubuntu)
  • Database version: 8.0.3-rc-log - MySQL Community Server (GPL) and 10.2.22-MariaDB-10.2.22+maria~xenial-log - mariadb.org binary distribution
  • PHP version: 7.2.15-1+ubuntu16.04.1+deb.sury.org+1 and 7.2.16-1+020190307202415.17+stretch1.gbpa7be82
  • phpMyAdmin version: 4.8.5 and 4.7.7 seem affected

Client configuration

  • Browser: Firefox version 66.03 and Safari 12.1
  • Operating system: OSX Mojave: macOS 10.14.4 (18E226)

Additional context

I have been able to find no other website that fails to respect this command+click functionality. I think this might be equvilent to the following bug:

#14270

@williamdes williamdes added Bug A problem or regression with an existing feature ui Issues relating to the user interface labels Apr 25, 2019
@Drakar2007
Copy link

The difference with respect to 14270 (and its few duplicates including #14363) is that right-click and open on foreign keys used to work (recently), and as far as I've seen CTRL+click has never really worked; i've gotten used to right-clicking and selecting "open in new tab", which of course is now broken as mentioned in those other tickets.

@williamdes williamdes added this to High priority in issues May 1, 2019
@williamdes williamdes moved this from High priority to ready to merge in issues Jun 12, 2019
@Drakar2007
Copy link

@williamdes - FWIW, your fix #15325 does not appear to fix the specific issue originally reported in this thread by @ftrotter - which is specifically that ctrl+click doesn't work right. I've tested your fix on the server you posted in the other thread and right-click & open in new tab / middle click DOES work (which is what i was most hoping for), but apparently ctrl+click is unaffected. (Though since it never functioned that way in any versions i've seen, i'm not sure it's possible either).

@williamdes
Copy link
Member

Though since it never functioned that way in any versions i've seen, i'm not sure it's possible either

Okay, thanks for the feedback !
I will maybe investigate :)

issues automation moved this from ready to merge to Closed Jun 22, 2019
@williamdes williamdes reopened this Jun 22, 2019
issues automation moved this from Closed to Needs triage Jun 22, 2019
@williamdes williamdes moved this from Needs triage to Reproduced in issues Jun 22, 2019
@williamdes
Copy link
Member

@ftrotter can you please try last version and explain what is still broken ?

@ftrotter
Copy link
Author

It will take me some time to come around to this.. but I will add it to our testing plan.

@ftrotter
Copy link
Author

I have just confirmed that the problem still exists using firefox, safari and chrome on the current version of OSX on the demo available at https://demo.phpmyadmin.net/master-config/

So it appears this is still a problem.

@williamdes williamdes self-assigned this Aug 10, 2021
@williamdes williamdes added this to the 5.1.2 milestone Aug 10, 2021
@williamdes williamdes added the has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete label Aug 10, 2021
williamdes added a commit that referenced this issue Aug 10, 2021
…e tab

Pull-request: #17067
Fixes: #15225

Signed-off-by: William Desportes <williamdes@wdes.fr>
williamdes added a commit that referenced this issue Aug 10, 2021
Signed-off-by: William Desportes <williamdes@wdes.fr>
issues automation moved this from Reproduced to Closed Aug 10, 2021
@williamdes
Copy link
Member

williamdes commented Aug 10, 2021

Hi everybody, this will be part of the release 5.1.2
If anyone can test and confirm that it works using the latest 5.1 version in development (phpMyAdmin 5.1+snapshot) ?

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug A problem or regression with an existing feature has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete ui Issues relating to the user interface
Projects
issues
  
Closed
Development

Successfully merging a pull request may close this issue.

3 participants