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

Question: start jar numbering at 1? #416

Closed
dergigi opened this issue Jul 20, 2022 · 4 comments
Closed

Question: start jar numbering at 1? #416

dergigi opened this issue Jul 20, 2022 · 4 comments
Labels
concept Wild idea, or too many details unknown yet question Further information is requested UI/UX Issue related to cosmetics, design, or user experience

Comments

@dergigi
Copy link
Contributor

dergigi commented Jul 20, 2022

Currently, the default of 5 jars is displayed as follows:

  • Jar #0
  • Jar #1
  • Jar #2
  • Jar #3
  • Jar #4

While this is in accordance to JoinMarket internals, and fine for nerds and computer science people, it's probably confusing for newcomers.

Question: should we use the more natural numbering of 1 to 5?

  • Jar #1
  • Jar #2
  • Jar #3
  • Jar #4
  • Jar #5

I'm talking about a change in the user-facing UI only, internally numbering will start at zero, of course.

@dergigi dergigi added question Further information is requested concept Wild idea, or too many details unknown yet UI/UX Issue related to cosmetics, design, or user experience labels Jul 20, 2022
@dergigi dergigi added this to the v0.1.0 - Tasty Parity milestone Jul 20, 2022
@ghost
Copy link

ghost commented Jul 20, 2022

Definitely start at 1! 👍

We just have to be careful to not confuse our UI indexes with the actual ones. Sweeping the forth (zero-indexed) jar when the user actually wanted to sweep the third one would be bad. 😅

@stachrom
Copy link

I would strongly suggest to keep the Numbers:
Jar #0
Jar #1
Jar #2
Jar #3
Jar #4

If we disconnect from the internal numbering - i fear truble.

Every now and than you need to go back to the cli and there you will find the "first Jar" is labeled as 0.
https://github.com/JoinMarket-Org/joinmarket-clientserver/blob/master/docs/USAGE.md#funding-wallet-and-displaying-balance

@dergigi
Copy link
Contributor Author

dergigi commented Jul 25, 2022

Every now and than you need to go back to the cli

That might be true now, but I hope that it won't be true in the future. Hmm. 🤔

@ghost
Copy link

ghost commented Aug 12, 2022

Closing this in favor of #435.

@ghost ghost closed this as completed Aug 12, 2022
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
concept Wild idea, or too many details unknown yet question Further information is requested UI/UX Issue related to cosmetics, design, or user experience
Projects
None yet
Development

No branches or pull requests

2 participants