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

feat: Add option to open links in different targets #2282

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Yoyasp
Copy link
Contributor

@Yoyasp Yoyasp commented Oct 30, 2024

Description

This will allow for links to be opened in different HTML targets meaning users can follow navigation links without leaving the current page they are on. This can be usefull if you want to guide them to external tools.

So far I have limited it to the add_link() function, but this can easily be expended to add_view() aswell

This was requested by @gugupy

ADDITIONAL INFORMATION


appbuilder.add_link("Flask documentation", href="https://flask-appbuilder.readthedocs.io", icon="fa-book", target="_blank")

app.run(host="0.0.0.0", port=8080, debug=True)

Check failure

Code scanning / CodeQL

Flask app is run in debug mode High

A Flask app appears to be run in debug mode. This may allow an attacker to run arbitrary code through the debugger.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant