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

Changed _UNKNOWN_ to _MACHINA_ #638

Merged
merged 3 commits into from
Jun 6, 2023
Merged

Conversation

Heistergand
Copy link
Contributor

@Heistergand Heistergand commented Jun 5, 2023

Changed unicode decorated text UN̴̖̈KNȮWṈ to simple text __MACHINA__.

Following the In-Game story, the unknown faction was given the name __MACHINA__, shown in scanners and stock intel as _̶̱̍_MĄ̷̍CHḬNÄ__ .

According to the discussion in the Telegram IITC Dev channel, it is more important that the user interface is easy to use. Better readability contributes to this.

@github-actions
Copy link

github-actions bot commented Jun 5, 2023

🤖 Pull request artifacts

file commit
IITC_Mobile-test.apk da18935
test-0.35.0.20230605.205023.zip da18935

See build on website

@modos189
Copy link
Contributor

modos189 commented Jun 5, 2023

Please change to plain __MACHINA__

That's why:

  1. In the developers telegram chat most people like the readable font
  2. I think it is more important: that the portal owner and the owner of the resonators have the same text (up: this change does not affect the name of the owner)

@xscreach
Copy link
Contributor

xscreach commented Jun 5, 2023

Imho the only valid reason to make this change is that stock intell now shows the layer as weird machina too (it was "unknown" before, not sure when that changed).

Even though or especially because it concerns only a comment here.
According to the discussion in the telegram IITC Dev channel, it is more important that the user interface is easy to use. Better readability contributes to this.
@modos189 modos189 merged commit 0c7f2a9 into IITC-CE:master Jun 6, 2023
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