Bug 1470998 - "Open link in new tab" in right click menu jumps to home page
"Open link in new tab" in right click menu jumps to home page
Status: NEW
Product: OpenShift Container Platform
Classification: Red Hat
Component: Management Console (Show other bugs)
Unspecified Unspecified
medium Severity low
: ---
: ---
Assigned To: Jessica Forrester
Depends On:
  Show dependency treegraph
Reported: 2017-07-14 04:52 EDT by Xingxing Xia
Modified: 2018-01-25 17:06 EST (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
Last Closed:
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Xingxing Xia 2017-07-14 04:52:53 EDT
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):
openshift v3.6.144

How reproducible:

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"

Actual results:
2. Go to the pod page for database pod
3. Go to home page "https://master/console/"

Expected results:
3. Should go to correct destination page as step 2

Additional info:
Comment 1 Robb Hamilton 2018-01-03 14:52:16 EST
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.
Comment 2 Robb Hamilton 2018-01-03 14:54:07 EST
Comment 3 Jessica Forrester 2018-01-03 15:49:52 EST
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.

Note You need to log in before you can comment on or make changes to this bug.