count_forks option for Lang cards and hide_border option for Repo cards #480
Conversation
countForks default = false Looks like my "Prettier" extension actived too...
Looks like my "Prettier" extension actived too...
Codecov Report
@@ Coverage Diff @@
## master #480 +/- ##
==========================================
- Coverage 91.82% 91.82% -0.01%
==========================================
Files 19 19
Lines 575 587 +12
Branches 159 163 +4
==========================================
+ Hits 528 539 +11
- Misses 41 42 +1
Partials 6 6
Continue to review full report at Codecov.
|
src/cards/repo-card.js
Outdated
| }; | ||
| const svgStars = | ||
| stargazers.totalCount > 0 && | ||
| iconWithLabel(icons.star, totalStars, "stargazers"); | ||
| const svgForks = | ||
| forkCount > 0 && iconWithLabel(icons.fork, totalForks, "forkcount"); | ||
|
|
||
| const starAndForkCount = FlexLayout({ | ||
| items: [svgStars, svgForks], | ||
| gap: 65 | ||
| }).join(""); | ||
|
|
||
| const card = new Card({ | ||
| title: header, | ||
| titlePrefixIcon: icons.contribs, | ||
| width: 400, | ||
| height, | ||
| colors: { | ||
| titleColor, | ||
| textColor, | ||
| iconColor, | ||
| bgColor | ||
| } | ||
| }); | ||
|
|
There was a problem hiding this comment.
Please make sure nothing else changes other than the code you want. all the code is messed up. make sure your codeeditor does not have wrong tabs/space configs
There was a problem hiding this comment.
Fixed the spacings
|
Hey @Bas950 i really would appreciate if you can add the exclude_repo feature on a different PR, let's add one feature per PR. |
Alright, will make different PRs for each feature |
/api/top-langs/?username=USER
/api/top-langs/?username=USER&count_forks=true
/api/pin/?username=USER&repo=REPO
/api/pin/?username=USER&repo=REPO&hide_border=true
(Hard to see... but it is removed.)
/api/top-langs/?username=USER&count_forks=true
/api/top-langs/?username=USER&count_forks=true&exclude_repo=REPO
Added prettier configs so users don't change the whole spacings.
closes #468