Skip to content

Commit

Permalink
Modified HTML for Algolia Crawler (twentyhq#5441)
Browse files Browse the repository at this point in the history
* Modified HTML for Algolia Crawler
* Added anchor tags within user-guide headers

To implement Algolia correctly, my changes would need to be deployed
first, as it cannot run on localhost. In the meantime, I simulated
Algolia crawling locally using Cheerio, and it worked successfully on my
end.
  • Loading branch information
ady-beraud authored May 16, 2024
1 parent 39b6ec7 commit d221946
Show file tree
Hide file tree
Showing 4 changed files with 52 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import { ReactNode } from 'react';
import styled from '@emotion/styled';

import { Theme } from '@/app/_components/ui/theme/theme';
import { wrapHeadingsWithAnchor } from '@/shared-utils/wrapHeadingsWithAnchor';

const StyledContent = styled.div`
flex: 1;
Expand All @@ -27,6 +28,10 @@ const StyledContent = styled.div`
font-family: var(--font-gabarito);
color: ${Theme.text.color.primary};
font-weight: 700;
a {
text-decoration: none;
color: ${Theme.text.color.primary};
}
}
h1 {
Expand Down Expand Up @@ -89,5 +94,5 @@ const StyledContent = styled.div`
`;

export const ArticleContent = ({ children }: { children: ReactNode }) => {
return <StyledContent>{children}</StyledContent>;
return <StyledContent>{wrapHeadingsWithAnchor(children)}</StyledContent>;
};
Original file line number Diff line number Diff line change
Expand Up @@ -54,10 +54,11 @@ const StyledHeader = styled.div`
}
`;

const StyledHeading = styled.div`
const StyledHeading = styled.h1`
font-size: 40px;
font-weight: 700;
font-family: var(--font-gabarito);
margin: 0px;
@media (max-width: 800px) {
font-size: 28px;
}
Expand All @@ -77,12 +78,13 @@ const StyledHeaderInfoSectionTitle = styled.div`
font-family: var(--font-gabarito);
`;

const StyledHeaderInfoSectionSub = styled.div`
const StyledHeaderInfoSectionSub = styled.p`
display: flex;
flex-direction: column;
gap: ${Theme.spacing(4)};
color: ${Theme.text.color.tertiary};
line-height: 1.8;
margin: 0px;
`;

const StyledRectangle = styled.div`
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ const StyledTitle = styled.div`
font-weight: 600;
`;

const StyledSubTopicItem = styled.div<{ isselected: boolean }>`
const StyledSubTopicItem = styled.a<{ isselected: boolean }>`
cursor: pointer;
display: flex;
flex-direction: row;
Expand Down Expand Up @@ -135,6 +135,7 @@ const UserGuideSidebarSection = ({
<StyledSubTopicItem
key={card.title}
isselected={isselected}
href={`/user-guide/${card.fileName}`}
onClick={() => router.push(`/user-guide/${card.fileName}`)}
onMouseEnter={() => setHoveredItem(card.title)}
onMouseLeave={() => setHoveredItem(null)}
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,40 @@
import React, {
Children,
cloneElement,
isValidElement,
ReactElement,
ReactNode,
} from 'react';

export const wrapHeadingsWithAnchor = (children: ReactNode): ReactNode => {
const hasChildren = (
element: ReactElement,
): element is ReactElement<{ children: ReactNode }> => {
return element.props.children !== undefined;
};

return Children.map(children, (child) => {
if (
isValidElement(child) &&
typeof child.type === 'string' &&
['h1', 'h2', 'h3', 'h4'].includes(child.type)
) {
const id = child.props.children
.toString()
.replace(/\s+/g, '-')
.toLowerCase();
return cloneElement(child as ReactElement<any>, {
id: id,
children: <a href={`#${id}`}>{child.props.children}</a>,
});
}

if (isValidElement(child) && hasChildren(child)) {
return cloneElement(child, {
children: wrapHeadingsWithAnchor(child.props.children),
});
}

return child;
});
};

0 comments on commit d221946

Please sign in to comment.