-
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
feat: improve speaker list page column organization #4824
feat: improve speaker list page column organization #4824
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/eventyay/open-event-frontend/80oc8tzvx |
966cb7f
to
32b4c57
Compare
Codecov Report
@@ Coverage Diff @@
## development #4824 +/- ##
===============================================
+ Coverage 23.48% 23.53% +0.05%
===============================================
Files 481 481
Lines 5085 5085
Branches 16 16
===============================================
+ Hits 1194 1197 +3
+ Misses 3887 3884 -3
Partials 4 4
Continue to review full report at Codecov.
|
Make the icons into "real links" so they can be opened in the browser in a new tab (e.g. using mouse right click) |
@@ -1,11 +1,13 @@ | |||
<div class="ui {{if this.device.isMobile 'horizontal' 'vertical'}} compact basic buttons"> | |||
<UiPopup @content={{t "View"}} @class="ui icon button" @click={{action this.props.actions.viewSpeaker this.record}} @position="left center"> | |||
{{this.record}} |
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.
Extra
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.
This is to show name.
Short description of what this resolves:
Fixes #4821
Changes proposed in this pull request:
Checklist
development
branch.