Skip to content

Commit

Permalink
fix: revert name to DropDownMenu
Browse files Browse the repository at this point in the history
Signed-off-by: axel7083 <42176370+axel7083@users.noreply.github.com>
  • Loading branch information
axel7083 committed May 27, 2024
1 parent 194d8ca commit cac7317
Show file tree
Hide file tree
Showing 15 changed files with 36 additions and 35 deletions.
6 changes: 3 additions & 3 deletions packages/renderer/src/lib/compose/ComposeActions.svelte
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<script lang="ts">
import { faArrowsRotate, faFileCode, faPlay, faRocket, faStop, faTrash } from '@fortawesome/free-solid-svg-icons';
import { Dropdown } from '@podman-desktop/ui-svelte';
import { DropdownMenu } from '@podman-desktop/ui-svelte';
import { createEventDispatcher, onMount } from 'svelte';
import { router } from 'tinro';
Expand Down Expand Up @@ -108,9 +108,9 @@ function openGenerateKube(): void {
// If dropdownMenu = true, we'll change style to the imported dropdownMenu style
// otherwise, leave blank.
let actionsStyle: typeof Dropdown | typeof FlatMenu;
let actionsStyle: typeof DropdownMenu | typeof FlatMenu;
if (dropdownMenu) {
actionsStyle = Dropdown;
actionsStyle = DropdownMenu;
} else {
actionsStyle = FlatMenu;
}
Expand Down
6 changes: 3 additions & 3 deletions packages/renderer/src/lib/container/ContainerActions.svelte
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ import {
faTerminal,
faTrash,
} from '@fortawesome/free-solid-svg-icons';
import { Dropdown } from '@podman-desktop/ui-svelte';
import { DropdownMenu } from '@podman-desktop/ui-svelte';
import { createEventDispatcher, onMount } from 'svelte';
import { router } from 'tinro';
Expand Down Expand Up @@ -127,9 +127,9 @@ function deployToKubernetes(): void {
// If dropdownMenu = true, we'll change style to the imported dropdownMenu style
// otherwise, leave blank.
let actionsStyle: typeof Dropdown | typeof FlatMenu;
let actionsStyle: typeof DropdownMenu | typeof FlatMenu;
if (dropdownMenu) {
actionsStyle = Dropdown;
actionsStyle = DropdownMenu;
} else {
actionsStyle = FlatMenu;
}
Expand Down
5 changes: 3 additions & 2 deletions packages/renderer/src/lib/image/ActionsMenu.svelte
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<script lang="ts">
import { Dropdown } from '@podman-desktop/ui-svelte';
import { DropdownMenu } from '@podman-desktop/ui-svelte';
import FlatMenu from '../ui/FlatMenu.svelte';
Expand All @@ -9,7 +9,8 @@ export let onBeforeToggle = () => {};
</script>

{#if dropdownMenu}
<Dropdown shownAsMenuActionItem="{dropdownMenuAsMenuActionItem}" onBeforeToggle="{onBeforeToggle}"><slot /></Dropdown>
<DropdownMenu shownAsMenuActionItem="{dropdownMenuAsMenuActionItem}" onBeforeToggle="{onBeforeToggle}"
><slot /></DropdownMenu>
{:else}
<FlatMenu><slot /></FlatMenu>
{/if}
14 changes: 7 additions & 7 deletions packages/renderer/src/lib/pod/PodActions.svelte
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import {
faStop,
faTrash,
} from '@fortawesome/free-solid-svg-icons';
import { Dropdown } from '@podman-desktop/ui-svelte';
import { DropdownMenu } from '@podman-desktop/ui-svelte';
import { createEventDispatcher, onMount } from 'svelte';
import { router } from 'tinro';
Expand Down Expand Up @@ -149,9 +149,9 @@ function deployToKubernetes(): void {
}
// If dropdownMenu = true, we'll change style to the imported dropdownMenu style
// otherwise, leave blank.
let actionsStyle: typeof Dropdown | typeof FlatMenu;
let actionsStyle: typeof DropdownMenu | typeof FlatMenu;
if (dropdownMenu) {
actionsStyle = Dropdown;
actionsStyle = DropdownMenu;
} else {
actionsStyle = FlatMenu;
}
Expand Down Expand Up @@ -217,7 +217,7 @@ if (dropdownMenu) {
detailed="{detailed}"
icon="{faExternalLinkSquareAlt}" />
{:else if openingUrls.length > 1}
<Dropdown icon="{faExternalLinkSquareAlt}" hidden="{dropdownMenu}" shownAsMenuActionItem="{true}">
<DropdownMenu icon="{faExternalLinkSquareAlt}" hidden="{dropdownMenu}" shownAsMenuActionItem="{true}">
{#each openingUrls as url}
<ListItemButtonIcon
title="Open {extractPort(url)}"
Expand All @@ -228,7 +228,7 @@ if (dropdownMenu) {
detailed="{detailed}"
icon="{faExternalLinkSquareAlt}" />
{/each}
</Dropdown>
</DropdownMenu>
{/if}
<ListItemButtonIcon
title="Restart Pod"
Expand Down Expand Up @@ -256,7 +256,7 @@ if (dropdownMenu) {
detailed="{detailed}"
icon="{faExternalLinkSquareAlt}" />
{:else if openingKubernetesUrls.size > 1}
<Dropdown
<DropdownMenu
title="Open Kubernetes Routes"
icon="{faExternalLinkSquareAlt}"
hidden="{dropdownMenu}"
Expand All @@ -271,7 +271,7 @@ if (dropdownMenu) {
detailed="{detailed}"
icon="{faExternalLinkSquareAlt}" />
{/each}
</Dropdown>
</DropdownMenu>
{/if}
{/if}
<ContributionActions
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import {
faRightFromBracket,
faRightToBracket,
} from '@fortawesome/free-solid-svg-icons';
import { Button, Dropdown, EmptyScreen, Tooltip } from '@podman-desktop/ui-svelte';
import { Button, DropdownMenu, EmptyScreen, Tooltip } from '@podman-desktop/ui-svelte';
import Fa from 'svelte-fa';
import EmbeddableCatalogExtensionList from '/@/lib/extensions/EmbeddableCatalogExtensionList.svelte';
Expand Down Expand Up @@ -147,14 +147,14 @@ import SettingsPage from './SettingsPage.svelte';
<!-- Authentication Provider Auth Request Sign In button end -->
{:else if sessionRequests.length > 1}
<!-- Authentication Provider Auth Requests DropDown start -->
<Dropdown>
<DropdownMenu>
{#each sessionRequests as request}
<Dropdown.MenuItem
<DropdownMenu.Item
title="Sign in to use {request.extensionLabel}"
onClick="{() => window.requestAuthenticationProviderSignIn(request.id)}"
icon="{faArrowRightToBracket}" />
{/each}
</Dropdown>
</DropdownMenu>
<!-- Authentication Provider Auth Requests DropDown end -->
{/if}
</div>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<script lang="ts">
import { faPlusCircle, faTrash, faUser, faUserPen } from '@fortawesome/free-solid-svg-icons';
import type * as containerDesktopAPI from '@podman-desktop/api';
import { Button, Dropdown, Input } from '@podman-desktop/ui-svelte';
import { Button, DropdownMenu, Input } from '@podman-desktop/ui-svelte';
import { onMount } from 'svelte';
import PasswordInput from '/@/lib/ui/PasswordInput.svelte';
Expand Down Expand Up @@ -347,22 +347,22 @@ function removeExistingRegistry(registry: containerDesktopAPI.Registry) {

<!-- Show/hide password end -->
<!-- containerDesktopAPI.Registry menu start -->
<Dropdown>
<Dropdown.MenuItem
<DropdownMenu>
<DropdownMenu.Item
title="Login"
onClick="{() => markRegistryAsModified(registry)}"
hidden="{!!registry.username && !!registry.secret}"
icon="{faUser}" />
<Dropdown.MenuItem
<DropdownMenu.Item
title="Edit password"
onClick="{() => markRegistryAsModified(registry)}"
hidden="{!registry.username && !registry.secret}"
icon="{faUserPen}" />
<Dropdown.MenuItem
<DropdownMenu.Item
title="Remove"
onClick="{() => removeExistingRegistry(registry)}"
icon="{faTrash}" />
</Dropdown>
</DropdownMenu>
</div>
{/if}
</div>
Expand Down
4 changes: 2 additions & 2 deletions packages/renderer/src/lib/ui/ListItemButtonIcon.svelte
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<script lang="ts">
import type { IconDefinition } from '@fortawesome/fontawesome-common-types';
import { Dropdown, isFontAwesomeIcon } from '@podman-desktop/ui-svelte';
import { DropdownMenu, isFontAwesomeIcon } from '@podman-desktop/ui-svelte';
import { onDestroy, onMount } from 'svelte';
import type { Unsubscriber } from 'svelte/motion';
import Fa from 'svelte-fa';
Expand Down Expand Up @@ -115,7 +115,7 @@ $: styleClass = detailed
<!-- If menu = true, use the menu, otherwise implement the button -->
{#if menu}
<!-- enabled menu -->
<Dropdown.MenuItem
<DropdownMenu.Item
title="{title}"
tooltip="{tooltip}"
icon="{icon}"
Expand Down
6 changes: 3 additions & 3 deletions packages/ui/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -56,9 +56,9 @@
"types": "./dist/alert/ErrorMessage.svelte.d.ts",
"svelte": "./dist/alert/ErrorMessage.svelte"
},
"./Dropdown": {
"types": "./dist/dropdown/index.d.ts",
"svelte": "./dist/dropdown/index.js"
"./DropdownMenu": {
"types": "./dist/dropdownMenu/index.d.ts",
"svelte": "./dist/dropdownMenu/index.js"
},
"./icons": {
"types": "./dist/icons/index.d.ts",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ import '@testing-library/jest-dom/vitest';
import { fireEvent, render, screen } from '@testing-library/svelte';
import { expect, test, vi } from 'vitest';

import DropdownMenu from './Dropdown.svelte';
import DropdownMenu from './DropdownMenu.svelte';

test('Expect the onBeforeToggle function to be called when the menu is clicked', async () => {
const onBeforeToggleMock = vi.fn();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,9 @@
*
* SPDX-License-Identifier: Apache-2.0
***********************************************************************/
import DropdownMenu from './Dropdown.svelte';
import DropdownMenu from './DropdownMenu.svelte';
import DropDownMenuItem from './DropDownMenuItem.svelte';

export default Object.assign(DropdownMenu, {
MenuItem: DropDownMenuItem,
Item: DropDownMenuItem,
});
4 changes: 2 additions & 2 deletions packages/ui/src/lib/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ import ErrorMessage from './alert/ErrorMessage.svelte';
import type { ButtonType } from './button/Button';
import Button from './button/Button.svelte';
import Checkbox from './checkbox/Checkbox.svelte';
import Dropdown from './dropdown';
import DropdownMenu from './dropdownMenu';
import Input from './inputs/Input.svelte';
import SearchInput from './inputs/SearchInput.svelte';
import NavPage from './layouts/NavPage.svelte';
Expand All @@ -41,8 +41,8 @@ export type { ButtonType };
export {
Button,
Checkbox,
DropdownMenu,
EmptyScreen,
Dropdown,
ErrorMessage,
FilteredEmptyScreen,
Input,
Expand Down

0 comments on commit cac7317

Please sign in to comment.