Skip to content

Commit

Permalink
Update scroll-margin-top (#8641)
Browse files Browse the repository at this point in the history
## Describe your changes

The app header bar height changed in
#8554 and now the heading can
be overlayed by it when clicking on the anchor. This PR fixes this by
using the correct height.

## GitHub Issue Link (if applicable)

## Testing Plan

- manual tests

---

**Contribution License Agreement**

By submitting this pull request you agree that all contributions to this
project are made under the Apache 2.0 license.
  • Loading branch information
raethlein committed May 10, 2024
1 parent 92e41ac commit 9ef698d
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -154,7 +154,7 @@ export const StyledLinkIcon = styled.a(({ theme }) => ({

export const StyledHeadingWithActionElements = styled.div(({ theme }) => ({
"h1, h2, h3, h4, h5, h6, span": {
scrollMarginTop: theme.spacing.threeXL,
scrollMarginTop: theme.sizes.headerHeight,
},
...sharedMarkdownStyle(theme),

Expand Down

0 comments on commit 9ef698d

Please sign in to comment.