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

Add RiiConnect24 DNS Server Wiimmfi method #89

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

spellboundtriangle
Copy link
Contributor

This pull request adds an additional method of working around ISPs blocking custom DNSes, using the RiiConnect24 DNS Server.

This also adds a note to the manual patch method about the requirement to be able to load ROMs from an SD card.

@NightScript370
Copy link
Member

I don't like having an entire header-based text which is just a pointer to a different website. Wouldn't something like this be better in an info box?

Now, let's say that we do end up saying that this section of text is needed. My problem with this, though, is that it is placed on a page that was really meant to have only two sections. Now, having 1/2 of the section being irrelevant to the overall majority is fine to have an increased page height, but now we're adding something that would make 2/3s of the page irrelevant to the general population. I would honestly prefer if these alternative methods were hidden via tabs instead, because at least those that wouldn't use it would only have a tiny page height increase.
This is important, because people are overly judgemental of long text, even if the text itself tells them that the other part of it is irrelevant to them. As such, on the wiimmfi page, if it looks exceptionally long, people are going to dismiss this as a tedious process and give out without trying. This isn't a problem for when only 1/2 of the text is relevant to the majority, but here, it's now been minimized to 1/3rd...

Opinions are welcome to combat mine.

@lifehackerhansol
Copy link
Member

I'm not quite in agreement with the "remove text" part. Most of these are valid forewording that really should exist in RC24's readme but I digress.

Tabs would be a welcome addition. The actual written content LGTM.

@NightScript370
Copy link
Member

@larsenv, would you be able to update the ReadME to include this extra text so that we ourselves don't need to provide it in such a big potentially-info box?

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.

3 participants