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

Open additional menu links in the same tab #275

Closed
zablvit opened this issue Nov 13, 2018 · 0 comments
Closed

Open additional menu links in the same tab #275

zablvit opened this issue Nov 13, 2018 · 0 comments

Comments

@zablvit
Copy link
Contributor

zablvit commented Nov 13, 2018

Requirement - what kind of business use case are you trying to solve?

tl;dr

Possibility to open additional menu links in global nav in the same tab (target="_self")

Detailed use case

I'm using keycloak gatekeeper to add login page to jaeger query and I want to add "Logout" link in top nav which will point to /oauth/logout which will end keycloack session and redirect to login page.

In current state this will open new tab, perform logout and redirect to login screen, however this all happens in a new tab and jaeger ui tab is still open. I need this all to happen in the same tab.

Problem - what in Jaeger blocks you from solving the requirement?

Hardcoded target="_blank" in here

Proposal - what do you suggest to solve the problem or improve the existing situation?

Add openInSameTab boolean field to ConfigMenuItem type which, if set to true will set link target to _self

zablvit added a commit to zablvit/jaeger-ui that referenced this issue Nov 17, 2018
…tracing#275)

Signed-off-by: Vitaliy Zabolotskyy <zablvit@gmail.com>
zablvit added a commit to zablvit/jaeger-ui that referenced this issue Nov 17, 2018
Signed-off-by: Vitaliy Zabolotskyy <zablvit@gmail.com>
tiffon pushed a commit that referenced this issue Jan 9, 2019
* Ability to open additional menu links in same tab (#275)

Signed-off-by: Vitaliy Zabolotskyy <zablvit@gmail.com>

* Add negative test case

Signed-off-by: Vitaliy Zabolotskyy <zablvit@gmail.com>

* Add helper function to create item links

Signed-off-by: Vitaliy Zabolotskyy <zablvit@gmail.com>

* Fix no-use-before-define lint error

Signed-off-by: Vitaliy Zabolotskyy <zablvit@gmail.com>

* Use anchorTarget in custom menu configuration

Signed-off-by: Joe Farro <joef@uber.com>

* Fix typo in test case

Signed-off-by: Joe Farro <joef@uber.com>
everett980 pushed a commit to everett980/jaeger-ui that referenced this issue Jan 16, 2019
…cing#275) (jaegertracing#278)

* Ability to open additional menu links in same tab (jaegertracing#275)

Signed-off-by: Vitaliy Zabolotskyy <zablvit@gmail.com>

* Add negative test case

Signed-off-by: Vitaliy Zabolotskyy <zablvit@gmail.com>

* Add helper function to create item links

Signed-off-by: Vitaliy Zabolotskyy <zablvit@gmail.com>

* Fix no-use-before-define lint error

Signed-off-by: Vitaliy Zabolotskyy <zablvit@gmail.com>

* Use anchorTarget in custom menu configuration

Signed-off-by: Joe Farro <joef@uber.com>

* Fix typo in test case

Signed-off-by: Joe Farro <joef@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
vvvprabhakar pushed a commit to vvvprabhakar/jaeger-ui that referenced this issue Jul 5, 2021
…cing#275) (jaegertracing#278)

* Ability to open additional menu links in same tab (jaegertracing#275)

Signed-off-by: Vitaliy Zabolotskyy <zablvit@gmail.com>

* Add negative test case

Signed-off-by: Vitaliy Zabolotskyy <zablvit@gmail.com>

* Add helper function to create item links

Signed-off-by: Vitaliy Zabolotskyy <zablvit@gmail.com>

* Fix no-use-before-define lint error

Signed-off-by: Vitaliy Zabolotskyy <zablvit@gmail.com>

* Use anchorTarget in custom menu configuration

Signed-off-by: Joe Farro <joef@uber.com>

* Fix typo in test case

Signed-off-by: Joe Farro <joef@uber.com>

Signed-off-by: vvvprabhakar <vvvprabhakar@gmail.com>
vvvprabhakar pushed a commit to vvvprabhakar/jaeger-ui that referenced this issue Jul 5, 2021
…cing#275) (jaegertracing#278)

* Ability to open additional menu links in same tab (jaegertracing#275)

Signed-off-by: Vitaliy Zabolotskyy <zablvit@gmail.com>

* Add negative test case

Signed-off-by: Vitaliy Zabolotskyy <zablvit@gmail.com>

* Add helper function to create item links

Signed-off-by: Vitaliy Zabolotskyy <zablvit@gmail.com>

* Fix no-use-before-define lint error

Signed-off-by: Vitaliy Zabolotskyy <zablvit@gmail.com>

* Use anchorTarget in custom menu configuration

Signed-off-by: Joe Farro <joef@uber.com>

* Fix typo in test case

Signed-off-by: Joe Farro <joef@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>

Signed-off-by: vvvprabhakar <vvvprabhakar@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants