From f261919feebad7e591ed08770c354ef2a1f3305c Mon Sep 17 00:00:00 2001 From: Andrew Morgan Date: Tue, 8 Nov 2022 16:35:53 +0000 Subject: [PATCH] Add long-winded comment about why display: table is needed --- assets/scss/custom.scss | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/assets/scss/custom.scss b/assets/scss/custom.scss index a491d917..528af5a0 100644 --- a/assets/scss/custom.scss +++ b/assets/scss/custom.scss @@ -340,6 +340,16 @@ footer { } table { + /* Docsy makes all tables "responsive tables", which causes Bootstrap 4 to create + * tables with a "display" property of "block". + * + * However, for "table-layout: fixed" to be effective, an element must have a + * "display" property of "table". + * + * Thus, we override the "display" property here. This may no longer be necessary once + * Docsy updates to Bootstrap v5+: https://github.com/google/docsy/issues/470. + * For more details, see + * https://github.com/matrix-org/matrix-spec/pull/1295/files#r1010759688 */ display: table; table-layout: fixed; width: 100%;