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

Modifying internal links #15

Merged
merged 1 commit into from Aug 21, 2019
Merged

Modifying internal links #15

merged 1 commit into from Aug 21, 2019

Conversation

clockback
Copy link
Contributor

Adding github.com/wesnoth/wesnoth as "internal link"

Adding github.com/wesnoth/wesnoth as "internal link"
@clockback
Copy link
Contributor Author

This is so that citations on wiki.wesnoth.org can link to their respective GitHub files without the icon indicating that it is an external link.

@irydacea
Copy link
Member

GitHub links are external, though. Why should we lie about this fact?

@clockback
Copy link
Contributor Author

clockback commented Aug 21, 2019

It is only an external link in so far as sourceforge.net/projects/wesnoth is an external link. To my knowledge, the special CSS for external links is presumably to indicate that the user who follows the link is aware that they are going to some place that is external to the de facto official Wesnoth domain. The GitHub repository for Wesnoth seems to be within that domain, just as the sourceforge project is, so I don't believe anybody is deceived.

But besides the justification for why it isn't inappropriate, I believe it would be undesirable to have extra CSS styling in a citation list on the wiki.

@irydacea
Copy link
Member

I don't think the citation list argument is particularly relevant. Wikipedia doesn't shy away from including external links in citation lists for example.

I'd like @Vultraz opinion on this. I didn't remember that we made an exception for SF.net.

@clockback
Copy link
Contributor Author

What you mention about Wikipedia is true. At the moment, if I cite a scenario, I format it like this:

A Summer of Storms, The Rise of Wesnoth

The problem is that introducing external links adds two icons. In the first link (to the scenario), it adds "</>" on the left, whereas in the second link (to the campaign), it adds a box with an arrow on the right. See this article for reference.

The icons are different, and though it is hard to articulate, I personally don't like it from a visual standpoint. Of course, if I am alone in this preference, I don't mind leaving things as they are. :)

@GregoryLundberg
Copy link

GregoryLundberg commented Aug 21, 2019

The issue is the definition of "internal" and "external".

From the point of view of the Wiki pages, themselves, SF, GH, etc., are all "external" because they are on different web servers. Technically, even the Forums are "external" because they are not part of the Wiki, itself.

But, from the point of view of the Wesnoth Project, those pages are "internal" since they are official parts of the project, no matter where they reside.

My feeling is the latter is better for our purposes.

Whichever definition is choosen, however, the underlying point from the OP stands: we should, at least, be consistent. If SF is "internal" so is GH; if GH is "external" then SF should be as well.

@irydacea irydacea merged commit 1923560 into wesnoth:master Aug 21, 2019
@irydacea irydacea self-assigned this Aug 21, 2019
@irydacea
Copy link
Member

Going for aesthetics here above everything else since I see that GH links are becoming more common.

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.

None yet

3 participants