Skip to content

fix: typos, color tags, and rename Midgards to Midgarts#116

Open
SapitoSucio wants to merge 1 commit into
llchrisll:masterfrom
SapitoSucio:some-tiny-fixes
Open

fix: typos, color tags, and rename Midgards to Midgarts#116
SapitoSucio wants to merge 1 commit into
llchrisll:masterfrom
SapitoSucio:some-tiny-fixes

Conversation

@SapitoSucio

Copy link
Copy Markdown
Contributor
  • Changed "Midgards" to "Midgarts" everywhere for consistency
  • Fixed incomplete and duplicated color codes
  • Fixed general typos (like "Corss" to "Cross")
  • Fixed random capitalized words in the middle of sentences
  • Added missing spaces after commas

- Se corrigen códigos de colores incompletos
- Algunos typos ( Corss -> Cross)
- Se cambia Midgards -> Midgarts
- Se agrega espacios después de comas
@reunite-ro

Copy link
Copy Markdown

woah.. last time.. I think Rune Midgard is the correct one.

@llchrisll

Copy link
Copy Markdown
Owner

Sorry, but this PR will gonna be nightmare to sync with my own file, I still didn't commit my additions to it....
Also I believe I intended Midgarts to be Midgards, can't remember the reason, maybe because some sites use Midgard and others Midgarts:
image
image

@SapitoSucio

Copy link
Copy Markdown
Contributor Author

Sorry, but this PR will gonna be nightmare to sync with my own file, I still didn't commit my additions to it.... Also I believe I intended Midgarts to be Midgards, can't remember the reason, maybe because some sites use Midgard and others Midgarts: image image

No problemo, king. I ask you then, which fix should I remove then aside of the conversion of Midgards to Midgarts?

@SapitoSucio

Copy link
Copy Markdown
Contributor Author

woah.. last time.. I think Rune Midgard is the correct one.

On the official site, it’s written with a "t":

https://ro.gnjoy.com/guide/runeMidgarts/

But after doing a little research, I noticed they sometimes use the "d" and other times the "t". Seems like they can’t make up their mind either xD

image image

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