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 repo_button nav link in a new tab/window #846

Open
CedricReichenbach opened this issue Feb 25, 2021 · 3 comments
Open

Open repo_button nav link in a new tab/window #846

CedricReichenbach opened this issue Feb 25, 2021 · 3 comments

Comments

@CedricReichenbach
Copy link

Is this a BUG REPORT or FEATURE REQUEST?:
feature

What happened: repo_button links open in the same tab.

What you expected to happen: It would be nice to be able to specify if they should be opened in a new tab/window (i.e. target=_blank).

How to reproduce it (as minimally and precisely as possible):
Configure a repo_button on a "nav" fragment, then click it.

Environment:

  • Syna Theme version: Tested in 0.17.4
@stp-ip
Copy link
Member

stp-ip commented Feb 26, 2021

Generally I think this might be something to use an edited "nav" fragment as a custom fragment.
There are already quite a few configuration options and enabling link target for all links seems too open.

Happy to discuss this so.

@CedricReichenbach
Copy link
Author

enabling link target for all links seems too open

I'm not sure I fully understand; would this mean all non-content/markdown links would be configurable with a target? In fact, I think this would be a good thing, as long as the default is leaving it unset.

For the repo link in particular, one could even argue to make it target="_blank" altogether, but obviously this would be a change in behavior and certainly some people would disagree.

@stp-ip
Copy link
Member

stp-ip commented Feb 28, 2021

Adding that option multiplies the varieties of the options and complicates the code as well. Currently my stance is that the complexity for that specific feature isn't warranted especially as once can duplicate the navigation fragment and add it in as a custom fragment instead.
See:
https://about.okkur.org/syna/docs/fragments/#custom-fragments

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants