diff --git a/ui/e2e/demo.test.js b/ui/e2e/demo.test.js index bd2c33e1f..5a461b413 100644 --- a/ui/e2e/demo.test.js +++ b/ui/e2e/demo.test.js @@ -62,7 +62,7 @@ describe("Demo", () => { }); await expect(labels.length).toBeGreaterThan(10); - await page.click(".modal-header > button.close"); + await page.click(".modal-header > button.btn-close"); }); it("opens silence modal on click", async () => { @@ -76,7 +76,7 @@ describe("Demo", () => { await page.waitForSelector("div.modal-content"); await page.waitForSelector(".modal-body > form"); - await page.click(".modal-header > nav > button.close"); + await page.click(".modal-header > nav > button.btn-close"); }); it("opens settings modal on click", async () => { @@ -88,8 +88,8 @@ describe("Demo", () => { "div.modal-open.components-animation-modal-enter-done" ); await page.waitForSelector("div.modal-content"); - await page.waitForSelector(".modal-body > form.accordion"); + await page.waitForSelector(".modal-body > div.accordion"); - await page.click(".modal-header > nav > button.close"); + await page.click(".modal-header > nav > button.btn-close"); }); }); diff --git a/ui/src/Common/Colors.ts b/ui/src/Common/Colors.ts index 399c7c63f..80efaca82 100644 --- a/ui/src/Common/Colors.ts +++ b/ui/src/Common/Colors.ts @@ -1,6 +1,6 @@ // fallback class for labels const DefaultLabelClassMap = Object.freeze({ - badge: "badge-default components-label-dark", + badge: "bg-default components-label-dark", btn: "btn-default components-label-dark", }); @@ -18,12 +18,12 @@ const BackgroundClassMap = Object.freeze({ }); const StaticColorLabelClassMap = Object.freeze({ - badge: "badge-info components-label-dark", + badge: "bg-info components-label-dark", btn: "btn-info components-label-dark", }); const AlertNameLabelClassMap = Object.freeze({ - badge: "badge-dark components-label-dark", + badge: "bg-dark components-label-dark", btn: "btn-dark components-label-dark", }); diff --git a/ui/src/Components/Accordion/__snapshots__/index.test.tsx.snap b/ui/src/Components/Accordion/__snapshots__/index.test.tsx.snap new file mode 100644 index 000000000..c7fb20db4 --- /dev/null +++ b/ui/src/Components/Accordion/__snapshots__/index.test.tsx.snap @@ -0,0 +1,44 @@ +// Jest Snapshot v1, https://goo.gl/fbAQLP + +exports[` matches snapshot 1`] = ` +" +
+
+

+ +

+
+
+
+
+

+ +

+
+
+ item 2 +
+
+
+
+

+ +

+
+
+
+
+" +`; diff --git a/ui/src/Components/Accordion/index.test.tsx b/ui/src/Components/Accordion/index.test.tsx index eef8f7991..d6b1d27d5 100644 --- a/ui/src/Components/Accordion/index.test.tsx +++ b/ui/src/Components/Accordion/index.test.tsx @@ -1,35 +1,50 @@ import { mount } from "enzyme"; -import { Accordion } from "."; +import toDiffableHtml from "diffable-html"; + +import { Accordion, AccordionItem } from "."; describe("", () => { + it("matches snapshot", () => { + const tree = mount( + + + + + + ); + expect(toDiffableHtml(tree.html())).toMatchSnapshot(); + }); +}); + +describe("", () => { it("doesn't render content by default", () => { - const tree = mount(); + const tree = mount(); expect(tree.text()).toBe("title"); }); it("doesn't render content when defaultIsOpen=false", () => { const tree = mount( - + ); expect(tree.text()).toBe("title"); }); it("renders content when defaultIsOpen=true", () => { const tree = mount( - + ); expect(tree.text()).toBe("titlecontent"); }); it("toggles content after header click", () => { - const tree = mount(); + const tree = mount(); expect(tree.text()).toBe("title"); - tree.find("div.card-header").simulate("click"); + tree.find("button.accordion-button").simulate("click"); expect(tree.text()).toBe("titlecontent"); - tree.find("div.card-header").simulate("click"); + tree.find("button.accordion-button").simulate("click"); expect(tree.text()).toBe("title"); }); }); diff --git a/ui/src/Components/Accordion/index.tsx b/ui/src/Components/Accordion/index.tsx index 650f04850..e8b4e1944 100644 --- a/ui/src/Components/Accordion/index.tsx +++ b/ui/src/Components/Accordion/index.tsx @@ -1,17 +1,6 @@ import { FC, ReactNode, useState } from "react"; -import { ToggleIcon } from "Components/ToggleIcon"; - -const Trigger: FC<{ text: string; isOpen: boolean }> = ({ text, isOpen }) => ( -
-
{text}
-
- -
-
-); - -const Accordion: FC<{ +export const AccordionItem: FC<{ text: string; content: ReactNode; defaultIsOpen?: boolean; @@ -19,16 +8,23 @@ const Accordion: FC<{ const [isOpen, setIsOpen] = useState(defaultIsOpen || false); return ( -
-
setIsOpen(!isOpen)} - > - +
+

+ +

+
+ {isOpen ?
{content}
: null}
-
{isOpen && content}
); }; -export { Accordion }; +export const Accordion: FC = ({ children }) => { + return
{children}
; +}; diff --git a/ui/src/Components/AlertAck/index.tsx b/ui/src/Components/AlertAck/index.tsx index 9c0e65f9a..a49fc4b7b 100644 --- a/ui/src/Components/AlertAck/index.tsx +++ b/ui/src/Components/AlertAck/index.tsx @@ -152,12 +152,12 @@ const AlertAck: FC<{ } > { if (!isAcking && !(response || error)) { diff --git a/ui/src/Components/Fetcher/index.tsx b/ui/src/Components/Fetcher/index.tsx index 3776eab1d..90f7845d0 100644 --- a/ui/src/Components/Fetcher/index.tsx +++ b/ui/src/Components/Fetcher/index.tsx @@ -234,7 +234,7 @@ const Fetcher: FC<{ return (
setIsHover(true)} onMouseLeave={() => setIsHover(false)} > diff --git a/ui/src/Components/Grid/AlertGrid/AlertGroup/Alert/AlertMenu.tsx b/ui/src/Components/Grid/AlertGrid/AlertGroup/Alert/AlertMenu.tsx index 9d9cb705b..ac2427064 100644 --- a/ui/src/Components/Grid/AlertGrid/AlertGroup/Alert/AlertMenu.tsx +++ b/ui/src/Components/Grid/AlertGrid/AlertGroup/Alert/AlertMenu.tsx @@ -129,7 +129,7 @@ const MenuContent: FC<{ } }} > - + Silence this alert
@@ -166,13 +166,13 @@ const AlertMenu: FC<{ {({ ref }) => ( diff --git a/ui/src/Components/Grid/AlertGrid/AlertGroup/Alert/__snapshots__/index.test.tsx.snap b/ui/src/Components/Grid/AlertGrid/AlertGroup/Alert/__snapshots__/index.test.tsx.snap index e297feda5..60c224b4f 100644 --- a/ui/src/Components/Grid/AlertGrid/AlertGroup/Alert/__snapshots__/index.test.tsx.snap +++ b/ui/src/Components/Grid/AlertGrid/AlertGroup/Alert/__snapshots__/index.test.tsx.snap @@ -2,7 +2,7 @@ exports[` matches snapshot when inhibited 1`] = ` " -
  • +
  • @@ -11,7 +11,7 @@ exports[` matches snapshot when inhibited 1`] = ` focusable=\\"false\\" data-prefix=\\"fas\\" data-icon=\\"search-minus\\" - class=\\"svg-inline--fa fa-search-minus fa-w-16 mr-1\\" + class=\\"svg-inline--fa fa-search-minus fa-w-16 me-1\\" role=\\"img\\" xmlns=\\"http://www.w3.org/2000/svg\\" viewbox=\\"0 0 512 512\\" @@ -38,7 +38,7 @@ exports[` matches snapshot when inhibited 1`] = ` focusable=\\"false\\" data-prefix=\\"fas\\" data-icon=\\"search-plus\\" - class=\\"svg-inline--fa fa-search-plus fa-w-16 mr-1\\" + class=\\"svg-inline--fa fa-search-plus fa-w-16 me-1\\" role=\\"img\\" xmlns=\\"http://www.w3.org/2000/svg\\" viewbox=\\"0 0 512 512\\" @@ -53,14 +53,14 @@ exports[` matches snapshot when inhibited 1`] = `
    - matches snapshot when inhibited 1`] = `
    - + matches snapshot when inhibited 1`] = `
    - + job: @@ -103,7 +103,7 @@ exports[` matches snapshot when inhibited 1`] = `
    - + cluster: @@ -139,7 +139,7 @@ exports[` matches snapshot when inhibited 1`] = ` exports[` matches snapshot with showAlertmanagers=false showReceiver=false 1`] = ` " -
  • +
  • @@ -148,7 +148,7 @@ exports[` matches snapshot with showAlertmanagers=false showReceiver=fa focusable=\\"false\\" data-prefix=\\"fas\\" data-icon=\\"search-minus\\" - class=\\"svg-inline--fa fa-search-minus fa-w-16 mr-1\\" + class=\\"svg-inline--fa fa-search-minus fa-w-16 me-1\\" role=\\"img\\" xmlns=\\"http://www.w3.org/2000/svg\\" viewbox=\\"0 0 512 512\\" @@ -175,7 +175,7 @@ exports[` matches snapshot with showAlertmanagers=false showReceiver=fa focusable=\\"false\\" data-prefix=\\"fas\\" data-icon=\\"search-plus\\" - class=\\"svg-inline--fa fa-search-plus fa-w-16 mr-1\\" + class=\\"svg-inline--fa fa-search-plus fa-w-16 me-1\\" role=\\"img\\" xmlns=\\"http://www.w3.org/2000/svg\\" viewbox=\\"0 0 512 512\\" @@ -190,14 +190,14 @@ exports[` matches snapshot with showAlertmanagers=false showReceiver=fa
    - matches snapshot with showAlertmanagers=false showReceiver=fa
    - + job: @@ -222,7 +222,7 @@ exports[` matches snapshot with showAlertmanagers=false showReceiver=fa
    - + cluster: diff --git a/ui/src/Components/Grid/AlertGrid/AlertGroup/Alert/index.test.tsx b/ui/src/Components/Grid/AlertGrid/AlertGroup/Alert/index.test.tsx index 909a5697e..de3a49055 100644 --- a/ui/src/Components/Grid/AlertGrid/AlertGroup/Alert/index.test.tsx +++ b/ui/src/Components/Grid/AlertGrid/AlertGroup/Alert/index.test.tsx @@ -374,7 +374,7 @@ describe("", () => { const tree = MountedAlert(alert, group, false, false, false); expect( tree - .find("span.components-label.badge.badge-secondary.cursor-pointer") + .find("span.components-label.badge.bg-secondary.cursor-pointer") .at(0) .text() ).toBe("just now"); @@ -385,7 +385,7 @@ describe("", () => { }); expect( tree - .find("span.components-label.badge.badge-secondary.cursor-pointer") + .find("span.components-label.badge.bg-secondary.cursor-pointer") .at(0) .text() ).toBe("a few seconds ago"); @@ -396,7 +396,7 @@ describe("", () => { }); expect( tree - .find("span.components-label.badge.badge-secondary.cursor-pointer") + .find("span.components-label.badge.bg-secondary.cursor-pointer") .at(0) .text() ).toBe("1 minute ago"); @@ -407,7 +407,7 @@ describe("", () => { }); expect( tree - .find("span.components-label.badge.badge-secondary.cursor-pointer") + .find("span.components-label.badge.bg-secondary.cursor-pointer") .at(0) .text() ).toBe("1 hour ago"); @@ -418,7 +418,7 @@ describe("", () => { }); expect( tree - .find("span.components-label.badge.badge-secondary.cursor-pointer") + .find("span.components-label.badge.bg-secondary.cursor-pointer") .at(0) .text() ).toBe("2 hours ago"); @@ -429,7 +429,7 @@ describe("", () => { }); expect( tree - .find("span.components-label.badge.badge-secondary.cursor-pointer") + .find("span.components-label.badge.bg-secondary.cursor-pointer") .at(0) .text() ).toBe("2 days ago"); diff --git a/ui/src/Components/Grid/AlertGrid/AlertGroup/Alert/index.tsx b/ui/src/Components/Grid/AlertGrid/AlertGroup/Alert/index.tsx index cafe46c2a..728d93a72 100644 --- a/ui/src/Components/Grid/AlertGrid/AlertGroup/Alert/index.tsx +++ b/ui/src/Components/Grid/AlertGrid/AlertGroup/Alert/index.tsx @@ -41,10 +41,10 @@ const Alert: FC<{ const classNames = [ "components-grid-alertgrid-alertgroup-alert", "list-group-item bg-transparent", - "pl-1 pr-0 py-0", + "ps-1 pe-0 py-0", "my-1", "rounded-0", - "border-left-1 border-right-0 border-top-0 border-bottom-0", + "border-start-1 border-end-0 border-top-0 border-bottom-0", BorderClassMap[alert.state] || "border-default", ]; diff --git a/ui/src/Components/Grid/AlertGrid/AlertGroup/Annotation/__snapshots__/index.test.tsx.snap b/ui/src/Components/Grid/AlertGrid/AlertGroup/Annotation/__snapshots__/index.test.tsx.snap index ad323767d..fb7f97618 100644 --- a/ui/src/Components/Grid/AlertGrid/AlertGroup/Annotation/__snapshots__/index.test.tsx.snap +++ b/ui/src/Components/Grid/AlertGrid/AlertGroup/Annotation/__snapshots__/index.test.tsx.snap @@ -34,7 +34,7 @@ exports[` matches snapshot when visible=false 1`] = ` focusable=\\"false\\" data-prefix=\\"fas\\" data-icon=\\"search-plus\\" - class=\\"svg-inline--fa fa-search-plus fa-w-16 mr-1\\" + class=\\"svg-inline--fa fa-search-plus fa-w-16 me-1\\" role=\\"img\\" xmlns=\\"http://www.w3.org/2000/svg\\" viewbox=\\"0 0 512 512\\" @@ -59,7 +59,7 @@ exports[` matches snapshot when visible=true 1`] = ` focusable=\\"false\\" data-prefix=\\"fas\\" data-icon=\\"search-minus\\" - class=\\"svg-inline--fa fa-search-minus fa-w-16 mr-1\\" + class=\\"svg-inline--fa fa-search-minus fa-w-16 me-1\\" role=\\"img\\" xmlns=\\"http://www.w3.org/2000/svg\\" viewbox=\\"0 0 512 512\\" diff --git a/ui/src/Components/Grid/AlertGrid/AlertGroup/Annotation/index.tsx b/ui/src/Components/Grid/AlertGrid/AlertGroup/Annotation/index.tsx index f35118318..abeb95c0c 100644 --- a/ui/src/Components/Grid/AlertGrid/AlertGroup/Annotation/index.tsx +++ b/ui/src/Components/Grid/AlertGrid/AlertGroup/Annotation/index.tsx @@ -48,7 +48,7 @@ const RenderNonLinkAnnotation: FC<{ onClick={() => setIsVisible(false)} className="cursor-pointer" > - + {name}: ) : ( <> - + {name} )} diff --git a/ui/src/Components/Grid/AlertGrid/AlertGroup/GroupFooter/__snapshots__/index.test.tsx.snap b/ui/src/Components/Grid/AlertGrid/AlertGroup/GroupFooter/__snapshots__/index.test.tsx.snap index 7be5c9f0d..83bd587a8 100644 --- a/ui/src/Components/Grid/AlertGrid/AlertGroup/GroupFooter/__snapshots__/index.test.tsx.snap +++ b/ui/src/Components/Grid/AlertGrid/AlertGroup/GroupFooter/__snapshots__/index.test.tsx.snap @@ -11,7 +11,7 @@ exports[` matches snapshot 1`] = ` focusable=\\"false\\" data-prefix=\\"fas\\" data-icon=\\"search-minus\\" - class=\\"svg-inline--fa fa-search-minus fa-w-16 mr-1\\" + class=\\"svg-inline--fa fa-search-minus fa-w-16 me-1\\" role=\\"img\\" xmlns=\\"http://www.w3.org/2000/svg\\" viewbox=\\"0 0 512 512\\" @@ -38,7 +38,7 @@ exports[` matches snapshot 1`] = ` focusable=\\"false\\" data-prefix=\\"fas\\" data-icon=\\"search-plus\\" - class=\\"svg-inline--fa fa-search-plus fa-w-16 mr-1\\" + class=\\"svg-inline--fa fa-search-plus fa-w-16 me-1\\" role=\\"img\\" xmlns=\\"http://www.w3.org/2000/svg\\" viewbox=\\"0 0 512 512\\" @@ -53,7 +53,7 @@ exports[` matches snapshot 1`] = `
    - + label1: @@ -63,7 +63,7 @@ exports[` matches snapshot 1`] = `
    - + label2: @@ -73,7 +73,7 @@ exports[` matches snapshot 1`] = `
    - + @cluster: @@ -83,7 +83,7 @@ exports[` matches snapshot 1`] = `
    - + @receiver: @@ -128,7 +128,7 @@ exports[` mathes snapshot when silence is rendered 1`] = ` focusable=\\"false\\" data-prefix=\\"fas\\" data-icon=\\"search-minus\\" - class=\\"svg-inline--fa fa-search-minus fa-w-16 mr-1\\" + class=\\"svg-inline--fa fa-search-minus fa-w-16 me-1\\" role=\\"img\\" xmlns=\\"http://www.w3.org/2000/svg\\" viewbox=\\"0 0 512 512\\" @@ -155,7 +155,7 @@ exports[` mathes snapshot when silence is rendered 1`] = ` focusable=\\"false\\" data-prefix=\\"fas\\" data-icon=\\"search-plus\\" - class=\\"svg-inline--fa fa-search-plus fa-w-16 mr-1\\" + class=\\"svg-inline--fa fa-search-plus fa-w-16 me-1\\" role=\\"img\\" xmlns=\\"http://www.w3.org/2000/svg\\" viewbox=\\"0 0 512 512\\" @@ -170,7 +170,7 @@ exports[` mathes snapshot when silence is rendered 1`] = `
  • - + label1: @@ -180,7 +180,7 @@ exports[` mathes snapshot when silence is rendered 1`] = `
    - + label2: @@ -190,7 +190,7 @@ exports[` mathes snapshot when silence is rendered 1`] = `
    - + @cluster: @@ -200,7 +200,7 @@ exports[` mathes snapshot when silence is rendered 1`] = `
    - + @receiver: @@ -255,10 +255,10 @@ exports[` mathes snapshot when silence is rendered 1`] = ` Mocked Silence
    - + — me@example.com - + Expired 14 hours ago
    diff --git a/ui/src/Components/Grid/AlertGrid/AlertGroup/GroupHeader/GroupMenu.tsx b/ui/src/Components/Grid/AlertGrid/AlertGroup/GroupHeader/GroupMenu.tsx index 9552aec0c..3a1b69638 100644 --- a/ui/src/Components/Grid/AlertGrid/AlertGroup/GroupHeader/GroupMenu.tsx +++ b/ui/src/Components/Grid/AlertGrid/AlertGroup/GroupHeader/GroupMenu.tsx @@ -174,7 +174,7 @@ const GroupMenu: FC<{ onClick={toggle} className={`${ themed ? "text-white with-click-light" : "text-muted" - } cursor-pointer badge components-label components-label-with-hover with-click mr-1`} + } cursor-pointer badge components-label components-label-with-hover with-click me-1`} data-toggle="dropdown" > diff --git a/ui/src/Components/Grid/AlertGrid/AlertGroup/GroupHeader/index.tsx b/ui/src/Components/Grid/AlertGrid/AlertGroup/GroupHeader/index.tsx index 6ada9dcce..20bef1780 100644 --- a/ui/src/Components/Grid/AlertGrid/AlertGroup/GroupHeader/index.tsx +++ b/ui/src/Components/Grid/AlertGrid/AlertGroup/GroupHeader/index.tsx @@ -72,7 +72,7 @@ const GroupHeader: FC<{ /> ))} - + {group.stateCount.active > 0 && ( - + {text} ); diff --git a/ui/src/Components/Grid/AlertGrid/AlertGroup/__snapshots__/Silences.test.tsx.snap b/ui/src/Components/Grid/AlertGrid/AlertGroup/__snapshots__/Silences.test.tsx.snap index f31e8a0fa..cdcf81181 100644 --- a/ui/src/Components/Grid/AlertGrid/AlertGroup/__snapshots__/Silences.test.tsx.snap +++ b/ui/src/Components/Grid/AlertGrid/AlertGroup/__snapshots__/Silences.test.tsx.snap @@ -26,10 +26,10 @@ exports[` renders ManagedSilence if silence is present in Alert Mocked Silence
    - + — me@example.com - + Expired 21 years ago
    diff --git a/ui/src/Components/Grid/AlertGrid/AlertGroup/index.test.tsx b/ui/src/Components/Grid/AlertGrid/AlertGroup/index.test.tsx index dcbe33950..9e18adc05 100644 --- a/ui/src/Components/Grid/AlertGrid/AlertGroup/index.test.tsx +++ b/ui/src/Components/Grid/AlertGrid/AlertGroup/index.test.tsx @@ -349,7 +349,7 @@ describe(" renderConfig", () => { tree .find("Alert") .at(0) - .find("span.badge-secondary.cursor-pointer") + .find("span.bg-secondary.cursor-pointer") .at(0) .simulate("click"); await act(() => promise); diff --git a/ui/src/Components/Grid/AlertGrid/Grid.tsx b/ui/src/Components/Grid/AlertGrid/Grid.tsx index f6b4399b6..18b3f72aa 100644 --- a/ui/src/Components/Grid/AlertGrid/Grid.tsx +++ b/ui/src/Components/Grid/AlertGrid/Grid.tsx @@ -192,7 +192,7 @@ const Grid: FC<{ ) } > - + Load more diff --git a/ui/src/Components/Grid/AlertGrid/GridLabelSelect.tsx b/ui/src/Components/Grid/AlertGrid/GridLabelSelect.tsx index d753d25b6..086e0cbdc 100644 --- a/ui/src/Components/Grid/AlertGrid/GridLabelSelect.tsx +++ b/ui/src/Components/Grid/AlertGrid/GridLabelSelect.tsx @@ -159,7 +159,7 @@ const GridLabelSelect: FC<{ useOnClickOutside(ref, hide, isVisible); return ( -
    +
    {({ ref }) => ( diff --git a/ui/src/Components/Grid/AlertGrid/Swimlane.tsx b/ui/src/Components/Grid/AlertGrid/Swimlane.tsx index 3ad220849..297a30149 100644 --- a/ui/src/Components/Grid/AlertGrid/Swimlane.tsx +++ b/ui/src/Components/Grid/AlertGrid/Swimlane.tsx @@ -22,7 +22,7 @@ const Swimlane: FC<{ return (
    - + @@ -48,7 +48,7 @@ const Swimlane: FC<{ /> )} - + diff --git a/ui/src/Components/Grid/ReloadNeeded/__snapshots__/index.test.tsx.snap b/ui/src/Components/Grid/ReloadNeeded/__snapshots__/index.test.tsx.snap index 74e31e607..8fa22c9e2 100644 --- a/ui/src/Components/Grid/ReloadNeeded/__snapshots__/index.test.tsx.snap +++ b/ui/src/Components/Grid/ReloadNeeded/__snapshots__/index.test.tsx.snap @@ -23,7 +23,7 @@ exports[` matches snapshot 1`] = ` focusable=\\"false\\" data-prefix=\\"fas\\" data-icon=\\"spinner\\" - class=\\"svg-inline--fa fa-spinner fa-w-16 fa-spin mr-2\\" + class=\\"svg-inline--fa fa-spinner fa-w-16 fa-spin me-2\\" role=\\"img\\" xmlns=\\"http://www.w3.org/2000/svg\\" viewbox=\\"0 0 512 512\\" diff --git a/ui/src/Components/Grid/ReloadNeeded/index.tsx b/ui/src/Components/Grid/ReloadNeeded/index.tsx index 824ed0c66..bebca5cb9 100644 --- a/ui/src/Components/Grid/ReloadNeeded/index.tsx +++ b/ui/src/Components/Grid/ReloadNeeded/index.tsx @@ -22,7 +22,7 @@ const ReloadNeeded: FC<{ className="screen-center-icon-big text-danger mb-4" />

    - + All API connection attempts failed. This migth be caused by authentication middleware, will try to reload.

    diff --git a/ui/src/Components/Grid/UpgradeNeeded/__snapshots__/index.test.tsx.snap b/ui/src/Components/Grid/UpgradeNeeded/__snapshots__/index.test.tsx.snap index 1948febd8..a702ac40d 100644 --- a/ui/src/Components/Grid/UpgradeNeeded/__snapshots__/index.test.tsx.snap +++ b/ui/src/Components/Grid/UpgradeNeeded/__snapshots__/index.test.tsx.snap @@ -25,7 +25,7 @@ exports[` matches snapshot 1`] = ` focusable=\\"false\\" data-prefix=\\"fas\\" data-icon=\\"spinner\\" - class=\\"svg-inline--fa fa-spinner fa-w-16 fa-spin mr-2\\" + class=\\"svg-inline--fa fa-spinner fa-w-16 fa-spin me-2\\" role=\\"img\\" xmlns=\\"http://www.w3.org/2000/svg\\" viewbox=\\"0 0 512 512\\" diff --git a/ui/src/Components/Grid/UpgradeNeeded/index.tsx b/ui/src/Components/Grid/UpgradeNeeded/index.tsx index 2adb1b65c..ea9f75cc9 100644 --- a/ui/src/Components/Grid/UpgradeNeeded/index.tsx +++ b/ui/src/Components/Grid/UpgradeNeeded/index.tsx @@ -27,7 +27,7 @@ const UpgradeNeeded: FC<{ />

    - + Upgrading to a new version: {newVersion}

    diff --git a/ui/src/Components/InhibitedByModal/InhibitedByModalContent.tsx b/ui/src/Components/InhibitedByModal/InhibitedByModalContent.tsx index 8ddd1c454..8076d4a17 100644 --- a/ui/src/Components/InhibitedByModal/InhibitedByModalContent.tsx +++ b/ui/src/Components/InhibitedByModal/InhibitedByModalContent.tsx @@ -13,9 +13,7 @@ const InhibitedByModalContent: FC<{ <>
    Inhibiting alerts
    - +
    ", () => { const tree = mount( ); - const toggle = tree.find("span.badge.badge-light"); + const toggle = tree.find("span.badge.bg-light"); toggle.simulate("click"); expect(tree.find("InhibitedByModalContent")).toHaveLength(0); expect(tree.find(".modal-content").find("svg.fa-spinner")).toHaveLength(1); @@ -32,7 +32,7 @@ describe("", () => { const tree = mount( ); - const toggle = tree.find("span.badge.badge-light"); + const toggle = tree.find("span.badge.bg-light"); toggle.simulate("click"); expect(tree.find(".modal-title").text()).toBe("Inhibiting alerts"); expect(tree.find(".modal-content").find("svg.fa-spinner")).toHaveLength(0); @@ -42,7 +42,7 @@ describe("", () => { const tree = mount( ); - const toggle = tree.find("span.badge.badge-light"); + const toggle = tree.find("span.badge.bg-light"); toggle.simulate("click"); act(() => { @@ -59,16 +59,16 @@ describe("", () => { expect(tree.find(".modal-title")).toHaveLength(0); }); - it("hides the modal when button.close is clicked", () => { + it("hides the modal when button.btn-close is clicked", () => { const tree = mount( ); - const toggle = tree.find("span.badge.badge-light"); + const toggle = tree.find("span.badge.bg-light"); toggle.simulate("click"); expect(tree.find(".modal-title").text()).toBe("Inhibiting alerts"); - tree.find("button.close").simulate("click"); + tree.find("button.btn-close").simulate("click"); act(() => { jest.runOnlyPendingTimers(); }); @@ -80,7 +80,7 @@ describe("", () => { const tree = mount( ); - const toggle = tree.find("span.badge.badge-light"); + const toggle = tree.find("span.badge.bg-light"); toggle.simulate("click"); expect(document.body.className.split(" ")).toContain("modal-open"); }); @@ -90,10 +90,10 @@ describe("", () => { ); - tree.find("span.badge.badge-light").simulate("click"); + tree.find("span.badge.bg-light").simulate("click"); expect(document.body.className.split(" ")).toContain("modal-open"); - tree.find("span.badge.badge-light").simulate("click"); + tree.find("span.badge.bg-light").simulate("click"); act(() => { jest.runOnlyPendingTimers(); }); @@ -105,7 +105,7 @@ describe("", () => { ); - const toggle = tree.find("span.badge.badge-light"); + const toggle = tree.find("span.badge.bg-light"); toggle.simulate("click"); act(() => { diff --git a/ui/src/Components/InhibitedByModal/index.tsx b/ui/src/Components/InhibitedByModal/index.tsx index e36853f33..7bcbbf53f 100644 --- a/ui/src/Components/InhibitedByModal/index.tsx +++ b/ui/src/Components/InhibitedByModal/index.tsx @@ -27,7 +27,7 @@ const InhibitedByModal: FC<{ <> diff --git a/ui/src/Components/LabelSetList/__snapshots__/index.test.tsx.snap b/ui/src/Components/LabelSetList/__snapshots__/index.test.tsx.snap index 88b15f024..a81e96a5b 100644 --- a/ui/src/Components/LabelSetList/__snapshots__/index.test.tsx.snap +++ b/ui/src/Components/LabelSetList/__snapshots__/index.test.tsx.snap @@ -9,7 +9,7 @@ exports[` matches snapshot with populated list 1`] = `
    • - + foo: @@ -19,7 +19,7 @@ exports[` matches snapshot with populated list 1`] = `
    • - + job: @@ -29,7 +29,7 @@ exports[` matches snapshot with populated list 1`] = `
    • - + instance: @@ -39,7 +39,7 @@ exports[` matches snapshot with populated list 1`] = `
    • - + cluster: diff --git a/ui/src/Components/LabelSetList/index.tsx b/ui/src/Components/LabelSetList/index.tsx index 9148862cb..a803c47b9 100644 --- a/ui/src/Components/LabelSetList/index.tsx +++ b/ui/src/Components/LabelSetList/index.tsx @@ -69,7 +69,7 @@ const LabelSetList: FC<{ />
    ) : ( -
    +

    No alerts matched

    diff --git a/ui/src/Components/Labels/FilterInputLabel/index.test.tsx b/ui/src/Components/Labels/FilterInputLabel/index.test.tsx index 4445a393b..889d72e72 100644 --- a/ui/src/Components/Labels/FilterInputLabel/index.test.tsx +++ b/ui/src/Components/Labels/FilterInputLabel/index.test.tsx @@ -185,7 +185,7 @@ describe(" onChange", () => { /> ); - const button = tree.find("svg.close"); + const button = tree.find("svg.fa-times"); button.simulate("click"); expect(alertStore.filters.values).toHaveLength(1); expect(alertStore.filters.values).toContainEqual( @@ -223,7 +223,7 @@ describe(" counter badge", () => { filter={alertStore.filters.values[0]} /> ); - const counter = tree.find(".badge-pill"); + const counter = tree.find(".rounded-pill"); expect(counter).toHaveLength(0); }); @@ -235,7 +235,7 @@ describe(" counter badge", () => { filter={alertStore.filters.values[0]} /> ); - const counter = tree.find(".badge-pill"); + const counter = tree.find(".rounded-pill"); expect(counter).toHaveLength(1); }); @@ -247,7 +247,7 @@ describe(" counter badge", () => { filter={alertStore.filters.values[1]} /> ); - const counter = tree.find(".badge-pill"); + const counter = tree.find(".rounded-pill"); expect(counter).toHaveLength(1); }); }); diff --git a/ui/src/Components/Labels/FilterInputLabel/index.tsx b/ui/src/Components/Labels/FilterInputLabel/index.tsx index 881aedbc3..1b0af9429 100644 --- a/ui/src/Components/Labels/FilterInputLabel/index.tsx +++ b/ui/src/Components/Labels/FilterInputLabel/index.tsx @@ -53,7 +53,7 @@ const FilterInputLabel: FC<{ alertStore.filters.values.filter( (f) => f.hits !== alertStore.info.totalAlerts ).length > 0 ? ( - {filter.hits} + {filter.hits} ) : null ) : ( @@ -63,7 +63,7 @@ const FilterInputLabel: FC<{ )} alertStore.filters.removeFilter(filter.raw)} /> diff --git a/ui/src/Components/Labels/FilteringCounterBadge/index.test.tsx b/ui/src/Components/Labels/FilteringCounterBadge/index.test.tsx index 21f405a07..70fe8c9fd 100644 --- a/ui/src/Components/Labels/FilteringCounterBadge/index.test.tsx +++ b/ui/src/Components/Labels/FilteringCounterBadge/index.test.tsx @@ -76,17 +76,17 @@ const validateOnClick = ( }; describe("", () => { - it("themed @state=unprocessed counter badge should have className 'badge-secondary'", () => { - validateClassName("unprocessed", "badge-secondary", true); + it("themed @state=unprocessed counter badge should have className 'bg-secondary'", () => { + validateClassName("unprocessed", "bg-secondary", true); }); - it("themed @state=active counter badge should have className 'badge-secondary'", () => { - validateClassName("active", "badge-danger", true); + it("themed @state=active counter badge should have className 'bg-secondary'", () => { + validateClassName("active", "bg-danger", true); }); - it("themed @state=suppressed counter badge should have className 'badge-secondary'", () => { - validateClassName("suppressed", "badge-success", true); + it("themed @state=suppressed counter badge should have className 'bg-secondary'", () => { + validateClassName("suppressed", "bg-success", true); }); - it("unthemed @state=suppressed counter badge should have className 'badge-light'", () => { - validateClassName("suppressed", "badge-light", false); + it("unthemed @state=suppressed counter badge should have className 'bg-light'", () => { + validateClassName("suppressed", "bg-light", false); }); it("@state=unprocessed counter badge should have empty style", () => { diff --git a/ui/src/Components/Labels/FilteringCounterBadge/index.tsx b/ui/src/Components/Labels/FilteringCounterBadge/index.tsx index 32610c9be..599f20884 100644 --- a/ui/src/Components/Labels/FilteringCounterBadge/index.tsx +++ b/ui/src/Components/Labels/FilteringCounterBadge/index.tsx @@ -58,7 +58,7 @@ const FilteringCounterBadge: FC<{ alertStore, name, value, - "badge-pill components-label-with-hover" + "rounded-pill components-label-with-hover" ); return ( @@ -72,8 +72,8 @@ const FilteringCounterBadge: FC<{ themed ? cs.className : [ - `badge-${defaultColor}`, - "badge-pill components-label-with-hover", + `bg-${defaultColor}`, + "rounded-pill components-label-with-hover", ...cs.baseClassNames, ].join(" ") } diff --git a/ui/src/Components/Labels/FilteringLabel/__snapshots__/index.test.tsx.snap b/ui/src/Components/Labels/FilteringLabel/__snapshots__/index.test.tsx.snap index ffff7ce9d..d953e9d69 100644 --- a/ui/src/Components/Labels/FilteringLabel/__snapshots__/index.test.tsx.snap +++ b/ui/src/Components/Labels/FilteringLabel/__snapshots__/index.test.tsx.snap @@ -3,7 +3,7 @@ exports[` matches snapshot 1`] = ` "
    - + foo: diff --git a/ui/src/Components/Labels/LabelWithPercent/__snapshots__/index.test.tsx.snap b/ui/src/Components/Labels/LabelWithPercent/__snapshots__/index.test.tsx.snap index a5d73bbc1..2c9c9ae3e 100644 --- a/ui/src/Components/Labels/LabelWithPercent/__snapshots__/index.test.tsx.snap +++ b/ui/src/Components/Labels/LabelWithPercent/__snapshots__/index.test.tsx.snap @@ -3,8 +3,8 @@ exports[` matches snapshot with isActive=true 1`] = ` "
    - - + + 25 @@ -19,7 +19,7 @@ exports[` matches snapshot with isActive=true 1`] = ` focusable=\\"false\\" data-prefix=\\"fas\\" data-icon=\\"times\\" - class=\\"svg-inline--fa fa-times fa-w-11 cursor-pointer text-reset ml-1 close\\" + class=\\"svg-inline--fa fa-times fa-w-11 cursor-pointer text-reset ms-1\\" role=\\"img\\" xmlns=\\"http://www.w3.org/2000/svg\\" viewbox=\\"0 0 352 512\\" @@ -31,7 +31,7 @@ exports[` matches snapshot with isActive=true 1`] = ` -
    +
    matches snapshot with isActive=true 1`] = ` exports[` matches snapshot with offset=0 1`] = ` "
    - - + + 25 @@ -61,7 +61,7 @@ exports[` matches snapshot with offset=0 1`] = ` -
    +
    matches snapshot with offset=0 1`] = ` exports[` matches snapshot with offset=25 1`] = ` "
    - - + + 25 @@ -91,7 +91,7 @@ exports[` matches snapshot with offset=25 1`] = ` -
    +
    - + {hits} @@ -60,14 +60,14 @@ const LabelWithPercent: FC<{ {isActive ? ( ) : null} -
    +
    {offset === 0 ? null : (
    matches snapshot 1`] = ` " - + foo: diff --git a/ui/src/Components/Labels/Utils.test.ts b/ui/src/Components/Labels/Utils.test.ts index 88be232ce..46bea5e53 100644 --- a/ui/src/Components/Labels/Utils.test.ts +++ b/ui/src/Components/Labels/Utils.test.ts @@ -46,20 +46,18 @@ describe("", () => { it("@state=active label should use StateLabelClassMap.active class", () => { const cs = GetClassAndStyle(alertStore, "@state", "active"); - expect(cs.colorClassNames).toContain(`badge-${StateLabelClassMap.active}`); + expect(cs.colorClassNames).toContain(`bg-${StateLabelClassMap.active}`); }); it("@state=suppressed label should use StateLabelClassMap.suppressed class", () => { const cs = GetClassAndStyle(alertStore, "@state", "suppressed"); - expect(cs.colorClassNames).toContain( - `badge-${StateLabelClassMap.suppressed}` - ); + expect(cs.colorClassNames).toContain(`bg-${StateLabelClassMap.suppressed}`); }); it("@state=unprocessed label should use StateLabelClassMap.unprocessed class", () => { const cs = GetClassAndStyle(alertStore, "@state", "unprocessed"); expect(cs.colorClassNames).toContain( - `badge-${StateLabelClassMap.unprocessed}` + `bg-${StateLabelClassMap.unprocessed}` ); }); diff --git a/ui/src/Components/Labels/Utils.ts b/ui/src/Components/Labels/Utils.ts index a5f2f3fe6..18016cec4 100644 --- a/ui/src/Components/Labels/Utils.ts +++ b/ui/src/Components/Labels/Utils.ts @@ -38,7 +38,7 @@ const GetClassAndStyle = ( } else if (name === StaticLabels.State) { data.colorClassNames.push( StateLabelClassMap[value as AlertStateT] - ? `${elementType}-${StateLabelClassMap[value as AlertStateT]}` + ? `bg-${StateLabelClassMap[value as AlertStateT]}` : DefaultLabelClassMap[elementType] ); } else if (alertStore.settings.values.staticColorLabels.includes(name)) { diff --git a/ui/src/Components/MainModal/Configuration/AlertGroupCollapseConfiguration.tsx b/ui/src/Components/MainModal/Configuration/AlertGroupCollapseConfiguration.tsx index c8aaf1f27..505383a69 100644 --- a/ui/src/Components/MainModal/Configuration/AlertGroupCollapseConfiguration.tsx +++ b/ui/src/Components/MainModal/Configuration/AlertGroupCollapseConfiguration.tsx @@ -34,7 +34,7 @@ const AlertGroupCollapseConfiguration: FC<{ const context = React.useContext(ThemeContext); return ( -
    +
    ) : null} {hideReverse ? null : ( -
    - +
    + onSortReverseChange(event.target.checked)} />