1
0
mirror of https://github.com/lensapp/lens.git synced 2025-05-20 05:10:56 +00:00

Show Custom Settings block for nav items

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>
This commit is contained in:
Alex Andreev 2022-04-26 13:35:14 +03:00
parent b7b25fb1ff
commit 2570c7ce0a
2 changed files with 37 additions and 35 deletions

View File

@ -14,7 +14,6 @@ import preferenceNavigationItemsInjectable from "../preference-navigation-items.
import { computed } from "mobx";
import { noop } from "../../../../utils";
import type { IComputedValue } from "mobx/dist/internal";
import { screen } from "@testing-library/dom";
describe("<PreferencesNavigation />", () => {
let di: DiContainer;
@ -50,14 +49,6 @@ describe("<PreferencesNavigation />", () => {
navigate: () => noop,
orderNumber: 1,
},
{
id: "kube",
label: "Kube",
isActive: computed(() => false),
isVisible: computed(() => false),
navigate: () => noop,
orderNumber: 2,
},
]);
di.override(preferenceNavigationItemsInjectable, () => generalNavItems);
@ -79,16 +70,6 @@ describe("<PreferencesNavigation />", () => {
expect(container).not.toHaveTextContent("Custom Settings");
});
it("does not render hidden navigation items", () => {
const { container } = render(
<PreferencesNavigation/>,
);
screen.debug();
expect(container).not.toHaveTextContent("Kube");
});
});
describe("when general + extension navigation items passed", () => {
@ -169,14 +150,13 @@ describe("<PreferencesNavigation />", () => {
expect(container).toHaveTextContent("lensapp-pod-menu");
});
it("does not render hidden extension navigation items", () => {
const { container } = render(
it("renders extension navigation items inside custom settings block", () => {
const { getByTestId } = render(
<PreferencesNavigation/>,
);
const settingsBlock = getByTestId("extension-settings");
screen.debug();
expect(container).not.toHaveTextContent("metrics-plugin");
expect(settingsBlock).toHaveTextContent("lensapp-node-menu");
});
});
});

View File

@ -13,6 +13,7 @@ import type {
import preferenceNavigationItemsInjectable from "./preference-navigation-items.injectable";
import { observer } from "mobx-react";
import { Icon } from "../../icon";
interface Dependencies {
navigationItems: IComputedValue<PreferenceNavigationItem[]>;
@ -20,23 +21,44 @@ interface Dependencies {
const NonInjectedPreferencesNavigation = ({
navigationItems,
}: Dependencies) => (
<Tabs
className="flex column"
scrollable={false}
onChange={(item: PreferenceNavigationItem) => item.navigate()}
>
<div className="header">Preferences</div>
}: Dependencies) => {
const generalNavItems = navigationItems.get().filter(item => !item.fromExtension);
const extensionNavItems = navigationItems.get().filter(item => item.fromExtension);
{navigationItems.get().map((item) => (
function renderTab(item: PreferenceNavigationItem) {
return (
<PreferencesNavigationTab
key={item.id}
item={item}
data-testid={`tab-link-for-${item.id}`}
/>
))}
</Tabs>
);
);
}
return (
<Tabs
className="flex column"
scrollable={false}
onChange={(item: PreferenceNavigationItem) => item.navigate()}
>
<div className="header">Preferences</div>
{generalNavItems.map(renderTab)}
{extensionNavItems.length > 0 && (
<div data-testid="extension-settings">
<hr/>
<div className="header flex items-center">
<Icon material="extension" smallest className="mr-3"/> Custom Settings
</div>
<div>
{extensionNavItems.map(renderTab)}
</div>
</div>
)}
</Tabs>
);
};
interface PreferenceNavigationTabProps extends React.DOMAttributes<HTMLElement> {
item: PreferenceNavigationItem;