-
Notifications
You must be signed in to change notification settings - Fork 258
Fix/Docs-Page-Mobile-Friendly #709
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.
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
Srinu346
wants to merge
13
commits into
keploy:main
Choose a base branch
from
Srinu346:fix/landingpage-product-section
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit
Hold shift + click to select a range
9ad61e6
fix(docs_page): mobile_view
Srinu346 91a1412
Merge branch 'keploy:main' into fix/landingpage-product-section
Srinu346 a1bd6ea
fix(docs_page): prettier_issue
Srinu346 d61d029
fix(docs_page): prettier_issue
Srinu346 af21e07
Merge branch 'main' into fix/landingpage-product-section
Achanandhi-M 8406497
Merge branch 'main' into fix/landingpage-product-section
Achanandhi-M 76b2106
Merge branch 'main' into fix/landingpage-product-section
Achanandhi-M 0b12f1a
Merge branch 'main' into fix/landingpage-product-section
Srinu346 92ea4a9
Merge branch 'main' into fix/landingpage-product-section
Srinu346 037d24f
Merge branch 'main' into fix/landingpage-product-section
Srinu346 ec7768e
Merge branch 'main' into fix/landingpage-product-section
Srinu346 7760b74
fix(ui): docspage-mobileview
Srinu346 a4df199
fix/docspage-mobile-view
Srinu346 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why there are two grid styles implement, we already have grid style in the code
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sir removing ( auto-cols-[300px]) was disturbing the desktop version in little way so i avoided it, if it's ok then i will align the boxes and remove the additional grid style .

There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No the point is there are two grid sections in the updated code, if you are solving for the small screen better use media screens dont change the existing style
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sir, I have used two grid styles one for small screen and one for desktop version which was already there, even after trying for using media screen, some styles were not getting overidded sir (grid-flow-row) was not getting overidded sir.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Screen.Recording.2025-10-31.010643.mp4