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 description links in new tab #2791

Merged
merged 1 commit into from
Sep 9, 2021
Merged

Open description links in new tab #2791

merged 1 commit into from
Sep 9, 2021

Conversation

fm-sys
Copy link
Contributor

@fm-sys fm-sys commented Feb 8, 2021

Summary

With this change, clicking on links in the description will open them in a new tab...

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits
  • Tests (unit, integration, api and/or acceptance) are included -> not necessary
  • Documentation (manuals or wiki) has been updated or is not required

package.json Outdated Show resolved Hide resolved
Copy link
Member

@juliusknorr juliusknorr left a comment

Choose a reason for hiding this comment

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

See inline

@fm-sys
Copy link
Contributor Author

fm-sys commented Sep 8, 2021

@juliushaertl would you mind another review?

Signed-off-by: Florian Meyer <meyer_florian@magenta.de>
@juliusknorr
Copy link
Member

Thanks a lot, i took the liberty to fix the linting to use tabs instead of spaces and add the noreferrer option as we use that in Nextcloud for all external links.

@fm-sys
Copy link
Contributor Author

fm-sys commented Sep 9, 2021

Thanks a lot, i took the liberty to fix the linting to use tabs instead of spaces and add the noreferrer option as we use that in Nextcloud for all external links.

Oh, sorry. Had done the same fix and forced pushed as well. Will try to revert my force push...

@juliusknorr
Copy link
Member

Ah, I just wondered about that. I'll let you take care then :)

@fm-sys
Copy link
Contributor Author

fm-sys commented Sep 9, 2021

@juliushaertl Force push reverted to commit 6335f50

@juliusknorr juliusknorr merged commit c011c53 into nextcloud:master Sep 9, 2021
@fm-sys fm-sys deleted the md_link_target branch September 9, 2021 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Open description links in new tab
3 participants