From: Michal Klobukowski Date: Tue, 17 Jul 2018 11:49:33 +0000 (+0200) Subject: Update icons in context menu X-Git-Url: https://git.arvados.org/arvados.git/commitdiff_plain/9b390921232a4a3d1279d59da96f00dabeba1150?hp=ff25c5d64dd7ba92f121726a31c56f50d4655d2a Update icons in context menu Feature #13834 Arvados-DCO-1.1-Signed-off-by: Michal Klobukowski --- diff --git a/src/components/context-menu/context-menu.test.tsx b/src/components/context-menu/context-menu.test.tsx index 86011a3cee..95a219f98a 100644 --- a/src/components/context-menu/context-menu.test.tsx +++ b/src/components/context-menu/context-menu.test.tsx @@ -7,19 +7,19 @@ import { mount, configure, shallow } from "enzyme"; import * as Adapter from "enzyme-adapter-react-16"; import ContextMenu from "./context-menu"; import { ListItem } from "@material-ui/core"; -import { IconTypes } from "../icon/icon"; +import { ShareIcon } from "../icon/icon"; configure({ adapter: new Adapter() }); describe("", () => { const items = [[{ - icon: IconTypes.ANNOUNCEMENT, + icon: ShareIcon, name: "Action 1.1" }, { - icon: IconTypes.ANNOUNCEMENT, + icon: ShareIcon, name: "Action 1.2" },], [{ - icon: IconTypes.ANNOUNCEMENT, + icon: ShareIcon, name: "Action 2.1" }]]; diff --git a/src/components/context-menu/context-menu.tsx b/src/components/context-menu/context-menu.tsx index a7b83bcfac..ccdc01b129 100644 --- a/src/components/context-menu/context-menu.tsx +++ b/src/components/context-menu/context-menu.tsx @@ -4,11 +4,11 @@ import * as React from "react"; import { Popover, List, ListItem, ListItemIcon, ListItemText, Divider } from "@material-ui/core"; import { DefaultTransformOrigin } from "../popover/helpers"; -import IconBase, { IconTypes } from "../icon/icon"; +import { IconType } from "../icon/icon"; export interface ContextMenuItem { name: string; - icon: IconTypes; + icon: IconType; } export type ContextMenuItemGroup = ContextMenuItem[]; @@ -22,7 +22,7 @@ export interface ContextMenuProps { export default class ContextMenu extends React.PureComponent { render() { - const { anchorEl, items, onClose, onItemClick } = this.props; + const { anchorEl, items, onClose, onItemClick} = this.props; return { key={actionIndex} onClick={() => onItemClick(item)}> - + {item.name} diff --git a/src/views-components/context-menu/action-sets/project-action-set.ts b/src/views-components/context-menu/action-sets/project-action-set.ts index 9a1b1d5b12..61ff9fe1ae 100644 --- a/src/views-components/context-menu/action-sets/project-action-set.ts +++ b/src/views-components/context-menu/action-sets/project-action-set.ts @@ -4,16 +4,16 @@ import { ContextMenuActionSet } from "../context-menu-action-set"; import actions from "../../../store/project/project-action"; -import { IconTypes } from "../../../components/icon/icon"; +import { ShareIcon, NewProjectIcon } from "../../../components/icon/icon"; export const projectActionSet: ContextMenuActionSet = [[{ - icon: IconTypes.FOLDER, + icon: NewProjectIcon, name: "New project", execute: (dispatch, resource) => { dispatch(actions.OPEN_PROJECT_CREATOR({ ownerUuid: resource.uuid })); } }, { - icon: IconTypes.ANNOUNCEMENT, + icon: ShareIcon, name: "Share", execute: () => { return; } }]]; \ No newline at end of file diff --git a/src/views-components/context-menu/action-sets/root-project-action-set.ts b/src/views-components/context-menu/action-sets/root-project-action-set.ts index 53642d03fb..afb428a31b 100644 --- a/src/views-components/context-menu/action-sets/root-project-action-set.ts +++ b/src/views-components/context-menu/action-sets/root-project-action-set.ts @@ -4,10 +4,10 @@ import { ContextMenuActionSet } from "../context-menu-action-set"; import actions from "../../../store/project/project-action"; -import { IconTypes } from "../../../components/icon/icon"; +import { NewProjectIcon } from "../../../components/icon/icon"; export const rootProjectActionSet: ContextMenuActionSet = [[{ - icon: IconTypes.FOLDER, + icon: NewProjectIcon, name: "New project", execute: (dispatch, resource) => { dispatch(actions.OPEN_PROJECT_CREATOR({ ownerUuid: resource.uuid }));