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

client: Middle click on title to open entry in new tab + mark entry as read when opening external link #1472

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

PhrozenByte
Copy link
Contributor

@PhrozenByte PhrozenByte commented Nov 23, 2023

This PR includes both #1470 and #1471 plus d7d15dc. Please see the respective PRs to learn about the changes. If you made the decision that you don't want both PRs but only one, just merge this PR - it will automatically close this and the respective other PR.

This PR will cause a merge conflict with #1469 once any of the two is merged. Resolving is easy, just make sure to remove the now unnecessary stopPropagation function.

Fixes #1470
Fixes #1471

Copy link

netlify bot commented Nov 23, 2023

Deploy Preview for selfoss canceled.

Name Link
🔨 Latest commit d7d15dc
🔍 Latest deploy log https://app.netlify.com/sites/selfoss/deploys/655ff28bfc0b2d00084a08b2

@PhrozenByte PhrozenByte marked this pull request as draft November 24, 2023 10:34
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