-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Add confirmation dialog after first establishing a monthly tip that indicates amount and date of first tip #3023
Comments
note that there should be letter/logo instead of paper plane, like we have it now |
Will this design also change the look of a regular tip that's not set as recurring? Assuming yes. Could we get a design added into the issue with both recurring and none-recurring implementations? Assuming it will look similar to the above but removing the |
Closing the loop here, both designs for recurring and none-recurring tips are outlined in brave/brave-core#1433. |
@jenn-rhim see @kjozwiak above |
Verification passed on
Used test plan from brave/brave-core#1433 Encountered: #3481 Verification passed on
Verified passed with
|
After a user has set up a monthly tip using the tipping panel, a confirmation dialog should appear in place of the current simple "Thank you" dialog that says thank you, and also indicates the name of the site being tipped, amount of monthly tip, and the date on which the first tip will be sent.
The text was updated successfully, but these errors were encountered: