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

Add valid href to navigation links #1034

Merged

Conversation

maxmarkus
Copy link
Contributor

Added configurable option to show href links in the navigation.

@JohannesDoberer JohannesDoberer self-assigned this Jan 8, 2020
@ndricimrr
Copy link
Contributor

ndricimrr commented Jan 10, 2020

I seem to be getting a weird behaviour with the ContextSwitcher. The href shows as 'undefined' for Environment 1,2,3 until 9. They work fine if opened in the same tab but not in new ones.
Looks like this in inspect-element:
<a href="/undefined" class="fd-menu__item is-selected svelte-1ldh2pm" title="Environment 1">Environment 1</a>
Other than that all works fine

Copy link
Contributor

@JohannesDoberer JohannesDoberer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@alexandra-simeonova alexandra-simeonova self-assigned this Jan 14, 2020
@ndricimrr ndricimrr self-assigned this Jan 15, 2020
Copy link
Contributor

@alexandra-simeonova alexandra-simeonova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@maxmarkus maxmarkus merged commit 868b8a9 into SAP:master Jan 27, 2020
@maxmarkus maxmarkus deleted the 1032-add-valid-href-to-navigation-links branch January 27, 2020 11:52
JohannesDoberer added a commit to JohannesDoberer/luigi that referenced this pull request Feb 12, 2020
* master:
  Debug cypress travis issue (SAP#1097)
  Fix communication from external mf using customMessages (SAP#1081)
  fix fd-modal mix-up (SAP#1068)
  who is using luigi in about page (SAP#1077)
  Add cobrick logo (SAP#1072)
  who is using luigi button (SAP#1066)
  Update README.md
  Update README.md
  release v0.7.4 (SAP#1069)
  fix items calculation in more btn of tab nav(SAP#1065)
  Add valid href to navigation links (SAP#1034)
  fix runtime error when contextSwitcher is not defined (SAP#1047)
  Improve API documentation (SAP#1025)
  Improve authorization doc structure (SAP#1024)
  Improve Overview page (SAP#1000)
  Improve application-setup.md (SAP#965)
  create content guidelines (#877)
  Documentation fixes (SAP#1003)

# Conflicts:
#	docs/luigi-core-api.md
#	test/e2e.sh
stanleychh pushed a commit to stanleychh/luigi that referenced this pull request Dec 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants