Skip to content

Commit

Permalink
Remove CSS workaround for scroll-anchoring (#1976)
Browse files Browse the repository at this point in the history
Docsy does not set the `height` anymore, but the `min-height`, which doesn't seem to affect scroll-anchoring.
See google/docsy@f22a70e

Signed-off-by: Kévin Commaille <zecakeh@tedomum.fr>
  • Loading branch information
zecakeh authored Oct 29, 2024
1 parent 1f5f2c4 commit 3b8f3a0
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 6 deletions.
6 changes: 0 additions & 6 deletions assets/scss/_styles_project.scss
Original file line number Diff line number Diff line change
Expand Up @@ -26,12 +26,6 @@ Custom SCSS for the Matrix spec
*/
@import "syntax.scss";

/* Workaround for https://github.com/google/docsy/issues/1116:
* fix scroll-anchoring */
.td-outer {
height: auto;
}

/* Overrides for the navbar */
.td-navbar {
box-shadow: 0px 0px 8px rgba(179, 179, 179, 0.25);
Expand Down
1 change: 1 addition & 0 deletions changelogs/internal/newsfragments/1976.clarification
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Remove CSS workaround for scroll-anchoring.

0 comments on commit 3b8f3a0

Please sign in to comment.