-
Notifications
You must be signed in to change notification settings - Fork 1k
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
fix: Add scroll to the setting navbar #1398
fix: Add scroll to the setting navbar #1398
Conversation
@NilanjanPramanik is attempting to deploy a commit to the formbricks Team on Vercel. A member of the Team first needs to authorize it. |
The changes in the PR seem to be adding scroll functionality to certain div elements. While this is a good approach to handle overflow of content, it might be better to consider a responsive design that adjusts the layout based on the screen size. This would provide a better user experience as it avoids the need for scrolling. However, if scrolling is necessary, it's recommended to use Here are the suggested changes: - <div className="fixed hidden h-full overflow-y-scroll bg-white py-2 pl-4 pr-10 md:block ">
+ <div className="fixed hidden h-full overflow-y-auto bg-white py-2 pl-4 pr-10 md:block ">
- <div className="fixed z-10 flex h-14 overflow-x-scroll w-full items-center justify-between border-b border-slate-200 bg-white px-4 sm:px-6 md:hidden">
+ <div className="fixed z-10 flex h-14 overflow-x-auto w-full items-center justify-between border-b border-slate-200 bg-white px-4 sm:px-6 md:hidden"> Please consider these changes and let me know if you have any questions or concerns. |
Thank you for following the naming conventions for pull request titles! 🙏 |
return ( | ||
<> | ||
<div className="fixed hidden h-full bg-white py-2 pl-4 pr-10 md:block "> | ||
<div className="fixed hidden h-full overflow-y-scroll bg-white py-2 pl-4 pr-10 md:block "> |
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.
Hi thank you for your contribution 🎉 , tried testing this on a 13" Mac but still the issue isn't resolved. Even on 100% zoom level not able to view the License
link in the Compliance section. Attaching both 100% and 90% views. Can you please fix this, will re-review it once you are done with the fix.
![Screenshot 2023-10-23 at 5 59 00 PM](https://private-user-images.githubusercontent.com/31568159/277336577-c8151f9e-0148-450b-99fd-5def507d3af7.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjIwMzM2NTksIm5iZiI6MTcyMjAzMzM1OSwicGF0aCI6Ii8zMTU2ODE1OS8yNzczMzY1NzctYzgxNTFmOWUtMDE0OC00NTBiLTk5ZmQtNWRlZjUwN2QzYWY3LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MjYlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzI2VDIyMzU1OVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWZkYzA3Y2RmNTI0NjdjZGI4YmE3ZjViYjdhZjVjMWQxODVkZDY1NWZmZjJlMDc1NjNhMWZhMjAyZTFjMjMxMWQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.UiV5cTpLdZW_yI_LhIqORNq9NR41Nftm40MWmkmG6h0)
![Screenshot 2023-10-23 at 5 58 46 PM](https://private-user-images.githubusercontent.com/31568159/277336643-82ba1ebc-cc4d-452e-aa74-317aa9802ad6.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjIwMzM2NTksIm5iZiI6MTcyMjAzMzM1OSwicGF0aCI6Ii8zMTU2ODE1OS8yNzczMzY2NDMtODJiYTFlYmMtY2M0ZC00NTJlLWFhNzQtMzE3YWE5ODAyYWQ2LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MjYlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzI2VDIyMzU1OVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTEwNmZkY2E1ZjFmYjFmYTE2NGRhMmM2NjJhY2UzNDdhNjE4YWQ3ZWEzMDBiMDA5NzkxZTlhNWE1MWI5ZTVjZGMmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.xGqLmIOHk-DOGUMP6M-D0ERPEnNjf0gWzgaa_q3F5s8)
cc: @mattinannt
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.
@neilchauhan2 yes, you are right. Unfortunately the contributor hasn't answered and I would like to merge this issue to complete the hackathon. Can you please approve the PR? :-)
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.
thank you :-) the scroll itself is solved; unfortunately the last item is still not visible on small screens. But we can tackle this in a next PR.
* main: (42 commits) chore: add revert to oss-friends (formbricks#745) fix: pass authOptions to getServerSession in authLayout (formbricks#1584) fix: fixed URL example for "Create Survey" API endpoint is wrong formbricks#1555 (formbricks#1586) fix(docs): default account info docs changes (formbricks#1583) chore: Add docker packages to Github Packages on release (formbricks#1585) feat: avatar upload (formbricks#1546) fix: editor crashing (formbricks#1582) fix: Add scroll to the setting navbar (formbricks#1398) are you sure modal missing fix:formbricks#1419 (formbricks#1485) fix: github linting issues (formbricks#1510) refactor: added authorization to airtable integration and create a common actions file (formbricks#1538) docs: add docs for airtable (formbricks#1541) fix: missing static generation store in revalidation due to pages dir (formbricks#1581) feat: Pagination for other values in multi choice (formbricks#1560) feat: $199 pricing model for unlimited plans (formbricks#1564) fix: make pricing pages consistent (formbricks#1567) fix: openText issue (formbricks#1579) fix: avoid blocking the request if it does not have an associated ip (formbricks#1540) feat: Add Unkey to OSS Friends (formbricks#1574) feat: FOR-683 Role Switch (formbricks#1450) ...
* main: (70 commits) chore: add revert to oss-friends (formbricks#745) fix: pass authOptions to getServerSession in authLayout (formbricks#1584) fix: fixed URL example for "Create Survey" API endpoint is wrong formbricks#1555 (formbricks#1586) fix(docs): default account info docs changes (formbricks#1583) chore: Add docker packages to Github Packages on release (formbricks#1585) feat: avatar upload (formbricks#1546) fix: editor crashing (formbricks#1582) fix: Add scroll to the setting navbar (formbricks#1398) are you sure modal missing fix:formbricks#1419 (formbricks#1485) fix: github linting issues (formbricks#1510) refactor: added authorization to airtable integration and create a common actions file (formbricks#1538) docs: add docs for airtable (formbricks#1541) fix: missing static generation store in revalidation due to pages dir (formbricks#1581) feat: Pagination for other values in multi choice (formbricks#1560) feat: $199 pricing model for unlimited plans (formbricks#1564) fix: make pricing pages consistent (formbricks#1567) fix: openText issue (formbricks#1579) fix: avoid blocking the request if it does not have an associated ip (formbricks#1540) feat: Add Unkey to OSS Friends (formbricks#1574) feat: FOR-683 Role Switch (formbricks#1450) ...
What does this PR do?
This PR added scroll to the setting navbar
Fixes #1190
Type of change
How should this be tested?
Checklist
Required
pnpm build
console.logs
git pull origin main
Appreciated