[Docs] Default Permissions: add spacing between permission tables#1091
[Docs] Default Permissions: add spacing between permission tables#1091PARTH-TUSSLE wants to merge 2 commits into
Conversation
Signed-off-by: PARTH-TUSSLE <parthgartan26feb@gmail.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds margin styles for h2 elements following a .csvtable in assets/scss/_content_project.scss. The reviewer suggested using .td-initial + h2 instead of .csvtable + h2 to maintain consistency with the existing stylesheet.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| .csvtable + h2 { | ||
| margin-top: 3rem; | ||
| margin-bottom: 1rem; | ||
| } |
There was a problem hiding this comment.
To maintain consistency with the existing stylesheet which uses the .td-initial class for styling these tables, consider using .td-initial + h2 instead of .csvtable + h2. This avoids relying on the specific .csvtable class in the global stylesheet and keeps the class usage unified.
| .csvtable + h2 { | |
| margin-top: 3rem; | |
| margin-bottom: 1rem; | |
| } | |
| .td-initial + h2 { | |
| margin-top: 3rem; | |
| margin-bottom: 1rem; | |
| } |
There was a problem hiding this comment.
@PARTH-TUSSLE Please review the Gemini suggestions and verify whether they're correct. If they are, please apply the changes; otherwise, reject them with the appropriate reason.
There was a problem hiding this comment.
@rishiraj38 , I've made the suitable changes according to Gemini suggestions.
|
🚀 Preview deployment: https://layer5io.github.io/docs/pr-preview/pr-1091/
|
|
@PARTH-TUSSLE Thank you for your contribution! Let's discuss this during the website call tomorrow at 5:30 PM IST | 7 AM CST Add it as an agenda item to the meeting minutes, if you would 🙂 |
Signed-off-by: PARTH-TUSSLE <parthgartan26feb@gmail.com>
3 participants
Notes for Reviewers
Added vertical spacing between permission tables on the Default Permissions page to improve readability.
Preview of the fix :-
This PR fixes #1075
Signed commits