Red Hat Bugzilla – Bug 1470998
"Open link in new tab" in right click menu jumps to home page
Last modified: 2018-01-25 17:06:27 EST
Description of problem:
For some links, "Open link in new tab" in right click menu jumps to home page. But left click returns correct destination page.
Version-Release number of selected component (if applicable):
Steps to Reproduce:
1. Create pods
$ oc new-app --file=https://raw.githubusercontent.com/openshift/origin/master/examples/sample-app/application-template-stibuild.json
2. In overview page, left click the donut of database pod (note: don't expand the app item first. Same below)
3. In overview page, right click the donut of database pod, click "Open link in new tab"
2. Go to the pod page for database pod
3. Go to home page "https://master/console/"
3. Should go to correct destination page as step 2
This bug also exists on the Monitoring page as the mini donut wrapped by an anchor element is used there as well. The bug occurs because the mini donut markup has an href attribute with an empty value in addition to an ng-click that does the right-click navigating. I noticed we do not include an href attribute on the full-size donut, so it does not have this bug. It seems like we should be consistent between the two different sized donuts.
summarizing discussion we had in person:
- We should update all of the pod donuts on the overview to be linking to a valid URL
- We need to stop going to the pods list filtered by label selector, since there are cases now where that list can actually be incorrect (i.e. it can include pods the controller doesnt actually own)
- We should go to the controller's page instead and scroll to the pods table
- Pod donuts on the controller page should probably no longer be clickable or should just jump to the pods table at the bottom of the page
- A single pod should still go directly to that pod's page since people are use to this behavior as a shortcut (it was a specifically requested shortcut)
- In general, when we have links that are for navigation we need to be using URLs, if we have links that are not for navigation they should not be <a> tags but should be <button> instead. If we want them to look like links we can use the btn-link class. This approach is the correct semantic tag usage as far as accessibility is concerned, and eliminates this issue where right-click -> open in new tab and ctrl(cmd)+click do not do the right thing.