Skip to content
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -12,16 +12,9 @@ export const TableColumn = styled(Flex)`
`;

export const SchemaTableContainer = styled(Flex)`
& .t--entity-item {
height: 28px;
grid-template-columns: 0 auto 1fr auto auto auto auto auto;

.entity-icon > .ads-v2-icon {
& .ads-v2-listitem {
.datasource-table-icon {
display: none;
}

.t--entity-name {
padding-left: 0;
}
}
`;
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ import {
isMatching,
SEARCH_ITEM_TYPES,
} from "./utils";
import { type Plugin, PluginType } from "entities/Plugin";
import { type Plugin, PluginType, UIComponentTypes } from "entities/Plugin";
import { integrationEditorURL } from "ee/RouteBuilder";
import type { AppState } from "ee/reducers";
import { getCurrentAppWorkspace } from "ee/selectors/selectedWorkspaceSelectors";
Expand Down Expand Up @@ -313,7 +313,9 @@ export const updateActionOperations = (
actionOps: ActionOperation[],
) => {
const restApiPlugin = plugins.find(
(plugin) => plugin.type === PluginType.API,
(plugin) =>
plugin.type === PluginType.API &&
plugin.uiComponent === UIComponentTypes.ApiEditorForm,
);
const newApiActionIdx = actionOps.findIndex(
(op) => op.title === "New blank API",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import React, { memo, useMemo, useRef, useState } from "react";
import Entity, { EntityClassNames } from "../Explorer/Entity";
import React, { memo, useMemo, useState } from "react";
import { EntityClassNames } from "../Explorer/Entity";
import { datasourceTableIcon } from "../Explorer/ExplorerIcons";
import QueryTemplates from "./QueryTemplates";
import type { DatasourceTable } from "entities/Datasource";
Expand All @@ -10,13 +10,20 @@ import { useSelector } from "react-redux";
import type { AppState } from "ee/reducers";
import { getDatasource, getPlugin } from "ee/selectors/entitiesSelector";
import { getPagePermissions } from "selectors/editorSelectors";
import { Menu, MenuTrigger, Button, Tooltip, MenuContent } from "@appsmith/ads";
import {
Menu,
MenuTrigger,
Button,
Tooltip,
MenuContent,
List,
ListItem,
} from "@appsmith/ads";
import { SHOW_TEMPLATES, createMessage } from "ee/constants/messages";
import styled from "styled-components";
import AnalyticsUtil from "ee/utils/AnalyticsUtil";
import type { Plugin } from "entities/Plugin";
import { omit } from "lodash";
import { Virtuoso } from "react-virtuoso";
import { useFeatureFlag } from "utils/hooks/useFeatureFlag";
import { FEATURE_FLAG } from "ee/entities/FeatureFlag";
import { hasCreateDSActionPermissionInApp } from "ee/utils/BusinessFeatures/permissionPageHelpers";
Expand Down Expand Up @@ -47,13 +54,16 @@ const StructureWrapper = styled.div`
height: 100%;
`;

const StyledList = styled(List)`
padding: 0;
`;

const DatasourceStructureItem = memo((props: DatasourceStructureItemProps) => {
const dbStructure = props.dbStructure;
let templateMenu = null;
const [active, setActive] = useState(false);

useCloseMenuOnScroll(SIDEBAR_ID, active, () => setActive(false));
const collapseRef = useRef<HTMLDivElement | null>(null);

const datasource = useSelector((state: AppState) =>
getDatasource(state, props.datasourceId),
Expand Down Expand Up @@ -137,17 +147,14 @@ const DatasourceStructureItem = memo((props: DatasourceStructureItemProps) => {
}, [active, props.tableName]);

return (
<Entity
action={onEntityClick}
active={activeState}
className={`datasourceStructure-${props.context}`}
collapseRef={collapseRef}
contextMenu={templateMenu}
entityId={`${props.datasourceId}-${dbStructure.name}-${props.context}`}
icon={datasourceTableIcon}
isDefaultExpanded={props?.isDefaultOpen}
name={dbStructure.name}
step={props.step}
<ListItem
className={`datasourceStructure-${props.context} t--entity-item`}
data-testid={`t--entity-item-${dbStructure.name}`}
isSelected={activeState}
onClick={onEntityClick}
rightControl={templateMenu}
startIcon={datasourceTableIcon}
title={dbStructure.name}
/>
);
});
Expand All @@ -163,26 +170,18 @@ type DatasourceStructureProps = Partial<DatasourceStructureItemProps> & {
};

const DatasourceStructure = (props: DatasourceStructureProps) => {
const Row = (index: number) => {
const structure = props.tables[index];

return (
<DatasourceStructureItem
{...omit(props, ["tables"])}
dbStructure={structure}
key={`${props.datasourceId}${structure.name}-${props.context}`}
showTemplates={props.showTemplates}
/>
);
};

return (
<StructureWrapper>
<Virtuoso
Comment thread
ankitakinger marked this conversation as resolved.
className="t--schema-virtuoso-container"
itemContent={Row}
totalCount={props.tables.length}
/>
<StyledList className="t--schema-virtuoso-container">
{props.tables.map((dbStructure) => (
<DatasourceStructureItem
{...omit(props, ["tables"])}
dbStructure={dbStructure}
key={`${props.datasourceId}${dbStructure.name}-${props.context}`}
showTemplates={props.showTemplates}
/>
))}
</StyledList>
</StructureWrapper>
);
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,10 +43,7 @@ export const DatasourceListContainer = styled.div`
height: 100%;
display: flex;
flex-direction: column;
padding-left: var(--ads-v2-spaces-5);
.t--entity.datasourceStructure-datasource-view-mode {
padding-right: var(--ads-v2-spaces-5);
}
padding: 0 var(--ads-v2-spaces-5);
&.t--gsheet-structure {
padding-bottom: var(--ads-v2-spaces-8);
.t--gsheet-structure-list {
Expand Down Expand Up @@ -203,7 +200,6 @@ export const DatasourceStructureSearchContainer = styled.div`
margin: var(--ads-v2-spaces-3) 0 var(--ads-v2-spaces-2) 0;
background: white;
flex-shrink: 0;
padding-right: var(--ads-v2-spaces-5);
display: flex;
align-items: center;
gap: var(--ads-v2-spaces-2);
Expand All @@ -212,7 +208,6 @@ export const DatasourceStructureSearchContainer = styled.div`
margin-bottom: var(--ads-v2-spaces-2);
}
&.t--search-container--query-editor {
padding-right: 0;
margin: 0;
}
`;
4 changes: 3 additions & 1 deletion app/client/src/pages/Editor/Explorer/ExplorerIcons.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,9 @@ export const datasourceIcon = (
/>
);

export const datasourceTableIcon = <Icon name="layout-5-line" size="md" />;
export const datasourceTableIcon = (
<Icon className="datasource-table-icon" name="layout-5-line" size="md" />
);

export const primaryKeyIcon = <Icon name="key-2-line" size="md" />;

Expand Down