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

UX bugs #237

Closed
2 of 7 tasks
nickplace opened this issue Nov 15, 2019 · 7 comments
Closed
2 of 7 tasks

UX bugs #237

nickplace opened this issue Nov 15, 2019 · 7 comments
Milestone

Comments

@nickplace
Copy link
Collaborator

nickplace commented Nov 15, 2019

  • Make it more clear that deployment is progressing. Migration confirm could be a dialog? Spinner for pending transactions? Loading bar?
  • Info dialog on migration page offsets page layout (does it still w/ new layouting?)
  • Ignore saved states w/ no data
  • [Object Object] printed in log when Tx deployment fails
  • Clear saved DAO configuration after successful migration.
  • Save references to recently deployed DAOs in localStorage? (Moved Here: Store DAO Deployents #240)
  • Move schemes to first step? (Moved Here: Stepper Reorg (W/ Templates) #244)
@nickplace nickplace changed the title UI Tweaks UX bugs Nov 16, 2019
@dOrgJelli
Copy link
Contributor

@nickplace could you elaborate on "Info dialog on migration page..."?

@dOrgJelli
Copy link
Contributor

Created this issue for "Save references to recently deployed DAOs": #240

@dOrgJelli
Copy link
Contributor

@pimato do you have an opinion on "Move schemes to first step?"

@dOrgJelli
Copy link
Contributor

@nickplace a spinner already exists when you're deploying, do you think that's enough?

@dOrgJelli dOrgJelli added this to the V1.1 milestone Nov 17, 2019
@dOrgJelli
Copy link
Contributor

dOrgJelli commented Nov 18, 2019

@pimato do you have an opinion on "Move schemes to first step?"

It looks like we do want to do this judging by the Beta spec here. I'll make another task that's dependent on DAO templates for moving this to the front of the stepper.

@nickplace
Copy link
Collaborator Author

@nickplace a spinner already exists when you're deploying, do you think that's enough?

That screen could be more clear, IMO. I honestly missed the "Do you want to deploy?" Dialog inside of the Log at first. Saw that the spinner was spinning, assumed no input was needed.

@orishim orishim added the V1 label Nov 30, 2019
@dOrgJelli
Copy link
Contributor

Closing as we've finished UI V2 which should address all of these issues.

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

No branches or pull requests

3 participants