-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Fix: Order success Change the download tickets to all tickets button #5860
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
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/eventyay/open-event-frontend/p5qu5iacm |
@mariobehling i perfectly fixed the issue |
Codecov Report
@@ Coverage Diff @@
## development #5860 +/- ##
============================================
Coverage 23.46% 23.46%
============================================
Files 512 512
Lines 5460 5460
Branches 63 63
============================================
Hits 1281 1281
Misses 4163 4163
Partials 16 16 Continue to review full report at Codecov.
|
Please keep the position and width of the buttons as before. |
@mariobehling we cannot set size of button it automatically increase or decrees by the content. |
@@ -84,7 +84,7 @@ body.dimmable.undetached.dimmed { | |||
} | |||
|
|||
.space-between { | |||
justify-content: space-between; | |||
justify-content: space-evenly; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can't just change the class content without impacting everywhere the class is used
Fixed already |
Fixes #5852
Short description of what this resolves:
Change the icon as well as fixed the button name see in thee screenshot for the better understanding.
scrennshot
Checklist
development
branch.