-
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: Speaker Name Shown in Overview Mode #5305
fix: Speaker Name Shown in Overview Mode #5305
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/eventyay/open-event-frontend/nvv8e7b0j |
@mariobehling @iamareebjamal please review this PR. |
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.
Codecov Report
@@ Coverage Diff @@
## development #5305 +/- ##
===============================================
- Coverage 22.78% 22.72% -0.06%
===============================================
Files 491 491
Lines 5245 5245
Branches 37 37
===============================================
- Hits 1195 1192 -3
- Misses 4045 4048 +3
Partials 5 5
Continue to review full report at Codecov.
|
@mariobehling @iamareebjamal i have implemented suggested changes. Please check. Changes:--I have remove space between speaker image icon and speaker name. ScreenshotsBefore After |
@mariobehling @iamareebjamal please approve these changes |
@mariobehling @iamareebjamal i have removed inline CSS. please review this. please suggest any changes if required. |
Plz squash your commits |
bf4b344
to
ed40dab
Compare
reduce-space-between-imageicon-and-name-of-speaker arrange-code-of-sessionitem.hbs remove-inline-CSS
ed40dab
to
0f308e4
Compare
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 to me.
@mariobehling @iamareebjamal please review this PR |
Thank you! |
Fixes #4815
Short description of what this resolves:
In the overview mode, where the sessions are not expanded the speaker names are not shown. This PR resolves it.
Now Speaker name is shown in the overview mode of Public Session Page.
Changes proposed in this pull request:
Checklist
development
branch.Path
/e/:event_id/schedule
Screenshots
Before
After