Skip to content

fix : Question section and Prev/Next Cards #764

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Gagan202005 wants to merge 4 commits into
base: main
Choose a base branch
from
+67 −24
Open

fix : Question section and Prev/Next Cards #764

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a8f5588
fix questions section
Gagan202005 Jan 27, 2026
22721ea
Merge branch 'main' into fix-Questions-section
Gagan202005 Jan 27, 2026
565135f
Merge branch 'main' into fix-Questions-section
Gagan202005 Jan 30, 2026
824c1cb
Merge branch 'main' into fix-Questions-section
Gagan202005 Feb 2, 2026
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
38 changes: 18 additions & 20 deletions src/components/KeployCloud.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,27 +4,25 @@ export const KeployCloud = () => {
return (
<section
id="cloud"
className="mb-8 mt-12 flex max-w-7xl items-center space-x-6 rounded-lg bg-[color:var(--ifm-card-background-color)] p-6 shadow-md"
className="mt-10 flex max-w-7xl items-center justify-between rounded-md border border-[color:var(--ifm-toc-border-color)] bg-[color:var(--ifm-background-surface-color)] px-4 py-3"
>
<div className="prose prose-orange max-w-3xl text-left">
<h1 className="text-left">Questions? 🤔💭</h1>
<p className="my-3 block text-left">
For any support please{" "}
<a
href="https://join.slack.com/t/keploy/shared_invite/zt-357qqm9b5-PbZRVu3Yt2rJIa6ofrwWNg"
className="text-orange-500 underline hover:text-orange-400"
>
join keploy slack community
</a>{" "}
to get help from fellow users, or{" "}
<a
href="https://calendar.app.google/cXVaj6hbMUjvmrnt9"
className="text-orange-500 underline hover:text-orange-400"
>
book a demo
</a>{" "}
if you're exploring enterprise use cases.
</p>
<p className="m-0 text-sm text-[color:var(--ifm-font-color-base)]">
Need help or have questions?
</p>

<div className="flex gap-4 text-sm">
<a
href="https://join.slack.com/t/keploy/shared_invite/zt-357qqm9b5-PbZRVu3Yt2rJIa6ofrwWNg"
className="font-medium text-orange-500 hover:underline"
>
Join Slack
</a>
<a
href="https://calendar.app.google/cXVaj6hbMUjvmrnt9"
className="font-medium text-orange-500 hover:underline"
Comment on lines +16 to +22
Copy link

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

External links should include rel="noopener noreferrer" for security. Without these attributes, the linked page can access the window.opener object, which is a security risk. Other external links in the codebase (e.g., src/components/Community.js:80, src/components/WhatIsKeploy.js:176) properly include these attributes.

Suggested change
className="font-medium text-orange-500 hover:underline"
>
Join Slack
</a>
<a
href="https://calendar.app.google/cXVaj6hbMUjvmrnt9"
className="font-medium text-orange-500 hover:underline"
className="font-medium text-orange-500 hover:underline"
rel="noopener noreferrer"
>
Join Slack
</a>
<a
href="https://calendar.app.google/cXVaj6hbMUjvmrnt9"
className="font-medium text-orange-500 hover:underline"
rel="noopener noreferrer"

Copilot uses AI. Check for mistakes.
Comment on lines +16 to +22
Copy link

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

External links should include rel="noopener noreferrer" for security. Without these attributes, the linked page can access the window.opener object, which is a security risk. Other external links in the codebase (e.g., src/components/Community.js:80, src/components/WhatIsKeploy.js:176) properly include these attributes.

Suggested change
className="font-medium text-orange-500 hover:underline"
>
Join Slack
</a>
<a
href="https://calendar.app.google/cXVaj6hbMUjvmrnt9"
className="font-medium text-orange-500 hover:underline"
className="font-medium text-orange-500 hover:underline"
rel="noopener noreferrer"
>
Join Slack
</a>
<a
href="https://calendar.app.google/cXVaj6hbMUjvmrnt9"
className="font-medium text-orange-500 hover:underline"
rel="noopener noreferrer"

Copilot uses AI. Check for mistakes.
>
Book a demo
</a>
</div>
</section>
);
Expand Down
34 changes: 34 additions & 0 deletions src/css/custom.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2599,3 +2599,37 @@ html[data-theme="dark"] div[class^="sidebar_"] > nav > ul > li > .menu__list-ite
}


/* === Keploy custom paginator styling === */
.keploy-doc-paginator {
margin-top: 1.5rem;
padding-top: 1rem;
border-top: 1px solid var(--ifm-color-emphasis-200);
Copy link

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The .keploy-doc-paginator class has a border-top style (line 2606 in custom.css), but there's already an hr element immediately before the paginator (line 252 in DocItem/index.js). This creates redundant visual separation. Consider removing either the hr element or the border-top style from the CSS to avoid double borders.

Suggested change
border-top: 1px solid var(--ifm-color-emphasis-200);

Copilot uses AI. Check for mistakes.
}

/* Force styles through wrapper */
.keploy-doc-paginator a {
border: 1px solid var(--ifm-color-emphasis-200);
border-radius: 4px;
padding: 0.6rem 0.75rem;
background: transparent;
box-shadow: none;
}

/* Hover state */
.keploy-doc-paginator a:hover {
border-color: var(--ifm-color-emphasis-400);
background-color: var(--ifm-hover-overlay);
text-decoration: none;
}

/* Compact text */
.keploy-doc-paginator .pagination-nav__label {
font-size: 0.7rem;
text-transform: uppercase;
opacity: 0.6;
}

.keploy-doc-paginator .pagination-nav__sublabel {
font-size: 0.85rem;
font-weight: 500;
}
16 changes: 12 additions & 4 deletions src/theme/DocItem/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -245,10 +245,18 @@ export default function DocItem(props) {
</article>
</div>
</article>
<div>
<KeployCloud />
</div>
<DocPaginator previous={metadata.previous} next={metadata.next} />
<hr className="my-8 border-[color:var(--ifm-toc-border-color)]" />

<KeployCloud />

<hr className="my-8 border-[color:var(--ifm-toc-border-color)]" />

<div className="keploy-doc-paginator">
<DocPaginator previous={metadata.previous} next={metadata.next} />
</div>

<hr className="my-6 border-[color:var(--ifm-color-emphasis-200)]" />

Comment on lines +250 to +259
Copy link

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The JSX elements added here have inconsistent indentation. Lines 250-258 are not properly indented to match the surrounding code structure. They should be indented to align with the parent container's content (same level as the closing article tag on line 247). Proper indentation improves code readability and maintainability.

Suggested change
<KeployCloud />
<hr className="my-8 border-[color:var(--ifm-toc-border-color)]" />
<div className="keploy-doc-paginator">
<DocPaginator previous={metadata.previous} next={metadata.next} />
</div>
<hr className="my-6 border-[color:var(--ifm-color-emphasis-200)]" />
<KeployCloud />
<hr className="my-8 border-[color:var(--ifm-toc-border-color)]" />
<div className="keploy-doc-paginator">
<DocPaginator previous={metadata.previous} next={metadata.next} />
</div>
<hr className="my-6 border-[color:var(--ifm-color-emphasis-200)]" />

Copilot uses AI. Check for mistakes.
Copy link

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line contains trailing whitespace. Remove the trailing spaces to maintain clean code formatting.

Suggested change

Copilot uses AI. Check for mistakes.
</div>
</div>

Expand Down
3 changes: 3 additions & 0 deletions src/theme/DocItem/styles.module.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,3 +20,6 @@
display: none;
}
}



Comment on lines +23 to +25
Copy link

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unnecessary blank lines added at the end of the file. These don't add any functional value and should be removed to keep the file clean.

Suggested change

Copilot uses AI. Check for mistakes.
Loading