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

Empty accounts list message doesn't correctly reflect the UI #1410

Closed
MMK21Hub opened this issue Jul 20, 2023 · 6 comments · Fixed by #1551
Closed

Empty accounts list message doesn't correctly reflect the UI #1410

MMK21Hub opened this issue Jul 20, 2023 · 6 comments · Fixed by #1551
Labels
bug Something isn't working junior jobs Easy tasks, perfect for new contributors

Comments

@MMK21Hub
Copy link
Contributor

MMK21Hub commented Jul 20, 2023

Operating System

Linux

Version of Prism Launcher

PrismLauncher 7.1
34d80a8

Version of Qt

5.12.8

Description of bug

When viewing the Accounts tab of Settings, and there aren't any registered accounts, a message is shown to instruct the user on how to add an account:

Welcome!
If you're new here, you can click the "Add" button to add your Mojang or Minecraft account.

However, this message references "The "Add" button", which isn't a button that exists in the Accounts UI. In addition, it doesn't mention Microsoft (MS) accounts, which are now used by the majority (?) of players. For context, there are three different buttons for adding accounts: Add Microsoft, Add Mojang and Add Offline.

Since legacy Minecraft accounts are increasingly rare (if they even still function), and Offline accounts require an Online account to be added first, it seems sensible to mention Mojang and MS accounts in this message. For example:

Welcome!
If you're new here, click the "Add Microsoft" button to sign in, or "Add Mojang" to use an old Mojang account.

The phrasing above points the user towards MS accounts as a primary option, which are used by "millions" of players. As an alternative, the option below is more similar to the current message (but a bit more verbose):

Welcome!
If you're new here, you can click the "Add Microsoft" button to add your Microsoft account, or "Add Mojang" to add your Mojang account.

Steps to reproduce

  1. Hide your registered accounts by renaming PrismLauncher/accounts.json to PrismLauncher/accounts.json.bak
  2. Open Prism Launcher
  3. Navigate to File > Settings > Accounts

Suspected cause

I assume that at some point in the past, the launcher simply had an "Add" button to add a Mojang/Minecraft account, but this was re-done when support for Microsoft accounts was added, without updating the new-user message.

The message is defined in AccountListPage.cpp.

Screenshot

A screenshot of Prism Launcher's Accounts page

@MMK21Hub MMK21Hub added the bug Something isn't working label Jul 20, 2023
@TheKodeToad TheKodeToad added the junior jobs Easy tasks, perfect for new contributors label Jul 20, 2023
@MMK21Hub
Copy link
Contributor Author

I've just realised that Mojang accounts do, in fact, still function. I've updated the issue description accordingly.

@TheKodeToad
Copy link
Member

This is a very easy task! if you have any git/github knowledge it'd be appreciated if you could change the string :P

@MMK21Hub
Copy link
Contributor Author

Sounds good - I'll take a look at making a PR! I'll use the first suggested message unless y'all have another preference.

@TheKodeToad
Copy link
Member

TheKodeToad commented Jul 23, 2023

Maybe

Welcome! If you're new here, you can click the "Add Microsoft" or "Add Mojang" button to log in.

@MMK21Hub
Copy link
Contributor Author

That sounds good!

@dannydorazio
Copy link

dannydorazio commented Aug 21, 2023

PR created— used grep to search through files and find two other instances of "minecraft account".
#1551

@Trial97 Trial97 linked a pull request Aug 21, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working junior jobs Easy tasks, perfect for new contributors
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants