[Docs] Hide extra scroll bar from the code snippets#30948
[Docs] Hide extra scroll bar from the code snippets#30948samiahmedsiddiqui wants to merge 2 commits intoyugabyte:masterfrom
Conversation
✅ Deploy Preview for infallible-bardeen-164bc9 ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
![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 modifies the SCSS for code blocks to set the overflow property to visible for pre elements. A review comment points out that the current implementation nests this rule within a comma-separated selector, which generates an unintended CSS selector (pre pre). The feedback suggests isolating the rule to target .highlight pre specifically to avoid this issue.
| pre{ | ||
| overflow: visible; | ||
| } | ||
| } |
There was a problem hiding this comment.
This nested rule is inside a comma-separated selector (> pre, .highlight), which will cause the generated CSS to be .td-content > pre pre, .td-content .highlight pre. The first selector, .td-content > pre pre, targets a pre element inside another pre element and is likely unintended. To ensure this style only applies to pre elements within .highlight, this rule should be moved out of this shared block and defined separately for .highlight.
| pre{ | |
| overflow: visible; | |
| } | |
| } | |
| } | |
| .highlight pre { | |
| overflow: visible; | |
| } |
1 participant
@netlify /stable/releases/yugabyte-clients/