Skip to content

Commit

Permalink
Addition of ancillary css partials to support bootstrap and pf3 while…
Browse files Browse the repository at this point in the history
… their dependencies are removed from package.json. In conjunction with their removal we are enabling the pf4 reset and removing the shield-inheritables rules. Also additional changes to rules and variables no longer in use.
  • Loading branch information
sg00dwin committed Aug 4, 2021
1 parent 9e2206e commit 5ed38f8
Show file tree
Hide file tree
Showing 62 changed files with 1,725 additions and 451 deletions.
2 changes: 0 additions & 2 deletions frontend/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,6 @@
"murmurhash-js": "1.0.x",
"null-loader": "^3.0.0",
"openshift-logos-icon": "1.7.1",
"patternfly": "3.59.5",
"pluralize": "^8.0.0",
"point-in-svg-path": "1.0.1",
"popper.js": "^1.15.0",
Expand Down Expand Up @@ -247,7 +246,6 @@
"acorn": "^7.0.0",
"acorn-jsx": "5.2.0",
"babel-loader": "^8.1.0",
"bootstrap-sass": "^3.3.7",
"browser-env": "3.x",
"cache-loader": "1.x",
"chalk": "2.4.x",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,12 +1,10 @@
@import '~patternfly/dist/sass/patternfly/color-variables';

.co-cloudshell-exec {
&__container-error {
background-color: var(--pf-global--BackgroundColor--100);
height: 100%;
}

&__error-msg {
color: $color-pf-red !important;
color: var(--pf-global--danger-color--100) !important;
}
}
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
@import '~bootstrap-sass/assets/stylesheets/bootstrap/variables';
@import '../../../../../../public/style/vars';
@import '../../../../../console-shared/src/styles/skeleton-screen.scss';

.co-volume-snapshot__body {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
@import '~@patternfly/patternfly/sass-utilities/colors';
@import '~bootstrap-sass/assets/stylesheets/bootstrap/variables';
@import '../../../../../public/style/vars';

.co-dashboard-body {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,7 @@ const ResourceQuotaItem: React.FC<ResourceQuotaItemProps> = ({ resourceQuota })
namespace={resourceQuota.metadata.namespace}
inline="true"
/>
{scopes && (
<QuotaScopesInline className="co-resource-quota-dashboard-scopes" scopes={scopes} />
)}
{scopes && <QuotaScopesInline scopes={scopes} />}
</div>
<QuotaGaugeCharts
quota={resourceQuota}
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
@import '~bootstrap-sass/assets/stylesheets/bootstrap/variables';
@import '../../../../../public/style/vars';

.ocs-yaml-editor-toolbar {
align-items: center;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
@import '../../../../../public/style/vars';
@import '~bootstrap-sass/assets/stylesheets/bootstrap/variables';

.ocs-form-footer {
padding: var(--pf-global--spacer--md) ($grid-gutter-width/2);
Expand Down
6 changes: 0 additions & 6 deletions frontend/packages/operator-lifecycle-manager/src/style.scss
Original file line number Diff line number Diff line change
@@ -1,9 +1,3 @@
// Bootstrap Core variables and mixins
@import '~bootstrap-sass/assets/stylesheets/bootstrap/variables';

// Patternfly Core variables and mixins
@import '~patternfly/dist/sass/patternfly/variables';

// OpenShift variables
@import '../../../public/style/vars';

Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,4 @@
@import '../../../../../public/style/vars';
@import '../../../../../node_modules/bootstrap-sass/assets/stylesheets/bootstrap/variables';
@import '~@patternfly/patternfly/sass-utilities/colors';

.odc-topology-list-view {
--odc-topology-list-view-type-icon-size: 16px;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
@import '../../../../../../node_modules/bootstrap-sass/assets/stylesheets/bootstrap/variables';
@import '../../../../../../public/style/vars';

.odc-topology-list-view__alert-cell {
min-width: 0; // To enable text-overflow: ellipsis within a css-grid area
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
@import '../../../../../../public/style/vars';
@import '../../../../../../node_modules/bootstrap-sass/assets/stylesheets/bootstrap/variables';

.odc-topology-list-view__metrics-cell {
&__metric-tooltip {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
@import '../../../../../../public/style/vars';
@import '../../../../../../node_modules/bootstrap-sass/assets/stylesheets/bootstrap/variables';

.odc-topology-list-view__metrics-cell {
&__metric-tooltip {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
@import '../../../../../../node_modules/bootstrap-sass/assets/stylesheets/bootstrap/variables';
@import '../../../../../../public/style/vars';

.odc-topology-list-view__detail--status {
text-align: right;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
@import '../../../../public/style/vars';
@import '../../../../node_modules/bootstrap-sass/assets/stylesheets/bootstrap/variables';

.odc-topology-filter-bar {
background-color: var(--pf-chart-global--Fill--Color--white);
Expand Down
14 changes: 7 additions & 7 deletions frontend/public/components/_build-pipeline.scss
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
$pipeline-aborted-color: $color-pf-black-300;
$pipeline-border-color: $color-pf-black-300;
$pipeline-failed-color: $color-pf-red-100;
$pipeline-aborted-color: $pf-color-black-300;
$pipeline-border-color: $pf-color-black-300;
$pipeline-failed-color: $pf-color-red-100;
$pipeline-font-base: 13px;
$pipeline-in-progress-color: $color-pf-blue-400;
$pipeline-new-color: $color-pf-blue-100;
$pipeline-pending-color: $color-pf-black-300;
$pipeline-in-progress-color: $pf-color-blue-400;
$pipeline-new-color: $pf-color-blue-100;
$pipeline-pending-color: $pf-color-black-300;
$pipeline-success-color: var(--pf-chart-color-green-400);
$pipeline-require-attention-color: $color-pf-gold-400;
$pipeline-require-attention-color: $pf-color-gold-400;
$pipeline-circle-animation-time: 0.35s;
$pipeline-circle-diameter: 18px;
$pipeline-line-border-width: 8px;
Expand Down
4 changes: 2 additions & 2 deletions frontend/public/components/_deploy-image.scss
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,8 @@
}

.co-image-name-results {
border-bottom: 1px solid $color-pf-black-200;
border-top: 1px solid $color-pf-black-200;
border-bottom: 1px solid $pf-color-black-200;
border-top: 1px solid $pf-color-black-200;
margin: 30px 0;
padding: 30px 0;

Expand Down
2 changes: 1 addition & 1 deletion frontend/public/components/_detail-table.scss
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@

&,
&__section {
border-color: $color-pf-black-200;
border-color: $pf-color-black-200;
border-style: solid;
}

Expand Down
10 changes: 5 additions & 5 deletions frontend/public/components/_dropdown.scss
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
@import '~@patternfly/patternfly/sass-utilities/scss-variables';

$color-bookmarker--hover: $color-pf-black-500;
$color-bookmarker: $color-pf-black-400;
$dropdown-background--hover: $color-pf-black-200; // pf-c-dropdown__menu-item--hover--BackgroundColor
$color-bookmarker--hover: $pf-color-black-500;
$color-bookmarker: $pf-color-black-400;
$dropdown-background--hover: $pf-color-black-200; // pf-c-dropdown__menu-item--hover--BackgroundColor

.dropdown--full-width {
&,
Expand All @@ -27,15 +27,15 @@ $dropdown-background--hover: $color-pf-black-200; // pf-c-dropdown__menu-item--h
}

.dropdown-menu__divider {
background-color: $color-pf-black-200;
background-color: $pf-color-black-200;
height: 1px;
flex-grow: 1;
margin: 9px 0;
overflow: hidden;
}

.dropdown-menu__header {
color: $color-pf-black-600;
color: $pf-color-black-600;
flex-grow: 1;
font-size: ($font-size-base - 1);
padding: 3px 10px;
Expand Down
4 changes: 2 additions & 2 deletions frontend/public/components/_global-notification.scss
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
$global-notification-text: $color-pf-white;
$global-notification-text: $pf-color-white;

.co-global-notification {
background-color: $color-pf-blue-400;
background-color: $pf-color-blue-400;
color: $global-notification-text;
padding: 4px ($grid-gutter-width / 2);
text-align: center;
Expand Down
12 changes: 6 additions & 6 deletions frontend/public/components/_horizontal-nav.scss
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ $co-m-horizontal-nav__menu-item-link-padding-lr: ($grid-gutter-width / 2);
display: flex;
flex-shrink: 0; // prevent collapse of tabs
list-style: none;
margin: 0;
margin: 0 !important;
overflow-x: auto;
overflow-y: hidden;
padding: 0;
Expand All @@ -16,7 +16,7 @@ $co-m-horizontal-nav__menu-item-link-padding-lr: ($grid-gutter-width / 2);
}
&--within-sidebar {
border-top: 1px solid $color-grey-background-border;
margin-bottom: $grid-gutter-width;
margin-bottom: $grid-gutter-width !important;
margin-left: -($grid-gutter-width / 2);
margin-right: -($grid-gutter-width / 2);
overflow: visible; // so focus indicator is not clipped since these don't need to scroll
Expand All @@ -42,7 +42,7 @@ $co-m-horizontal-nav__menu-item-link-padding-lr: ($grid-gutter-width / 2);
button {
background: transparent;
border: 0;
color: $color-pf-black-700;
color: $pf-color-black-700;
display: block;
padding: 6px $co-m-horizontal-nav__menu-item-link-padding-lr 8px;
position: relative;
Expand All @@ -52,7 +52,7 @@ $co-m-horizontal-nav__menu-item-link-padding-lr: ($grid-gutter-width / 2);
color: #252525;
text-decoration: none;
&::after {
background: $color-pf-black-400;
background: $pf-color-black-400;
}
}
&::after {
Expand All @@ -68,9 +68,9 @@ $co-m-horizontal-nav__menu-item-link-padding-lr: ($grid-gutter-width / 2);
&.co-m-horizontal-nav-item--active {
a,
button {
color: $link-color;
color: var(--pf-global--link--Color);
&::after {
background: $link-color;
background: var(--pf-global--link--Color);
}
}
}
Expand Down
6 changes: 3 additions & 3 deletions frontend/public/components/_image-stream.scss
Original file line number Diff line number Diff line change
Expand Up @@ -19,13 +19,13 @@

.co-images-stream-tag-timeline__circle-icon,
.co-images-stream-tag-timeline__square-icon {
color: $color-pf-black-300;
color: $pf-color-black-300;
left: 8px;
position: relative;
}

.co-images-stream-tag-timeline__info {
background: $color-pf-black-150;
background: $pf-color-black-150;
border-radius: 3px;
line-height: 30px;
margin: 8px 0 24px 40px;
Expand All @@ -43,7 +43,7 @@
}

.co-images-stream-tag-timeline__line {
border: 2px solid $color-pf-black-300;
border: 2px solid $pf-color-black-300;
left: 14px;
position: relative;
}
Expand Down
3 changes: 0 additions & 3 deletions frontend/public/components/_import-yaml-results.scss
Original file line number Diff line number Diff line change
@@ -1,6 +1,3 @@
// Bootstrap Core variables and mixins
@import '~bootstrap-sass/assets/stylesheets/bootstrap/variables';

.co-import-yaml-results-page {
height: 100%;
}
Expand Down
2 changes: 1 addition & 1 deletion frontend/public/components/_nav-title.scss
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
.co-actions {
align-items: flex-start; // Keep dropdown menu positioned correctly when titles wrap
align-items: baseline; // Keep dropdown menu positioned correctly
display: flex;
@media(max-width: $screen-xs){
align-items: flex-end;
Expand Down
4 changes: 0 additions & 4 deletions frontend/public/components/_quota.scss
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,6 @@
justify-content: space-evenly;
}

.co-resource-quota-dashboard-scopes {
font-size: $font-size-h5;
}

.co-resource-quota-gauge-chart {
min-width: 170px;
}
Expand Down
4 changes: 4 additions & 0 deletions frontend/public/components/_resource-dropdown.scss
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,15 @@
display: flex;
align-items: center;
flex: 1;
.small {
font-weight: normal !important;
}
.text-muted {
margin-left: 10px;
}
}
.co-search-group__accordion-toggle {
margin-top: 30px;
.pf-c-accordion__toggle-text {
display: flex;
flex-direction: column;
Expand Down
2 changes: 1 addition & 1 deletion frontend/public/components/_resource.scss
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@

@keyframes co-resource-icon__deleting {
to {
color: $color-pf-black-600;
color: $pf-color-black-600;
}
}

Expand Down
12 changes: 6 additions & 6 deletions frontend/public/components/_row-filter.scss
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
$color-row-filter-border: $color-pf-black-300;
$color-row-filter-border--active: $color-pf-blue-300;
$color-row-filter-border: $pf-color-black-300;
$color-row-filter-border--active: $pf-color-blue-300;

.co-m-row-filter__controls {
display: flex;
Expand Down Expand Up @@ -37,7 +37,7 @@ $color-row-filter-border--active: $color-pf-blue-300;
}

.row-filter__box {
background: $color-pf-black-150;
background: $pf-color-black-150;
display: inline-block;
margin: 0 -1px -1px 0;
padding: 7px 10px;
Expand All @@ -61,7 +61,7 @@ $color-row-filter-border--active: $color-pf-blue-300;
}

.row-filter__box--active {
background: $color-pf-blue-50;
background: $pf-color-blue-50;
border-color: $color-row-filter-border--active !important;
// z-index is because latter blocks overlap the joining outline of former blocks
z-index: 1;
Expand All @@ -71,14 +71,14 @@ $color-row-filter-border--active: $color-pf-blue-300;
}

.row-filter__box--empty {
color: $color-pf-black-500 !important;
color: $pf-color-black-500 !important;
.row-filter__number-bubble {
background: inherit;
}
}

.row-filter__number-bubble {
background: $color-pf-white;
background: $pf-color-white;
border-radius: 4px;
border: 1px solid $color-row-filter-border;
display: inline-block;
Expand Down
2 changes: 1 addition & 1 deletion frontend/public/components/_sysevent-icon.scss
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
.co-sysevent-icon {
background-color: #fff;
border: solid 3px $color-pf-black-200;
border: solid 3px $pf-color-black-200;
border-radius: 50%;
color: #fff;
position: relative;
Expand Down

0 comments on commit 5ed38f8

Please sign in to comment.