Skip to content

Commit 08793fb

Browse files
rhamiltof0x11
rhamilto
authored and
f0x11
committed
Removing orphaned tech-preview less and markup
Fixes openshift#2186 openshift@128b78d360f3d8999 2ed6ff7813edaa99f315ef1#diff-685dfa901a119f6590e773891ffd6052L506 removed the relevant javascript bits.
1 parent 2ee2b08 commit 08793fb

13 files changed

+12
-116
lines changed

Diff for: ‎app/index.html

-1
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,6 @@
3939
</head>
4040
<body class="console-os" ng-class="{ 'has-project-bar': view.hasProject }">
4141
<!-- Add your site or application content here -->
42-
<p ng-if="globalTechPreviewIndicator" ng-cloak class="tech-preview-banner">Technology preview is enabled</p>
4342
<osc-header></osc-header>
4443
<toast-notifications></toast-notifications>
4544
<notification-drawer-wrapper></notification-drawer-wrapper>

Diff for: ‎app/styles/_catalog.less

-6
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,6 @@
44
.landing {
55
@media(min-width: @screen-sm-min) {
66
top: (@navbar-os-header-height-desktop + @landing-search-area-height);
7-
.tech-preview & {
8-
top: (@navbar-os-header-height-desktop + @tech-preview-banner-height + @landing-search-area-height);
9-
}
107
}
118
h1, .nav-tabs {
129
margin-bottom: 0 !important;
@@ -24,8 +21,5 @@
2421
.catalog-search,
2522
.landing-side-bar {
2623
top: @navbar-os-header-height-desktop;
27-
.tech-preview & {
28-
top: @navbar-os-header-height-desktop + @tech-preview-banner-height;
29-
}
3024
}
3125
}

Diff for: ‎app/styles/_layouts.less

-12
Original file line numberDiff line numberDiff line change
@@ -11,25 +11,13 @@
1111
body {
1212
height: 1px;
1313
padding-top: @navbar-os-header-height-mobile;
14-
&.tech-preview {
15-
padding-top: (@navbar-os-header-height-mobile + @tech-preview-banner-height);
16-
}
1714
@media(min-width: @screen-sm-min) {
1815
padding-top: @navbar-os-header-height-desktop;
19-
&.tech-preview {
20-
padding-top: (@navbar-os-header-height-desktop + @tech-preview-banner-height);
21-
}
2216
}
2317
&.has-project-bar {
2418
padding-top: (@navbar-os-header-height-mobile + @project-bar-height-mobile);
25-
&.tech-preview {
26-
padding-top: (@navbar-os-header-height-mobile + @project-bar-height-mobile + @tech-preview-banner-height);
27-
}
2819
@media(min-width: @screen-sm-min) {
2920
padding-top: (@navbar-os-header-height-desktop + @project-bar-height-desktop);
30-
&.tech-preview {
31-
padding-top: (@navbar-os-header-height-desktop + @project-bar-height-desktop + @tech-preview-banner-height);
32-
}
3321
.container-pf-nav-pf-vertical {
3422
margin-left: @nav-pf-vertical-width;
3523
&.collapsed-nav {

Diff for: ‎app/styles/_log.less

-12
Original file line numberDiff line numberDiff line change
@@ -135,18 +135,6 @@
135135
top: (@log-scroll-offset-top + @navbar-os-header-height-desktop);
136136
}
137137
}
138-
.tech-preview & {
139-
top: (@log-scroll-offset-top + @navbar-os-header-height-mobile + @project-bar-height-mobile + @tech-preview-banner-height);
140-
@media (min-width: @screen-sm-min) {
141-
top: (@log-scroll-offset-top + @navbar-os-header-height-desktop + @project-bar-height-desktop + @tech-preview-banner-height);
142-
}
143-
}
144-
.tech-preview .chromeless & {
145-
top: (@log-scroll-offset-top + @navbar-os-header-height-mobile + @tech-preview-banner-height);
146-
@media (min-width: @screen-sm-min) {
147-
top: (@log-scroll-offset-top + @navbar-os-header-height-desktop + @tech-preview-banner-height);
148-
}
149-
}
150138
}
151139
}
152140
.log-view-output {

Diff for: ‎app/styles/_navbar-vertical.less

-3
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,6 @@
3636
vertical-align: middle;
3737
width: 0;
3838
}
39-
.layout-pf-fixed .tech-preview & {
40-
top: @tech-preview-banner-height;
41-
}
4239
.navbar-brand {
4340
padding: 0;
4441
@media(min-width: @screen-sm-min) {

Diff for: ‎app/styles/_notifications.less

-8
Original file line numberDiff line numberDiff line change
@@ -36,14 +36,6 @@ notification-drawer-wrapper {
3636
&.hide-remove {
3737
display: block !important;
3838
}
39-
.tech-preview & {
40-
height: calc(100vh ~"-" (@navbar-os-header-height-mobile + @project-bar-height-mobile + @tech-preview-banner-height));
41-
top: (@navbar-os-header-height-mobile + @project-bar-height-mobile + @tech-preview-banner-height);
42-
@media(min-width: @screen-sm-min) {
43-
height: calc(100vh ~"-" (@navbar-os-header-height-desktop + @project-bar-height-desktop + @tech-preview-banner-height) ~"-" 20px); // 20px bottom gutter
44-
top: (@navbar-os-header-height-desktop + @project-bar-height-desktop + @tech-preview-banner-height);
45-
}
46-
}
4739
}
4840
// the whole block is clickable, need to set pointer on all of these
4941
// for the correct visual

Diff for: ‎app/styles/_project-bar.less

-6
Original file line numberDiff line numberDiff line change
@@ -239,10 +239,4 @@
239239
background-color: @navbar-pf-vertical-color;
240240
}
241241
}
242-
.tech-preview & {
243-
top: @navbar-os-header-height-mobile + @tech-preview-banner-height;
244-
@media (min-width: @screen-sm-min) {
245-
top: @navbar-os-header-height-desktop + @tech-preview-banner-height;
246-
}
247-
}
248242
}

Diff for: ‎app/styles/_tech-preview.less

-15
This file was deleted.

Diff for: ‎app/styles/_variables.less

-1
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,6 @@
8989
@scrollbar-width: 15px;
9090
@status-bg-color: #E6ECF1;
9191
@status-border-color: #BFCEDB;
92-
@tech-preview-banner-height: 20px;
9392
@tileHeadingIconFontSize: 33px;
9493
@tooltip-bg: #111;
9594

Diff for: ‎app/styles/_vertical-nav.less

-18
Original file line numberDiff line numberDiff line change
@@ -130,12 +130,6 @@
130130
}
131131
}
132132
}
133-
.tech-preview & {
134-
top: (@navbar-os-header-height-mobile + @project-bar-height-mobile + @tech-preview-banner-height);
135-
@media(min-width: @screen-sm-min) {
136-
top: (@navbar-os-header-height-desktop + @project-bar-height-desktop + @tech-preview-banner-height);
137-
}
138-
}
139133
}
140134
.nav-item-pf-header {
141135
font-size: (@font-size-base + 1);
@@ -144,16 +138,4 @@
144138
padding: 4px 0 2px 0;
145139
margin-left: 20px;
146140
}
147-
.tech-preview & {
148-
top: (@navbar-os-header-height-mobile + @tech-preview-banner-height);
149-
@media(min-width: @screen-sm-min) {
150-
top: (@navbar-os-header-height-desktop + @tech-preview-banner-height);
151-
}
152-
}
153-
.tech-preview.has-project-bar & {
154-
top: (@navbar-os-header-height-mobile + @project-bar-height-mobile + @tech-preview-banner-height);
155-
@media(min-width: @screen-sm-min) {
156-
top: (@navbar-os-header-height-desktop + @project-bar-height-desktop + @tech-preview-banner-height);
157-
}
158-
}
159141
}

Diff for: ‎app/styles/main.less

-1
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,6 @@
4646
@import "_system-status.less";
4747
@import "_tables.less";
4848
@import "_tile.less";
49-
@import "_tech-preview.less";
5049
@import "_tooltip.less";
5150
@import "_typography.less";
5251
@import "_spacers.less";

Diff for: ‎dist/index.html

-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@
2424
<style type="text/css"></style>
2525
</head>
2626
<body class="console-os" ng-class="{ 'has-project-bar': view.hasProject }">
27-
<p ng-if="globalTechPreviewIndicator" ng-cloak class="tech-preview-banner">Technology preview is enabled</p>
2827
<osc-header></osc-header>
2928
<toast-notifications></toast-notifications>
3029
<notification-drawer-wrapper></notification-drawer-wrapper>

Diff for: ‎dist/styles/main.css

+12-32
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)