From e3189860ecb9be282fe15e2010927cef30595a28 Mon Sep 17 00:00:00 2001 From: Shubham Padia Date: Thu, 1 Aug 2024 05:48:57 +0000 Subject: [PATCH] css: Use padding-top instead of putting margin-top on first div. It has the same visual effect and we do not have to use a div selector. Having the `div` selector there affects the performance. See https://chat.zulip.org/#narrow/stream/6-frontend/topic/CSS.20selector.20performance/near/1845719 for more details. (cherry picked from commit 2081fb52a1f090213627e1539ed05c4b1e344968) --- web/styles/settings.css | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/web/styles/settings.css b/web/styles/settings.css index f152adc96f..4e040fb88a 100644 --- a/web/styles/settings.css +++ b/web/styles/settings.css @@ -551,8 +551,8 @@ input[type="checkbox"] { } } -.organization-settings-parent > div:first-of-type { - margin-top: 10px; +.organization-settings-parent { + padding-top: 10px; } #id_org_profile_preview { @@ -563,8 +563,8 @@ input[type="checkbox"] { margin-bottom: 20px; } -.inline-block.organization-permissions-parent div:first-of-type { - margin-top: 10px; +.inline-block.organization-permissions-parent { + padding-top: 10px; } .language_selection_widget .language_selection_button {