-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
fix: Order Page: Remove empty spaces #5561
fix: Order Page: Remove empty spaces #5561
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/eventyay/open-event-frontend/7vlcssx8x |
726eab6
to
54cb07c
Compare
Codecov Report
@@ Coverage Diff @@
## development #5561 +/- ##
============================================
Coverage 23.71% 23.71%
============================================
Files 498 498
Lines 5255 5255
Branches 44 44
============================================
Hits 1246 1246
Misses 4003 4003
Partials 6 6 Continue to review full report at Codecov.
|
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.
Please keep "Download tickets" and "Download Invoice" in the right side column.
47498d5
to
5da52c8
Compare
@mariobehling @iamareebjamal sir, implemented suggested changes. This PR now solves #5566 #5559 both |
Before, the buttons spanned the width of entire right column, now they don't |
@iamareebjamal sir, is it good. please review. |
app/styles/partials/overrides.scss
Outdated
@@ -82,3 +82,8 @@ body.dimmable.undetached.dimmed { | |||
margin-left: auto !important; | |||
margin-top: auto !important; | |||
} | |||
|
|||
.display-flex { |
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.
d-flex class is already defined. Just add a class for space between
app/templates/orders/view.hbs
Outdated
<Orders::EventInfo | ||
@data={{this.model.order}} /> | ||
<br/> | ||
<div class="{{if (eq this.device.isMobile false) 'display-flex'}}"> |
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.
Why do you need to check that it is not mobile?
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.
@iamareebjamal sir, if we implement this class at mobile device, buttons are not stacking on one another.
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.
Looks good, no need for stacking
Fixes #5559 #5566
Short description of what this resolves:
Remove spaces between boxes. Enhance mobile view.
Changes proposed in this pull request:
Checklist
development
branch.screenshots
before
after