Skip to content

SCAL-213939 Refactor layout styles and improve responsiveness#351

Merged
shivam-kumar-ts merged 7 commits into
mainfrom
SCAL-213939
Sep 17, 2025
Merged

SCAL-213939 Refactor layout styles and improve responsiveness#351
shivam-kumar-ts merged 7 commits into
mainfrom
SCAL-213939

Conversation

@shivam-kumar-ts
Copy link
Copy Markdown
Contributor

@shivam-kumar-ts shivam-kumar-ts commented Sep 9, 2025 •
edited
Loading

  • Used display flex instead position in css
  • Refactor layout styles and improve responsiveness by introducing new variables for header height.
  • Update document and sidebar components to utilize these variables for consistent spacing.
  • Adjust scrollbar visibility and enhance the document body structure for better usability across devices.
  • Refactor layout styles to remove header height dependencies
  • Display transition animation
  • Update document view styles to prevent overflow
  • Corrected the issues, it takes me to the correct page and shows the correct anchor target in the URL, but it doesn't navigate me to the target section.
  • Adjust sidebar height calculation and enable vertical scrolling on mobile

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 9, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
developer-docs Ready Ready Preview Comment Sep 17, 2025 10:46am

@mnk-blr
Copy link
Copy Markdown
Contributor

mnk-blr commented Sep 9, 2025 •
edited
Loading

🎉 Snyk checks have passed. No issues have been found so far.

✅ security/snyk check is complete. No issues have been found. (View Details)

✅ license/snyk check is complete. No issues have been found. (View Details)

adityamittal3107
adityamittal3107 reviewed Sep 10, 2025
View reviewed changes
Comment thread src/components/Document/index.scss Outdated
h6[id] {
scroll-margin-top: 70px;
}
// h1[id],
Copy link
Copy Markdown
Contributor

@adityamittal3107 adityamittal3107 Sep 10, 2025

Choose a reason for hiding this comment

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

Why have we commented this?

@shivam-kumar-ts
SCAL-213939 Enhance scrolling behavior by adding a timeout for hash …
b90f965 
…navigation and applying an offset to keep headers visible
@shivam-kumar-ts shivam-kumar-ts merged commit 301555a into main Sep 17, 2025
7 checks passed
@shivam-kumar-ts shivam-kumar-ts mentioned this pull request Sep 22, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@adityamittal3107 adityamittal3107 adityamittal3107 approved these changes

@sastaachar sastaachar Awaiting requested review from sastaachar

@ShashiSubramanya ShashiSubramanya Awaiting requested review from ShashiSubramanya

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@shivam-kumar-ts @mnk-blr @adityamittal3107