-
Notifications
You must be signed in to change notification settings - Fork 362
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
Make author names in forum quotes clickable #7363
Comments
This could be part of #4845. |
@Samq64 I don't think so.
There is no reason that people who want clickable author names in quotes will also want clickable mentions, so there's really no point in merging them.
Putting both in more-links might not be a bad idea though. Both these addons serve the same purpose as more-links - they make things that could be links, but are not links, links.
|
|
Also, how we will handle non-user quote names? Like |
The best solution would be to not turn non-usernames into links. |
Maybe a TOLORS user exists, that's the problem. It would be very unpredictable. How do we know we're quoting TOLORS the post or TOLORS the user? |
Then the quote is ambiguous anyways. Someone who doesn't know the abbreviation might actually think you're quoting the TOLORS user. This problem wouldn't exist if you'd explicitly say "The Official List Of Rejected Suggestions", which isn't a valid username. |
Hmm, this is ok, but it's hard to know where it would fit... |
I've implemented it as a new addon, but I think it would fit well into more-links. |
I'll just open the pull request if you're fine with the idea, I can put the code into an existing addon later. |
This feels like #4845 all over again where most people agree it's a nice feature to have but no one can agree where to put it. |
Let's style it normally so it's unobtrusive for non-users. |
Why this would be helpful
This will make it easier to go to the profile of a user whose post was quoted.
How this addon works
It turns the author names in forum quotes into links:
Please excuse my choice of example post
It also has an option for hidden links:
Additional context
I already have this implemented, and I'm ready to create a pull request if this gets accepted.
The text was updated successfully, but these errors were encountered: