mirror of
https://github.com/lensapp/lens.git
synced 2025-05-20 05:10:56 +00:00
TopBar home icon (#3740)
* Adding home button Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com> * Highlight catalog entity on route match Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com> * Wait for general entites to load Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com> * Adding WelcomePage catalog entity Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com> * Highlight general entities in hotbar if their route matches Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com> * Fixing tests Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com> * Moving active entities sync to helper function Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com> * Fix welcome page test Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>
This commit is contained in:
parent
995cdac328
commit
16c4656a7d
@ -76,6 +76,10 @@ export class CatalogCategoryRegistry {
|
||||
|
||||
return this.getForGroupKind(group, data.kind);
|
||||
}
|
||||
|
||||
getByName(name: string) {
|
||||
return this.items.find(category => category.metadata?.name == name);
|
||||
}
|
||||
}
|
||||
|
||||
export const catalogCategoryRegistry = new CatalogCategoryRegistry();
|
||||
|
||||
@ -21,7 +21,7 @@
|
||||
|
||||
import { observable } from "mobx";
|
||||
import { GeneralEntity } from "../../common/catalog-entities/general";
|
||||
import { catalogURL, preferencesURL } from "../../common/routes";
|
||||
import { catalogURL, preferencesURL, welcomeURL } from "../../common/routes";
|
||||
import { catalogEntityRegistry } from "../catalog";
|
||||
|
||||
export const catalogEntity = new GeneralEntity({
|
||||
@ -62,9 +62,29 @@ const preferencesEntity = new GeneralEntity({
|
||||
}
|
||||
});
|
||||
|
||||
const welcomePageEntity = new GeneralEntity({
|
||||
metadata: {
|
||||
uid: "welcome-page-entity",
|
||||
name: "Welcome Page",
|
||||
source: "app",
|
||||
labels: {}
|
||||
},
|
||||
spec: {
|
||||
path: welcomeURL(),
|
||||
icon: {
|
||||
material: "meeting_room",
|
||||
background: "#3d90ce"
|
||||
}
|
||||
},
|
||||
status: {
|
||||
phase: "active",
|
||||
}
|
||||
});
|
||||
|
||||
const generalEntities = observable([
|
||||
catalogEntity,
|
||||
preferencesEntity
|
||||
preferencesEntity,
|
||||
welcomePageEntity
|
||||
]);
|
||||
|
||||
export function syncGeneralEntities() {
|
||||
|
||||
36
src/main/catalog-sources/helpers/general-active-sync.ts
Normal file
36
src/main/catalog-sources/helpers/general-active-sync.ts
Normal file
@ -0,0 +1,36 @@
|
||||
/**
|
||||
* Copyright (c) 2021 OpenLens Authors
|
||||
*
|
||||
* Permission is hereby granted, free of charge, to any person obtaining a copy of
|
||||
* this software and associated documentation files (the "Software"), to deal in
|
||||
* the Software without restriction, including without limitation the rights to
|
||||
* use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of
|
||||
* the Software, and to permit persons to whom the Software is furnished to do so,
|
||||
* subject to the following conditions:
|
||||
*
|
||||
* The above copyright notice and this permission notice shall be included in all
|
||||
* copies or substantial portions of the Software.
|
||||
*
|
||||
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
||||
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
|
||||
* FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
|
||||
* COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
|
||||
* IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
|
||||
* CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
|
||||
*/
|
||||
|
||||
import { when } from "mobx";
|
||||
import { catalogCategoryRegistry } from "../../../common/catalog";
|
||||
import { catalogEntityRegistry } from "../../../renderer/api/catalog-entity-registry";
|
||||
import { isActiveRoute } from "../../../renderer/navigation";
|
||||
|
||||
export async function setEntityOnRouteMatch() {
|
||||
await when(() => catalogEntityRegistry.entities.size > 0);
|
||||
|
||||
const entities = catalogEntityRegistry.getItemsForCategory(catalogCategoryRegistry.getByName("General"));
|
||||
const activeEntity = entities.find(entity => isActiveRoute(entity.spec.path));
|
||||
|
||||
if (activeEntity) {
|
||||
catalogEntityRegistry.activeEntity = activeEntity;
|
||||
}
|
||||
}
|
||||
@ -104,7 +104,6 @@ export class Catalog extends React.Component<Props> {
|
||||
}, {fireImmediately: true}),
|
||||
]);
|
||||
}
|
||||
|
||||
addToHotbar(item: CatalogEntityItem<CatalogEntity>): void {
|
||||
HotbarStore.getInstance().addToHotbar(item.entity);
|
||||
}
|
||||
|
||||
@ -32,6 +32,9 @@ jest.mock(
|
||||
ipcRenderer: {
|
||||
on: jest.fn(),
|
||||
},
|
||||
app: {
|
||||
getPath: () => "tmp",
|
||||
},
|
||||
})
|
||||
);
|
||||
|
||||
|
||||
@ -23,7 +23,7 @@ import "./cluster-manager.scss";
|
||||
|
||||
import React from "react";
|
||||
import { Redirect, Route, Switch } from "react-router";
|
||||
import { observer } from "mobx-react";
|
||||
import { disposeOnUnmount, observer } from "mobx-react";
|
||||
import { BottomBar } from "./bottom-bar";
|
||||
import { Catalog } from "../+catalog";
|
||||
import { Preferences } from "../+preferences";
|
||||
@ -35,9 +35,18 @@ import { HotbarMenu } from "../hotbar/hotbar-menu";
|
||||
import { EntitySettings } from "../+entity-settings";
|
||||
import { Welcome } from "../+welcome";
|
||||
import * as routes from "../../../common/routes";
|
||||
import { reaction } from "mobx";
|
||||
import { navigation } from "../../navigation";
|
||||
import { setEntityOnRouteMatch } from "../../../main/catalog-sources/helpers/general-active-sync";
|
||||
|
||||
@observer
|
||||
export class ClusterManager extends React.Component {
|
||||
componentDidMount() {
|
||||
disposeOnUnmount(this, [
|
||||
reaction(() => navigation.location, () => setEntityOnRouteMatch(), { fireImmediately: true })
|
||||
]);
|
||||
}
|
||||
|
||||
render() {
|
||||
return (
|
||||
<div className="ClusterManager">
|
||||
|
||||
@ -41,9 +41,16 @@ jest.mock(
|
||||
}
|
||||
),
|
||||
},
|
||||
app: {
|
||||
getPath: () => "tmp",
|
||||
},
|
||||
})
|
||||
);
|
||||
|
||||
jest.mock("../../+catalog", () => ({
|
||||
previousActiveTab: jest.fn()
|
||||
}));
|
||||
|
||||
const goBack = jest.fn();
|
||||
const goForward = jest.fn();
|
||||
|
||||
@ -76,6 +83,12 @@ describe("<TopBar/>", () => {
|
||||
expect(container).toBeInstanceOf(HTMLElement);
|
||||
});
|
||||
|
||||
it("renders home button", async () => {
|
||||
const { getByTestId } = render(<TopBar/>);
|
||||
|
||||
expect(await getByTestId("home-button")).toBeInTheDocument();
|
||||
});
|
||||
|
||||
it("renders history arrows", async () => {
|
||||
const { getByTestId } = render(<TopBar/>);
|
||||
|
||||
|
||||
@ -28,6 +28,9 @@ import { webContents } from "@electron/remote";
|
||||
import { observable } from "mobx";
|
||||
import { ipcRendererOn } from "../../../common/ipc";
|
||||
import { watchHistoryState } from "../../remote-helpers/history-updater";
|
||||
import { isActiveRoute, navigate } from "../../navigation";
|
||||
import { catalogRoute, catalogURL } from "../../../common/routes";
|
||||
import { previousActiveTab } from "../+catalog";
|
||||
|
||||
interface Props extends React.HTMLAttributes<any> {
|
||||
}
|
||||
@ -68,6 +71,10 @@ export const TopBar = observer(({ children, ...rest }: Props) => {
|
||||
);
|
||||
};
|
||||
|
||||
const goHome = () => {
|
||||
navigate(`${catalogURL()}/${previousActiveTab.get()}`);
|
||||
};
|
||||
|
||||
const goBack = () => {
|
||||
webContents.getAllWebContents().find((webContent) => webContent.getType() === "window")?.goBack();
|
||||
};
|
||||
@ -85,6 +92,13 @@ export const TopBar = observer(({ children, ...rest }: Props) => {
|
||||
return (
|
||||
<div className={styles.topBar} {...rest}>
|
||||
<div className={styles.history}>
|
||||
<Icon
|
||||
data-testid="home-button"
|
||||
material="home"
|
||||
className="ml-5"
|
||||
onClick={goHome}
|
||||
disabled={isActiveRoute(catalogRoute)}
|
||||
/>
|
||||
<Icon
|
||||
data-testid="history-back"
|
||||
material="arrow_back"
|
||||
|
||||
Loading…
Reference in New Issue
Block a user