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

More links respect the new window/tab configuration. #653

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

poacherone
Copy link
Collaborator

@poacherone poacherone commented May 7, 2020

More links in the resources, organizations and licensing modules now use the configuration (common/configuration.ini) whether to open new windows/tabs or not. This continues the work of PR #651 adding most links within the CORAL modules.

External urls, as well as documents, attachments etc. still open new windows. The one exception is the expression comparison in licensing, where the nature of the functionality suggest multiple windows, so I have left that unchanged. But am willing to discuss that further, if anyone disagrees!

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