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

HWI Bridge design improvement #1015

Merged
merged 12 commits into from Mar 22, 2021
Merged

Conversation

ben-kaufman
Copy link
Contributor

According to the Figma files shared by @GBKS @AlexaAker (BitcoinDesign/Meta#69 for more details)

As you can see in the screenshots below, the main changes are:

  • The HWI Bridge settings screen now has 3 options: local, remote, and advanced.
  • If remote or advanced were chosen, a warning will show up reminding the user to edit their Preferences.
  • HWI whitelist iframe was removed and instead has now a link directing to it, as part of the instructions displayed below the options.
  • The app Preferences screen now hides by default all "Advanced" settings normally unnecessary for users.
  • The main page will now detect if the URL of a user accessing Specter is not local (meaning Specter likely runs remotely), and will direct the user to update the HWI settings - if the remote option there is not already selected.
  • The toolbar at the top right corner now shows a Specter icon to get back to the helper page (and the price settings now colored in blue)
  • All the terminology around the HWI bridge has been simplified to terms like "USB connections", and "Remote/ local" Specter.

Next thing I hope to do is to separate the HWI software to its own packaged app - so it will be possible for users to use Specter remotely with the browser, and have the HWI Bridge app just separately installed. I think this would simplify the instructions for users quite a bit, but needs some more work.

CC:
@GBKS
@ConorOkus
@AlexaAker

Please take a look at the screenshots/ try the changes, and let me know what you think, and if you have any comments or further suggestions.

Screen Shot 2021-03-13 at 10 06 12

Screen Shot 2021-03-13 at 10 40 08

Screen Shot 2021-03-13 at 10 40 24

Screen Shot 2021-03-13 at 10 40 31

Screen Shot 2021-03-13 at 10 41 18

Screen Shot 2021-03-13 at 10 41 47

Screen Shot 2021-03-13 at 10 42 21

Screen Shot 2021-03-13 at 10 42 33

@k9ert
Copy link
Collaborator

k9ert commented Mar 22, 2021

Great work! Special thnaks to @GBKS @AlexaAker and the rest of the UI-team for the great recommendations in multiple iterations on how to improve the UI/UX.
Merging!

@k9ert k9ert merged commit 7e0a035 into cryptoadvance:master Mar 22, 2021
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.

None yet

2 participants