From 0c01e0b2fe06afeea3314372a3d71b0a8bdd9655 Mon Sep 17 00:00:00 2001 From: Kris Date: Tue, 16 Jan 2018 19:05:12 -0500 Subject: [PATCH] UX: Implementing a z-index system (#5507) --- .../stylesheets/common/admin/admin_base.scss | 6 +-- .../stylesheets/common/admin/customize.scss | 2 +- .../stylesheets/common/base/_topic-list.scss | 4 +- .../stylesheets/common/base/compose.scss | 4 +- .../common/base/crawler_layout.scss | 2 +- .../stylesheets/common/base/discourse.scss | 2 +- app/assets/stylesheets/common/base/emoji.scss | 4 +- .../stylesheets/common/base/header.scss | 8 +-- .../stylesheets/common/base/menu-panel.scss | 2 +- app/assets/stylesheets/common/base/modal.scss | 12 ++--- .../stylesheets/common/base/onebox.scss | 2 +- .../stylesheets/common/base/share_link.scss | 2 +- .../common/base/topic-admin-menu.scss | 4 +- .../stylesheets/common/base/topic-post.scss | 2 +- app/assets/stylesheets/common/base/topic.scss | 2 +- .../stylesheets/common/components/banner.scss | 2 +- app/assets/stylesheets/common/d-editor.scss | 4 +- .../common/foundation/variables.scss | 50 +++++++++++++++++++ app/assets/stylesheets/common/input_tip.scss | 2 +- .../common/select-kit/select-kit.scss | 4 +- .../stylesheets/common/topic-entrance.scss | 2 +- .../stylesheets/common/topic-timeline.scss | 8 +-- app/assets/stylesheets/desktop/compose.scss | 2 +- app/assets/stylesheets/desktop/discourse.scss | 2 +- app/assets/stylesheets/desktop/modal.scss | 2 +- .../stylesheets/desktop/topic-list.scss | 2 +- .../stylesheets/desktop/topic-post.scss | 18 +++---- app/assets/stylesheets/desktop/topic.scss | 10 ++-- app/assets/stylesheets/desktop/user-card.scss | 2 +- app/assets/stylesheets/mobile/compose.scss | 6 +-- app/assets/stylesheets/mobile/discourse.scss | 2 +- app/assets/stylesheets/mobile/modal.scss | 2 +- app/assets/stylesheets/mobile/topic-list.scss | 4 +- app/assets/stylesheets/mobile/topic-post.scss | 12 ++--- app/assets/stylesheets/mobile/topic.scss | 6 +-- 35 files changed, 125 insertions(+), 75 deletions(-) diff --git a/app/assets/stylesheets/common/admin/admin_base.scss b/app/assets/stylesheets/common/admin/admin_base.scss index e0595b27128..2054ba8037a 100644 --- a/app/assets/stylesheets/common/admin/admin_base.scss +++ b/app/assets/stylesheets/common/admin/admin_base.scss @@ -195,7 +195,7 @@ $mobile-breakpoint: 700px; position: absolute; width: 460px; right: 0px; - z-index: 990; + z-index: z("dropdown"); box-shadow: 0 2px 6px rgba(0,0,0, .8); margin-top: -2px; background-color: $secondary; @@ -403,7 +403,7 @@ $mobile-breakpoint: 700px; // The admin-nav becomes a slide-out menu at the mobile-nav breakpoint @media (max-width: $mobile-breakpoint) { position: absolute; - z-index: 0; + z-index: z("base") - 1; width: 50%; } } @@ -411,7 +411,7 @@ $mobile-breakpoint: 700px; .admin-detail { width: 76.5765%; @media (max-width: $mobile-breakpoint) { - z-index: 10; + z-index: z("base"); width: 100%; } background-color: $secondary; diff --git a/app/assets/stylesheets/common/admin/customize.scss b/app/assets/stylesheets/common/admin/customize.scss index 8dd51a662a5..9a23fb7d738 100644 --- a/app/assets/stylesheets/common/admin/customize.scss +++ b/app/assets/stylesheets/common/admin/customize.scss @@ -61,7 +61,7 @@ bottom: 0; left: 0; right: 0; - z-index: 100000; + z-index: z("fullscreen"); background-color: white; width: 100%; padding: 0; diff --git a/app/assets/stylesheets/common/base/_topic-list.scss b/app/assets/stylesheets/common/base/_topic-list.scss index 75283fefc56..154db643381 100644 --- a/app/assets/stylesheets/common/base/_topic-list.scss +++ b/app/assets/stylesheets/common/base/_topic-list.scss @@ -1,6 +1,6 @@ .show-more { width: 100%; - z-index: 1; + z-index: z("base"); &.has-topics { position: absolute; top: 7px; @@ -259,7 +259,7 @@ ol.category-breadcrumb { position: absolute; border: 1px solid $primary-low; background-color: $secondary; - z-index: 100; + z-index: z("dropdown"); a.badge-category, a.badge-category-parent { line-height: $line-height-large; diff --git a/app/assets/stylesheets/common/base/compose.scss b/app/assets/stylesheets/common/base/compose.scss index 8ab3d0c5cdb..216c6bff30d 100644 --- a/app/assets/stylesheets/common/base/compose.scss +++ b/app/assets/stylesheets/common/base/compose.scss @@ -16,7 +16,7 @@ @media screen and (max-width: 1200px) { min-width: 0; } - z-index: 999; + z-index: z("composer","content"); transition: height 250ms ease, background 250ms ease, transform 250ms ease, max-width 250ms ease; background-color: $secondary; box-shadow: 0 -1px 40px rgba(0,0,0, .12); @@ -271,7 +271,7 @@ } .autocomplete { - z-index: 999999; + z-index: z("dropdown"); position: absolute; width: 240px; background-color: $secondary; diff --git a/app/assets/stylesheets/common/base/crawler_layout.scss b/app/assets/stylesheets/common/base/crawler_layout.scss index beb3208c46f..88d94164627 100644 --- a/app/assets/stylesheets/common/base/crawler_layout.scss +++ b/app/assets/stylesheets/common/base/crawler_layout.scss @@ -3,7 +3,7 @@ body.crawler { width: 100%; position: absolute; top: 0; - z-index: 1001; + z-index: z("max"); background-color: #fff; padding: 10px; box-shadow: 0 2px 4px -1px rgba(0,0,0,0.25); diff --git a/app/assets/stylesheets/common/base/discourse.scss b/app/assets/stylesheets/common/base/discourse.scss index e67069318ad..7c798d7a2bf 100644 --- a/app/assets/stylesheets/common/base/discourse.scss +++ b/app/assets/stylesheets/common/base/discourse.scss @@ -353,7 +353,7 @@ select { top: -18px !important; right: -18px !important; box-shadow: 0 0 0 9999px rgba($primary, 0.85); - z-index: 1040; + z-index: z("modal","overlay"); } .ring-first-notification { diff --git a/app/assets/stylesheets/common/base/emoji.scss b/app/assets/stylesheets/common/base/emoji.scss index 6b4fa8ae5a0..9c14fdc753e 100644 --- a/app/assets/stylesheets/common/base/emoji.scss +++ b/app/assets/stylesheets/common/base/emoji.scss @@ -7,7 +7,7 @@ img.emoji { .emoji-picker { box-shadow: 0 1px 5px rgba(0,0,0,0.4); background-clip: padding-box; - z-index: 10000; + z-index: z("modal","content"); position: fixed; display: none; flex-direction: row; @@ -177,7 +177,7 @@ img.emoji { } .emoji-picker-modal.fadeIn { - z-index: 9999; + z-index: z("modal","overlay"); position: fixed; left: 0; top: 0; diff --git a/app/assets/stylesheets/common/base/header.scss b/app/assets/stylesheets/common/base/header.scss index bd94603e3d1..24ec7986e7b 100644 --- a/app/assets/stylesheets/common/base/header.scss +++ b/app/assets/stylesheets/common/base/header.scss @@ -2,7 +2,7 @@ width: 100%; position: absolute; top: 0; - z-index: 1001; + z-index: z("header"); background-color: $header_background; box-shadow: 0 2px 4px -1px rgba(0,0,0, .25); @@ -110,7 +110,7 @@ position: absolute; top: 100%; left: 0; - z-index: 1101; + z-index: z("header") + 1; // Higher than .menu-panel width: 100%; height: 0; content: ""; @@ -135,7 +135,7 @@ .ring { position: absolute; top: -9px; - z-index: 1; + z-index: z("base"); margin-left: 0; } .header-dropdown-toggle { @@ -143,7 +143,7 @@ } .badge-notification { position: absolute; - z-index: 1; + z-index: z("base"); left: 0; top: -4px; min-width: 6px; diff --git a/app/assets/stylesheets/common/base/menu-panel.scss b/app/assets/stylesheets/common/base/menu-panel.scss index ec25a53c81d..e54693bbcec 100644 --- a/app/assets/stylesheets/common/base/menu-panel.scss +++ b/app/assets/stylesheets/common/base/menu-panel.scss @@ -21,7 +21,7 @@ border: 1px solid $primary-low; box-shadow: 0 2px 2px rgba(0,0,0, .25); background-color: $secondary; - z-index: 1100; + z-index: z("header"); padding: 0.5em; width: 300px; diff --git a/app/assets/stylesheets/common/base/modal.scss b/app/assets/stylesheets/common/base/modal.scss index e472e86d474..62743539eba 100644 --- a/app/assets/stylesheets/common/base/modal.scss +++ b/app/assets/stylesheets/common/base/modal.scss @@ -1,12 +1,12 @@ .modal-open { .dropdown-menu { - z-index: 2050; + z-index: z("modal","dropdown"); } .popover { - z-index: 2060; + z-index: z("modal","popover"); } .tooltip { - z-index: 2070; + z-index: z("modal","tooltip"); } } @@ -26,7 +26,7 @@ right: 0; bottom: 0; left: 0; - z-index: 1040; + z-index: z("modal","overlay"); background-color: #111; &.fade { opacity: 0; @@ -98,7 +98,7 @@ top: 0; width: 100%; height: 100%; - z-index: 1050; + z-index: z("modal","content"); overflow: auto; } .modal-form { @@ -169,7 +169,7 @@ position: relative; } .jsfu-shade { - z-index: 1; + z-index: z("base"); position: absolute; top: 0; left: 0; diff --git a/app/assets/stylesheets/common/base/onebox.scss b/app/assets/stylesheets/common/base/onebox.scss index 21764084cf3..f306c81d384 100644 --- a/app/assets/stylesheets/common/base/onebox.scss +++ b/app/assets/stylesheets/common/base/onebox.scss @@ -376,7 +376,7 @@ aside.onebox.twitterstatus .onebox-body { .onebox.imgur-album { .outer-box { position: absolute; - z-index: 935; + z-index: z("base"); overflow: hidden; font-size: $font-down-1; color: #fff; diff --git a/app/assets/stylesheets/common/base/share_link.scss b/app/assets/stylesheets/common/base/share_link.scss index d5c50914f38..8a5f9145ca4 100644 --- a/app/assets/stylesheets/common/base/share_link.scss +++ b/app/assets/stylesheets/common/base/share_link.scss @@ -3,7 +3,7 @@ #share-link { position: absolute; left: 20px; - z-index: 1200; + z-index: z("dropdown"); box-shadow: 0 1px 5px rgba(0,0,0, .4); background-color: $secondary; padding: 6px 10px 10px 10px; diff --git a/app/assets/stylesheets/common/base/topic-admin-menu.scss b/app/assets/stylesheets/common/base/topic-admin-menu.scss index 7816086dc94..56f5ba16e65 100644 --- a/app/assets/stylesheets/common/base/topic-admin-menu.scss +++ b/app/assets/stylesheets/common/base/topic-admin-menu.scss @@ -4,7 +4,7 @@ position: fixed; top: 120px; right: 10px; - z-index: 999; + z-index: z("dropdown"); height: 35px; } @@ -13,7 +13,7 @@ width: 205px; padding: 10px; border: 1px solid $primary-low; - z-index: 999; + z-index: z("dropdown"); ul { list-style: none; diff --git a/app/assets/stylesheets/common/base/topic-post.scss b/app/assets/stylesheets/common/base/topic-post.scss index db64550c3f9..91c2b11e27a 100644 --- a/app/assets/stylesheets/common/base/topic-post.scss +++ b/app/assets/stylesheets/common/base/topic-post.scss @@ -174,7 +174,7 @@ aside.quote { background-color: blend-primary-secondary(50%); color: dark-light-choose($secondary, $primary); padding: 10px; - z-index: 401; + z-index: z("dropdown"); opacity: 0.9; .d-icon { diff --git a/app/assets/stylesheets/common/base/topic.scss b/app/assets/stylesheets/common/base/topic.scss index 0496b31af5a..85e6072e610 100644 --- a/app/assets/stylesheets/common/base/topic.scss +++ b/app/assets/stylesheets/common/base/topic.scss @@ -9,7 +9,7 @@ .progress-back-container { position: fixed; bottom: 40px; - z-index: 950; + z-index: z("dropdown"); margin-right: 0px; animation-duration: 0.5s; animation-name: button-jump-up; diff --git a/app/assets/stylesheets/common/components/banner.scss b/app/assets/stylesheets/common/components/banner.scss index c96ff0c2035..47e09f43f31 100644 --- a/app/assets/stylesheets/common/components/banner.scss +++ b/app/assets/stylesheets/common/components/banner.scss @@ -7,7 +7,7 @@ background: $tertiary-low; box-shadow: 0 2px 4px -1px rgba(0,0,0, .25); color: $primary; - z-index: 1001; + z-index: z("base") + 1; overflow: auto; &.overlay { diff --git a/app/assets/stylesheets/common/d-editor.scss b/app/assets/stylesheets/common/d-editor.scss index 5583b337787..a762e1b6a5d 100644 --- a/app/assets/stylesheets/common/d-editor.scss +++ b/app/assets/stylesheets/common/d-editor.scss @@ -8,12 +8,12 @@ position: absolute; background-color: black; opacity: 0.8; - z-index: 1000; + z-index: z("modal","overlay"); } .d-editor-modals { position: absolute; - z-index: 1001; + z-index: z("modal","content"); } .d-editor { diff --git a/app/assets/stylesheets/common/foundation/variables.scss b/app/assets/stylesheets/common/foundation/variables.scss index 1fc83e5e089..59d0dfb0bcd 100644 --- a/app/assets/stylesheets/common/foundation/variables.scss +++ b/app/assets/stylesheets/common/foundation/variables.scss @@ -53,6 +53,56 @@ $line-height-large: 1.4; // Normal or small text @import "plugins_variables"; @import "common/foundation/math"; +// Z-index +// -------------------------------------------------- + +$z-layers: ( + "max": 9999, + "fullscreen": 1700, + "modal": ( + "tooltip": 1600, + "popover": 1500, + "dropdown": 1400, + "content": 1300, + "overlay": 1200, + ), + "mobile-composer": 1100, + "header": 1000, + "tooltip": 600, + "composer": ( + "popover": 500, + "content": 400, + ), + "dropdown": 300, + "usercard": 200, + "timeline": 100, + "base": 1 + ); + +@function map-has-nested-keys($map, $keys...) { + @each $key in $keys { + @if not map-has-key($map, $key) { + @return false; + } + $map: map-get($map, $key); + } + @return true; +} + +@function map-deep-get($map, $keys...) { + @each $key in $keys { + $map: map-get($map, $key); + } + @return $map; +} + +@function z($layers...) { + @if not map-has-nested-keys($z-layers, $layers...) { + @warn "No layer defined for `#{inspect($layers...)}` in $z-layers map. Check variables.scss, property omitted."; + } + @return map-deep-get($z-layers, $layers...); +} + // Color utilities // -------------------------------------------------- diff --git a/app/assets/stylesheets/common/input_tip.scss b/app/assets/stylesheets/common/input_tip.scss index ab72d84a33d..0d5de0c8abe 100644 --- a/app/assets/stylesheets/common/input_tip.scss +++ b/app/assets/stylesheets/common/input_tip.scss @@ -5,7 +5,7 @@ position: absolute; display: block; padding: 5px 10px; - z-index: 101; + z-index: z("tooltip"); &.bad { background: $danger-medium; color: white; diff --git a/app/assets/stylesheets/common/select-kit/select-kit.scss b/app/assets/stylesheets/common/select-kit/select-kit.scss index 60b116b3736..72f67b1f0d2 100644 --- a/app/assets/stylesheets/common/select-kit/select-kit.scss +++ b/app/assets/stylesheets/common/select-kit/select-kit.scss @@ -1,5 +1,5 @@ .mobile-view .select-kit.is-expanded { - z-index: 1000; + z-index: z("dropdown"); } .select-box-kit, .select-kit { @@ -30,7 +30,7 @@ } &.is-expanded { - z-index: 999; + z-index: z("dropdown"); .select-kit-body { display: -webkit-box; diff --git a/app/assets/stylesheets/common/topic-entrance.scss b/app/assets/stylesheets/common/topic-entrance.scss index 40b48283f47..f615b3a2b3e 100644 --- a/app/assets/stylesheets/common/topic-entrance.scss +++ b/app/assets/stylesheets/common/topic-entrance.scss @@ -4,7 +4,7 @@ padding: 5px; background: $secondary; box-shadow: 0 0px 2px rgba(0,0,0, .2); - z-index: 100; + z-index: z("dropdown"); position: absolute; width: 133px; diff --git a/app/assets/stylesheets/common/topic-timeline.scss b/app/assets/stylesheets/common/topic-timeline.scss index ccf9a55b912..a81771d585d 100644 --- a/app/assets/stylesheets/common/topic-timeline.scss +++ b/app/assets/stylesheets/common/topic-timeline.scss @@ -4,7 +4,7 @@ .timeline-container { box-sizing: border-box; - z-index: 499; + z-index: z("timeline"); margin-left: 757px; @media all and (min-width: 1140px) { @@ -51,7 +51,7 @@ border-top: 1px solid dark-light-choose($primary-low, $secondary-low); box-shadow: 0px -2px 4px -1px rgba(0,0,0,.25); padding-top: 20px; - z-index: 100000; + z-index: z("fullscreen"); .back-button { display: none; } @@ -225,7 +225,7 @@ background-color: dark-light-choose($tertiary-low, $tertiary-high); height: 100%; float: left; - z-index: 501; + z-index: z("base") + 1; } .timeline-scroller-content { @@ -261,7 +261,7 @@ color: $tertiary; margin-right: 1em; } - z-index: 502; + z-index: z("base") + 2; // above timeline-handle } .back-button { diff --git a/app/assets/stylesheets/desktop/compose.scss b/app/assets/stylesheets/desktop/compose.scss index 76d6ddf9f38..7a55eb77eb8 100644 --- a/app/assets/stylesheets/desktop/compose.scss +++ b/app/assets/stylesheets/desktop/compose.scss @@ -73,7 +73,7 @@ bottom: 10px; left: 51%; overflow-y: auto; - z-index: 110; + z-index: z("composer","popover"); padding: 10px 10px 35px 10px; box-shadow: 3px 3px 3px rgba(0,0,0, 0.34); background: $highlight-medium; diff --git a/app/assets/stylesheets/desktop/discourse.scss b/app/assets/stylesheets/desktop/discourse.scss index 9f5e6980034..bf9fa9b2fe7 100644 --- a/app/assets/stylesheets/desktop/discourse.scss +++ b/app/assets/stylesheets/desktop/discourse.scss @@ -140,7 +140,7 @@ input { vertical-align: middle; &:focus { - z-index: 2; + z-index: z("base") + 1; } } diff --git a/app/assets/stylesheets/desktop/modal.scss b/app/assets/stylesheets/desktop/modal.scss index e43777c75a1..9d3dd6a25b1 100644 --- a/app/assets/stylesheets/desktop/modal.scss +++ b/app/assets/stylesheets/desktop/modal.scss @@ -10,7 +10,7 @@ position: fixed; top: 50%; left: 50%; - z-index: 1050; + z-index: z("modal","content"); overflow: auto; width: 610px; height: auto; diff --git a/app/assets/stylesheets/desktop/topic-list.scss b/app/assets/stylesheets/desktop/topic-list.scss index 1c6c0a39708..33d615f672a 100644 --- a/app/assets/stylesheets/desktop/topic-list.scss +++ b/app/assets/stylesheets/desktop/topic-list.scss @@ -268,7 +268,7 @@ top: 130px; padding: 5px; background-color: $secondary; - z-index: 99999; + z-index: z("dropdown"); } button.dismiss-read { diff --git a/app/assets/stylesheets/desktop/topic-post.scss b/app/assets/stylesheets/desktop/topic-post.scss index ef82dfce383..281d18cbf49 100644 --- a/app/assets/stylesheets/desktop/topic-post.scss +++ b/app/assets/stylesheets/desktop/topic-post.scss @@ -27,7 +27,7 @@ h1 .topic-statuses .topic-status i { } .reply-to-tab { - z-index: 400; + z-index: z("base") + 1; float: right; margin: -2px 25px 0 0; color: dark-light-choose($primary-medium, $secondary-medium); @@ -177,7 +177,7 @@ nav.post-controls { text-align: left; bottom: -2px; right: 15px; - z-index: 1000; + z-index: z("dropdown"); h3 { margin-top: 0; @@ -256,7 +256,7 @@ nav.post-controls { } .topic-meta-data h5 { position: absolute; - z-index: 1; + z-index: z("base"); font-size: $font-down-1; a { font-weight: bold; @@ -613,7 +613,7 @@ $topic-avatar-width: 45px; width: $topic-body-width; float: left; position: relative; - z-index: 2; + z-index: z("base"); border-top: 1px solid $primary-low; padding: 12px $topic-body-width-padding 15px $topic-body-width-padding; } @@ -623,7 +623,7 @@ $topic-avatar-width: 45px; padding-top: 15px; width: $topic-avatar-width; float: left; - z-index: 3; + z-index: z("base"); } .gap { @@ -659,7 +659,7 @@ $topic-avatar-width: 45px; position: absolute; bottom: 115%; left: 0; - z-index: 999; + z-index: z("dropdown"); display: none; float: left; width: 550px; @@ -724,7 +724,7 @@ $topic-avatar-width: 45px; .selected-posts { width: 200px; position: fixed; - z-index: 1000; + z-index: z("dropdown"); background-color: $tertiary-low; border: 1px solid $tertiary; padding: 5px; @@ -815,7 +815,7 @@ $topic-avatar-width: 45px; .select-posts { position: absolute; right: 0; - z-index: 490; + z-index: z("base") + 1; top: 3em; height: 100px; @@ -847,7 +847,7 @@ a.attachment:before { color: rgba($primary-low, .8); font: 6.429em/1 FontAwesome; content: "\f014"; - z-index: -5; + z-index: z("base"); } .topic-meta-data { diff --git a/app/assets/stylesheets/desktop/topic.scss b/app/assets/stylesheets/desktop/topic.scss index 1d5cf118c66..7e5b63bef72 100644 --- a/app/assets/stylesheets/desktop/topic.scss +++ b/app/assets/stylesheets/desktop/topic.scss @@ -17,7 +17,7 @@ .edits {margin-right: 5px;} #topic-title { - z-index: 1000; + z-index: z("base"); padding-top: 14px; margin-bottom: 10px; @@ -81,7 +81,7 @@ right: 50%; width: 0; bottom: 0; - z-index: 500; + z-index: z("timeline"); &.docked { position: absolute; bottom: -70px; @@ -155,13 +155,13 @@ top: 9px; width: 100%; text-align: center; - z-index: 1; + z-index: z("base"); } .d-icon { position: absolute; right: 8px; bottom: 9px; - z-index: 1; + z-index: z("base"); } h4 { display: inline; @@ -195,7 +195,7 @@ position: fixed; width: 100%; font-size: $font-0; - z-index: 495 + z-index: z("dropdown"); } @media all diff --git a/app/assets/stylesheets/desktop/user-card.scss b/app/assets/stylesheets/desktop/user-card.scss index 3134af4628c..e55a5acea58 100644 --- a/app/assets/stylesheets/desktop/user-card.scss +++ b/app/assets/stylesheets/desktop/user-card.scss @@ -9,7 +9,7 @@ $user_card_background: $secondary; width: 500px; left: -9999px; top: -9999px; - z-index: 990; + z-index: z("usercard"); box-shadow: 1px 2px 6px rgba(0,0,0, .25); margin-top: -2px; color: $user_card_primary; diff --git a/app/assets/stylesheets/mobile/compose.scss b/app/assets/stylesheets/mobile/compose.scss index 0afe0dcb057..05a71110da9 100644 --- a/app/assets/stylesheets/mobile/compose.scss +++ b/app/assets/stylesheets/mobile/compose.scss @@ -7,7 +7,7 @@ } #reply-control { - z-index: 1002; //d-header is 1001 + z-index: z("mobile-composer"); .reply-area { padding: 0 10px; @media screen and (max-width: 374px) { @@ -81,7 +81,7 @@ &.show-preview { .d-editor-preview-wrapper { position: fixed; - z-index: 1000000; + z-index: z("base") + 1; top: 0; bottom: 0; left: 0; @@ -99,7 +99,7 @@ position: fixed; right: 5px; bottom: 5px; - z-index: 1000001; + z-index: z("base") + 2; } } diff --git a/app/assets/stylesheets/mobile/discourse.scss b/app/assets/stylesheets/mobile/discourse.scss index 47ac9efc471..88f270868fe 100644 --- a/app/assets/stylesheets/mobile/discourse.scss +++ b/app/assets/stylesheets/mobile/discourse.scss @@ -112,7 +112,7 @@ blockquote { left: 0; display: block; position: absolute; - z-index: 10000000; + z-index: z("dropdown"); background-color: $secondary; width: 100%; list-style: none; diff --git a/app/assets/stylesheets/mobile/modal.scss b/app/assets/stylesheets/mobile/modal.scss index adc091b0c34..ee7453ecfb2 100644 --- a/app/assets/stylesheets/mobile/modal.scss +++ b/app/assets/stylesheets/mobile/modal.scss @@ -12,7 +12,7 @@ .bootbox.modal { position: fixed; top: 10%; - z-index: 1050; + z-index: z("modal","content"); overflow: auto; width: 100%; height: auto; diff --git a/app/assets/stylesheets/mobile/topic-list.scss b/app/assets/stylesheets/mobile/topic-list.scss index 6c8500053f9..c2808e80a51 100644 --- a/app/assets/stylesheets/mobile/topic-list.scss +++ b/app/assets/stylesheets/mobile/topic-list.scss @@ -28,7 +28,7 @@ .nav-pills .drop { border: 1px solid $primary-low; position: absolute; - z-index: 1000; + z-index: z("dropdown"); background-color: $secondary; padding: 0 10px 10px 10px; width: 150px; @@ -342,7 +342,7 @@ tr.category-topic-link { position: relative; top: 100%; left: 0; - z-index: 1000; + z-index: z("dropdown"); display: none; float: left; width: 280px; diff --git a/app/assets/stylesheets/mobile/topic-post.scss b/app/assets/stylesheets/mobile/topic-post.scss index 399ff28484d..4d827a511e6 100644 --- a/app/assets/stylesheets/mobile/topic-post.scss +++ b/app/assets/stylesheets/mobile/topic-post.scss @@ -79,7 +79,7 @@ span.badge-posts { text-align: left; bottom: 0; left: 135px; - z-index: 1000; + z-index: z("dropdown"); h3 { margin-top: 0; @@ -125,7 +125,7 @@ span.badge-posts { } a.reply-to-tab { - z-index: 400; + z-index: z("base") + 1; right: 80px; color: dark-light-choose($primary-medium, $secondary-medium); margin: -3px 10px 0 0; @@ -311,7 +311,7 @@ span.post-count { } #topic-title { - z-index: 1000; + z-index: z("base") + 1; margin: 0 0 0 0 !important; padding: 15px 0; } @@ -329,7 +329,7 @@ span.post-count { .quote-button.visible { display: block; - z-index: 936; + z-index: z("tooltip"); } iframe { @@ -397,7 +397,7 @@ iframe { button.select-post { display: inline-block; position: absolute; - z-index: 401; // 400 is the reply-to tab + z-index: z("base") + 1; left: 200px; background-color: srgb-scale($tertiary, $secondary, 60%); color: $secondary; @@ -443,7 +443,7 @@ span.highlighted { .topic-avatar { float: left; margin-right: 10px; - z-index:999; /* must render on top of topic-body + topic-meta-data, otherwise not tappable */ + z-index: z("base") + 1; /* must render on top of topic-body + topic-meta-data, otherwise not tappable */ } .topic-meta-data { diff --git a/app/assets/stylesheets/mobile/topic.scss b/app/assets/stylesheets/mobile/topic.scss index 0af940cc9e0..7b14a58c3f6 100644 --- a/app/assets/stylesheets/mobile/topic.scss +++ b/app/assets/stylesheets/mobile/topic.scss @@ -61,7 +61,7 @@ width: 0; right: 0; bottom: 0; - z-index: 950; + z-index: z("timeline"); margin-right: 148px; } @@ -126,7 +126,7 @@ top: 9px; width: 100%; text-align: center; - z-index: 1; + z-index: z("base"); } h4 { display: inline; @@ -137,7 +137,7 @@ position: absolute; right: 8px; bottom: 9px; - z-index: 1; + z-index: z("base"); } .bg { position: absolute;