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

chore(import): display warning on large gaplimit values #658

Merged
merged 5 commits into from
Sep 12, 2023

Conversation

theborakompanioni
Copy link
Collaborator

Following changes should improve user experience and prevent some mistakes while importing wallets by adding visual feedback:

  • Adds a max blockheight value: 10_000_000 - just to prevent errors during import (should be lower than the current blockheight anyway)
  • Adds a max gaplimit value: 10_000 - rather high , but users should not be prevented from doing it, if they really want to
  • Warn on "large" gaplimit values: any value greater than 250 (arbitrarily chosen) will display a warning

📸

@theborakompanioni theborakompanioni added the UI/UX Issue related to cosmetics, design, or user experience label Sep 11, 2023
@theborakompanioni theborakompanioni self-assigned this Sep 11, 2023
@theborakompanioni theborakompanioni force-pushed the chore/warn-on-large-gaplimit branch from afd7990 to 85a3b51 Compare September 11, 2023 12:33
@editwentyone
Copy link

ACK

@theborakompanioni
Copy link
Collaborator Author

Will change wording from "an extreme decline of performance and responsiveness is to be expected" to something like "can lead to a decline of performance and responsiveness".

@theborakompanioni theborakompanioni merged commit 313c077 into master Sep 12, 2023
@theborakompanioni theborakompanioni deleted the chore/warn-on-large-gaplimit branch September 12, 2023 09:37
Copy link

@editwentyone editwentyone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perfect

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI/UX Issue related to cosmetics, design, or user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants