From a973371cf27946fc18d31cd967c67b5308b91ac1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Emilio=20Cobos=20=C3=81lvarez?= Date: Mon, 9 Jan 2023 18:02:23 +0000 Subject: [PATCH] style: Remove non-standard values of caption-side for good They have been disabled since bug 1688695. There's a variety of table-caption bugs that would be easier to fix if we didn't have to account for side-captions. Differential Revision: https://phabricator.services.mozilla.com/D165690 --- components/style/values/specified/table.rs | 20 +------------------- 1 file changed, 1 insertion(+), 19 deletions(-) diff --git a/components/style/values/specified/table.rs b/components/style/values/specified/table.rs index 81f3414e8f6..88f917ac78d 100644 --- a/components/style/values/specified/table.rs +++ b/components/style/values/specified/table.rs @@ -8,15 +8,9 @@ /// /// Note that despite having "physical" names, these are actually interpreted /// according to the table's writing-mode: Top and Bottom are treated as -/// block-start and -end respectively, and Left and Right are treated as -/// line-start and -end. +/// block-start and -end respectively. /// /// https://drafts.csswg.org/css-tables/#propdef-caption-side -#[cfg(feature = "gecko")] -fn caption_side_non_standard_enabled(_: &crate::parser::ParserContext) -> bool { - static_prefs::pref!("layout.css.caption-side-non-standard.enabled") -} - #[allow(missing_docs)] #[derive( Clone, @@ -39,16 +33,4 @@ fn caption_side_non_standard_enabled(_: &crate::parser::ParserContext) -> bool { pub enum CaptionSide { Top, Bottom, - #[cfg(feature = "gecko")] - #[parse(condition = "caption_side_non_standard_enabled")] - Right, - #[cfg(feature = "gecko")] - #[parse(condition = "caption_side_non_standard_enabled")] - Left, - #[cfg(feature = "gecko")] - #[parse(condition = "caption_side_non_standard_enabled")] - TopOutside, - #[cfg(feature = "gecko")] - #[parse(condition = "caption_side_non_standard_enabled")] - BottomOutside, }