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

Minor extension UI fixes #653

Merged
merged 4 commits into from
Mar 6, 2024
Merged

Minor extension UI fixes #653

merged 4 commits into from
Mar 6, 2024

Conversation

emccorson
Copy link
Contributor

@emccorson emccorson commented Mar 4, 2024

Fixed

  • Make enter key submit form on import keys page
  • Stop yellow leaking in progress indicator
  • Capitalize seed phrase confirmation message

Changed

  • Make seed phrase warning fast in development

Progress indicator before:

2024-03-05-153455_620x244_scrot

After:

2024-03-05-153554_646x218_scrot

Copy link
Contributor

github-actions bot commented Mar 4, 2024

@github-actions github-actions bot temporarily deployed to pull request March 4, 2024 06:15 Inactive
@emccorson emccorson force-pushed the fix/minor-extension-ui-fixes branch from 0bdbe94 to 71383ff Compare March 5, 2024 02:58
@github-actions github-actions bot temporarily deployed to pull request March 5, 2024 03:02 Inactive
@github-actions github-actions bot temporarily deployed to pull request March 5, 2024 04:13 Inactive
emccorson added a commit that referenced this pull request Mar 5, 2024
The form contains two buttons and unless the type attribute is
explicitly set, the default behaviour is type="submit" which means
pressing the enter key clicks the first (and wrong) one.
@emccorson emccorson force-pushed the fix/minor-extension-ui-fixes branch from 22cdd0a to e897e21 Compare March 5, 2024 06:07
@emccorson emccorson marked this pull request as ready for review March 5, 2024 06:08
@github-actions github-actions bot temporarily deployed to pull request March 5, 2024 06:12 Inactive
Copy link
Collaborator

@pedrorezende pedrorezende left a comment

Choose a reason for hiding this comment

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

LGTM!

@@ -21,7 +24,7 @@ export const SeedPhraseWarning = ({
if (countdown > 0) {
setCountdown((countdown) => countdown - 1);
}
}, 1000);
}, TIMEOUT);
Copy link
Collaborator

Choose a reason for hiding this comment

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

🙏

The form contains two buttons and unless the type attribute is
explicitly set, the default behaviour is type="submit" which means
pressing the enter key clicks the first (and wrong) one.
@emccorson emccorson force-pushed the fix/minor-extension-ui-fixes branch from e897e21 to de7b931 Compare March 6, 2024 03:42
@github-actions github-actions bot temporarily deployed to pull request March 6, 2024 03:47 Inactive
@emccorson emccorson merged commit de7b931 into main Mar 6, 2024
8 checks passed
seleniumforest pushed a commit to seleniumforest/namada-interface that referenced this pull request Apr 11, 2024
The form contains two buttons and unless the type attribute is
explicitly set, the default behaviour is type="submit" which means
pressing the enter key clicks the first (and wrong) one.
seleniumforest pushed a commit to seleniumforest/namada-interface that referenced this pull request Apr 11, 2024
seleniumforest pushed a commit to seleniumforest/namada-interface that referenced this pull request Apr 11, 2024
seleniumforest pushed a commit to seleniumforest/namada-interface that referenced this pull request Apr 11, 2024
@mateuszjasiuk mateuszjasiuk deleted the fix/minor-extension-ui-fixes branch September 13, 2024 09:32
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