mirror of
https://github.com/datahub-project/datahub.git
synced 2025-08-22 16:18:10 +00:00
feat(ui) Show source documentation when editing entity documentation (#8516)
Co-authored-by: Aseem Bansal <asmbansal2@gmail.com>
This commit is contained in:
parent
faa368c2be
commit
c77b0d0260
@ -19,18 +19,20 @@ const ContentWrapper = styled.div`
|
|||||||
|
|
||||||
interface Props {
|
interface Props {
|
||||||
description: string;
|
description: string;
|
||||||
|
isExpandable?: boolean;
|
||||||
|
limit?: number;
|
||||||
}
|
}
|
||||||
|
|
||||||
export default function DescriptionSection({ description }: Props) {
|
export default function DescriptionSection({ description, isExpandable, limit }: Props) {
|
||||||
const isOverLimit = description && removeMarkdown(description).length > ABBREVIATED_LIMIT;
|
const isOverLimit = description && removeMarkdown(description).length > ABBREVIATED_LIMIT;
|
||||||
const [isExpanded, setIsExpanded] = useState(!isOverLimit);
|
const [isExpanded, setIsExpanded] = useState(!isOverLimit);
|
||||||
const routeToTab = useRouteToTab();
|
const routeToTab = useRouteToTab();
|
||||||
const isCompact = React.useContext(CompactContext);
|
const isCompact = React.useContext(CompactContext);
|
||||||
const shouldShowReadMore = !useIsOnTab('Documentation');
|
const shouldShowReadMore = !useIsOnTab('Documentation') || isExpandable;
|
||||||
|
|
||||||
// if we're not in compact mode, route them to the Docs tab for the best documentation viewing experience
|
// if we're not in compact mode, route them to the Docs tab for the best documentation viewing experience
|
||||||
function readMore() {
|
function readMore() {
|
||||||
if (isCompact) {
|
if (isCompact || isExpandable) {
|
||||||
setIsExpanded(true);
|
setIsExpanded(true);
|
||||||
} else {
|
} else {
|
||||||
routeToTab({ tabName: 'Documentation' });
|
routeToTab({ tabName: 'Documentation' });
|
||||||
@ -47,7 +49,7 @@ export default function DescriptionSection({ description }: Props) {
|
|||||||
)}
|
)}
|
||||||
{!isExpanded && (
|
{!isExpanded && (
|
||||||
<NoMarkdownViewer
|
<NoMarkdownViewer
|
||||||
limit={ABBREVIATED_LIMIT}
|
limit={limit || ABBREVIATED_LIMIT}
|
||||||
readMore={
|
readMore={
|
||||||
shouldShowReadMore ? <Typography.Link onClick={readMore}>Read More</Typography.Link> : undefined
|
shouldShowReadMore ? <Typography.Link onClick={readMore}>Read More</Typography.Link> : undefined
|
||||||
}
|
}
|
||||||
|
@ -1,6 +1,6 @@
|
|||||||
import React, { useState, useEffect } from 'react';
|
import React, { useState, useEffect } from 'react';
|
||||||
import { message } from 'antd';
|
import { message } from 'antd';
|
||||||
import styled from 'styled-components';
|
import styled from 'styled-components/macro';
|
||||||
import analytics, { EventType, EntityActionType } from '../../../../../analytics';
|
import analytics, { EventType, EntityActionType } from '../../../../../analytics';
|
||||||
import { GenericEntityUpdate } from '../../../types';
|
import { GenericEntityUpdate } from '../../../types';
|
||||||
import { useEntityData, useEntityUpdate, useMutationUrn, useRefetch } from '../../../EntityContext';
|
import { useEntityData, useEntityUpdate, useMutationUrn, useRefetch } from '../../../EntityContext';
|
||||||
@ -9,10 +9,17 @@ import { DiscardDescriptionModal } from './DiscardDescriptionModal';
|
|||||||
import { EDITED_DESCRIPTIONS_CACHE_NAME } from '../../../utils';
|
import { EDITED_DESCRIPTIONS_CACHE_NAME } from '../../../utils';
|
||||||
import { DescriptionEditorToolbar } from './DescriptionEditorToolbar';
|
import { DescriptionEditorToolbar } from './DescriptionEditorToolbar';
|
||||||
import { Editor } from './editor/Editor';
|
import { Editor } from './editor/Editor';
|
||||||
|
import SourceDescription from './SourceDesription';
|
||||||
|
|
||||||
const EditorContainer = styled.div`
|
const EditorContainer = styled.div`
|
||||||
|
flex: 1;
|
||||||
|
`;
|
||||||
|
|
||||||
|
const EditorSourceWrapper = styled.div`
|
||||||
overflow: auto;
|
overflow: auto;
|
||||||
height: 100%;
|
display: flex;
|
||||||
|
flex-direction: column;
|
||||||
|
flex: 1;
|
||||||
`;
|
`;
|
||||||
|
|
||||||
type DescriptionEditorProps = {
|
type DescriptionEditorProps = {
|
||||||
@ -138,9 +145,12 @@ export const DescriptionEditor = ({ onComplete }: DescriptionEditorProps) => {
|
|||||||
onClose={handleConfirmClose}
|
onClose={handleConfirmClose}
|
||||||
disableSave={!isDescriptionUpdated}
|
disableSave={!isDescriptionUpdated}
|
||||||
/>
|
/>
|
||||||
<EditorContainer>
|
<EditorSourceWrapper>
|
||||||
<Editor content={updatedDescription} onChange={handleEditorChange} />
|
<EditorContainer>
|
||||||
</EditorContainer>
|
<Editor content={updatedDescription} onChange={handleEditorChange} />
|
||||||
|
</EditorContainer>
|
||||||
|
<SourceDescription />
|
||||||
|
</EditorSourceWrapper>
|
||||||
{confirmCloseModalVisible && (
|
{confirmCloseModalVisible && (
|
||||||
<DiscardDescriptionModal
|
<DiscardDescriptionModal
|
||||||
cancelModalVisible={confirmCloseModalVisible}
|
cancelModalVisible={confirmCloseModalVisible}
|
||||||
|
@ -0,0 +1,31 @@
|
|||||||
|
import React from 'react';
|
||||||
|
import styled from 'styled-components';
|
||||||
|
import { useEntityData } from '../../../EntityContext';
|
||||||
|
import { ANTD_GRAY } from '../../../constants';
|
||||||
|
import DescriptionSection from '../../../containers/profile/sidebar/AboutSection/DescriptionSection';
|
||||||
|
import { getPlatformName } from '../../../utils';
|
||||||
|
|
||||||
|
const SourceDescriptionWrapper = styled.div`
|
||||||
|
border-top: 1px solid ${ANTD_GRAY[4]};
|
||||||
|
padding: 16px 0 16px 32px;
|
||||||
|
`;
|
||||||
|
|
||||||
|
const Title = styled.div`
|
||||||
|
font-weight: 700;
|
||||||
|
padding-bottom: 16px;
|
||||||
|
`;
|
||||||
|
|
||||||
|
export default function SourceDescription() {
|
||||||
|
const { entityData } = useEntityData();
|
||||||
|
const platformName = getPlatformName(entityData);
|
||||||
|
const sourceDescription = entityData?.properties?.description;
|
||||||
|
|
||||||
|
if (!sourceDescription || !entityData?.platform) return null;
|
||||||
|
|
||||||
|
return (
|
||||||
|
<SourceDescriptionWrapper>
|
||||||
|
<Title>{platformName ? <span>{platformName}</span> : <>Source</>} Documentation:</Title>
|
||||||
|
<DescriptionSection description={sourceDescription} limit={200} isExpandable />
|
||||||
|
</SourceDescriptionWrapper>
|
||||||
|
);
|
||||||
|
}
|
@ -11,7 +11,8 @@ export const OnChangeMarkdown = ({ onChange }: OnChangeMarkdownProps): null => {
|
|||||||
|
|
||||||
const onDocChanged = useCallback(
|
const onDocChanged = useCallback(
|
||||||
({ state }) => {
|
({ state }) => {
|
||||||
const markdown = getMarkdown(state);
|
let markdown = getMarkdown(state);
|
||||||
|
if (markdown === ' ') markdown = '';
|
||||||
onChange(markdown);
|
onChange(markdown);
|
||||||
},
|
},
|
||||||
[onChange, getMarkdown],
|
[onChange, getMarkdown],
|
||||||
|
Loading…
x
Reference in New Issue
Block a user