[pull] master from anuraghazra:master (#80)

* feat: add min width languages card (anuraghazra#1953)

* Add min width to languages card

* Add test for min width languages card

* feat: increase lang card min width

Co-authored-by: Rick Staa <[email protected]>

* ci: add top-issues-dashboard action

This commit adds the top issues dashboard action which creates a issue displaying the top issues of the week.

* ci: fix title of top-issues action

Co-authored-by: Qi Mo Lin <[email protected]>
Co-authored-by: Rick Staa <[email protected]>

Author: 3 people

.github/workflows/top-issues-dashboard.yml

@@ -0,0 +1,20 @@
+name: Update top issues dashboard
+on:
+  schedule:
+    - cron: "0 0 */7 * *"
+
+jobs:
+  showAndLabelTopIssues:
+    name: Update top issues Dashboard.
+    runs-on: ubuntu-latest
+    steps:
+      - name: Run top issues action
+        uses: rickstaa/top-issues-action@v1
+        env:
+          github_token: ${{ secrets.GITHUB_TOKEN }}
+          label: false
+          dashboard: true
+          top_issues: true
+          top_bugs: true
+          top_features: true
+          top_pull_requests: true

src/cards/top-languages-card.js

@@ -13,6 +13,7 @@ const {
 } = require("../common/utils");
 
 const DEFAULT_CARD_WIDTH = 300;
+const MIN_CARD_WIDTH = 230;
 const DEFAULT_LANGS_COUNT = 5;
 const DEFAULT_LANG_COLOR = "#858585";
 const CARD_PADDING = 25;
@@ -24,12 +25,12 @@ const CARD_PADDING = 25;
 /**
  * @param {Lang[]} arr
  */
- const getLongestLang = (arr) =>
- arr.reduce(
-   (savedLang, lang) =>
-     lang.name.length > savedLang.name.length ? lang : savedLang,
-   { name: "", size: null, color: "" },
- );
+const getLongestLang = (arr) =>
+  arr.reduce(
+    (savedLang, lang) =>
+      lang.name.length > savedLang.name.length ? lang : savedLang,
+    { name: "", size: null, color: "" },
+  );
 
 /**
  * @param {{
@@ -261,7 +262,11 @@ const renderTopLanguages = (topLangs, options = {}) => {
     String(langs_count),
   );
 
-  let width = isNaN(card_width) ? DEFAULT_CARD_WIDTH : card_width;
+  let width = isNaN(card_width)
+    ? DEFAULT_CARD_WIDTH
+    : card_width < MIN_CARD_WIDTH
+    ? MIN_CARD_WIDTH
+    : card_width;
   let height = calculateNormalLayoutHeight(langs.length);
 
   let finalLayout = "";
@@ -307,3 +312,4 @@ const renderTopLanguages = (topLangs, options = {}) => {
 };
 
 module.exports = renderTopLanguages;
+module.exports.MIN_CARD_WIDTH = MIN_CARD_WIDTH;

tests/renderTopLanguages.test.js

@@ -103,6 +103,21 @@ describe("Test renderTopLanguages", () => {
     expect(document.querySelector("svg")).toHaveAttribute("width", "400");
   });
 
+  it("should render with min width", () => {
+    document.body.innerHTML = renderTopLanguages(langs, { card_width: 190 });
+
+    expect(document.querySelector("svg")).toHaveAttribute(
+      "width",
+      renderTopLanguages.MIN_CARD_WIDTH.toString(),
+    );
+
+    document.body.innerHTML = renderTopLanguages(langs, { card_width: 100 });
+    expect(document.querySelector("svg")).toHaveAttribute(
+      "width",
+      renderTopLanguages.MIN_CARD_WIDTH.toString(),
+    );
+  });
+
   it("should render default colors properly", () => {
     document.body.innerHTML = renderTopLanguages(langs);