-
-
Notifications
You must be signed in to change notification settings - Fork 771
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
bounty: css fix for wonky contributor info #741
Conversation
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.
lgtm pending review by design
Codecov Report
@@ Coverage Diff @@
## master #741 +/- ##
======================================
Coverage 34.4% 34.4%
======================================
Files 101 101
Lines 5610 5610
Branches 647 647
======================================
Hits 1930 1930
Misses 3606 3606
Partials 74 74 Continue to review full report at Codecov.
|
Can you resolve the conflicts here? @thelostone-mc |
@PixelantDesign Are you good with the changes reflected in the attached screenshots? |
Reduced the spacing between names as it look better (discussed offline with @PixelantDesign ) @mbeacom conflicts resolved! we are good to go and fixed up Also which variable naming style are we sticking to ? camelCase / under_score / hyp-hen ? :P |
Description
Issue
Fix
Fixed wonky contributor info
Updates:
Added comma separated info
Checklist
Affected core subsystem(s)
@PixelantDesign