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

Attachment to new tab #1994

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

Conversation

lucmuller
Copy link

This pull request implements the code review suggested changes in #1767
I would be glad to see them merged as having the attachement opened in a new tab would be helpful !

@dregad
Copy link
Member

dregad commented Apr 18, 2024

Thanks for your contribution. I'll review when I have some time, and get back to you.

@vboctor
Copy link
Member

vboctor commented Apr 21, 2024

Thanks, @lucmuller, for your PR.

This functionality comes up every few years with different implementations. At one point, we had links open in the current tab with an icon next to them to have the user choose to open in a new tab. We then removed it since it was considered redundant to the browser's native support for opening in the current window vs. the new tab vs. the new window. I'm not sure why we would add this again.

@dregad
Copy link
Member

dregad commented Apr 22, 2024

@vboctor just making sure you're aware, that this is not about URL links (what you seem to be referring to), but links to attachments.

For context, please read https://mantisbt.org/forums/viewtopic.php?f=2&t=25561

@lucmuller
Copy link
Author

The point of my PR is only to respond to code reviews in #1767.

About links, there's already a configuration to handle them :
$g_html_make_links = LINKS_NEW_WINDOW | LINKS_NOOPENER;

This PR would allow a new configuration to handle filedownload in the same way :
$g_attachment_to_new_tab = TRUE;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants