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

[redesign] Purchase tickets views #3349

Merged
merged 21 commits into from
Apr 14, 2021

Conversation

bgptr
Copy link
Collaborator

@bgptr bgptr commented Mar 30, 2021

Closes #3286
Closes #3344
Closes #3384

In this diff, I've implemented the new design of Purchase tickets views. Above that:

  • add shared PurchaseTicketsForm ,AutoBuyerSettingsModal TicketAutoBuyerForm components
  • introduced the PiUiButton component, which is a wrapper for the pi-ui's Button component with decrediton styles.
  • update and expand tests

@bgptr bgptr force-pushed the purchase-tickets-view-update branch 2 times, most recently from a032d3f to 158f933 Compare April 4, 2021 09:00
@bgptr bgptr changed the title [WIP][redesign] Purchase tickets views [redesign] Purchase tickets views Apr 4, 2021
@bgptr bgptr force-pushed the purchase-tickets-view-update branch 2 times, most recently from 0f750bd to c888b11 Compare April 11, 2021 16:30
@bgptr bgptr force-pushed the purchase-tickets-view-update branch from c888b11 to 94b5fb1 Compare April 13, 2021 15:12
Copy link
Member

@matheusd matheusd left a comment

Choose a reason for hiding this comment

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

Looks very nice! Confirmed it fixed the issue in #3384

@alexlyp
Copy link
Member

alexlyp commented Apr 14, 2021

Rebase please

@bgptr bgptr force-pushed the purchase-tickets-view-update branch from 94b5fb1 to 93a3778 Compare April 14, 2021 14:07
@bgptr
Copy link
Collaborator Author

bgptr commented Apr 14, 2021

@alexlyp rebased

@alexlyp alexlyp merged commit 5c36621 into decred:master Apr 14, 2021
@bgptr bgptr deleted the purchase-tickets-view-update branch August 27, 2021 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants