Skip to content

Commit

Permalink
Merge pull request #13940 from saraycp/adjust_colors
Browse files Browse the repository at this point in the history
Adjust colors after Bootstrap 5 migration
  • Loading branch information
saraycp committed Mar 3, 2023
2 parents 4b62fbf + 2916596 commit f26fc84
Show file tree
Hide file tree
Showing 8 changed files with 13 additions and 9 deletions.
1 change: 1 addition & 0 deletions src/api/app/assets/stylesheets/webui/application.scss
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,7 @@
@import 'request_show_redesign/build_results';
@import 'accordion-reviews-component';
@import 'tokens';
@import 'colors';

html {
overflow-y: scroll !important;
Expand Down
3 changes: 3 additions & 0 deletions src/api/app/assets/stylesheets/webui/colors.scss
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
.bg-gray-800 {
background-color: $gray-800;
}
6 changes: 3 additions & 3 deletions src/api/app/assets/stylesheets/webui/navbar.scss
Original file line number Diff line number Diff line change
@@ -1,9 +1,10 @@
.navbar-dark {
// Dropdown: Your profile & Logout
.dropdown-menu {
@extend .bg-dark;
background-color: $gray-800;

.dropdown-item:hover {
@extend .bg-dark;
background-color: $gray-800;
color: $gray-300;
}
}
Expand All @@ -19,7 +20,6 @@
padding-left: 1rem;
overflow-y: auto;
visibility: hidden;
background-color: #343a40;
color: $gray-400;
transition: visibility .3s ease-in-out, -webkit-transform .3s ease-in-out;
transition: transform .3s ease-in-out, visibility .3s ease-in-out;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
%nav.navbar.fixed-bottom.navbar-dark.bg-dark.border-top.border-gray-500.p-0
%nav.navbar.fixed-bottom.navbar-dark.bg-gray-800.border-top.border-gray-500.p-0
%ul.nav.justify-content-center.w-100.nav-justified
- if User.session
%li.nav-item
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
.navbar-collapse.navbar-dark{ class: "#{collapse_type}-collapse" }
.navbar-collapse.navbar-dark.bg-gray-800{ class: "#{collapse_type}-collapse" }
.navbar-nav
.nav.justify-content-end.py-2
%button.navbar-toggler{ type: 'button', 'data-toggle': collapse_type, aria: { expanded: 'false', label: "Toggle #{collapse_type}" } }
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
.bg-dark.border-top.border-gray-500#toggle-sidebar-button
.navbar-dark.bg-gray-800.border-top.border-gray-500#toggle-sidebar-button
%ul.nav.flex-column.ms-auto.pt-0.text-nowrap.menu-options
%li.nav-item
%a.nav-link{ type: 'button' }
Expand Down
2 changes: 1 addition & 1 deletion src/api/app/views/layouts/webui/_top_navigation.html.haml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
%nav.navbar.navbar-dark.bg-dark.fixed-top
%nav.navbar.navbar-dark.bg-gray-800.fixed-top
.container-fluid.d-flex.flex-nowrap.justify-content-between.w-100
= link_to(root_path, class: 'navbar-brand', alt: 'Logo') do
= image_tag('obs-logo_small.svg')
Expand Down
4 changes: 2 additions & 2 deletions src/api/app/views/layouts/webui/webui.html.haml
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@
#grid
#top-navigation-area
= render partial: 'layouts/webui/top_navigation'
.bg-dark#left-navigation-area{ class: "#{'collapsed' if sidebar_collapsed?}" }
.navbar-dark.bg-gray-800#left-navigation-area{ class: "#{'collapsed' if sidebar_collapsed?}" }
= render partial: 'layouts/webui/left_navigation', locals: { spider_bot: @spider_bot }
= render partial: 'layouts/webui/left_navigation_collapse_button'
.d-flex.flex-column#content-area
Expand Down Expand Up @@ -77,7 +77,7 @@

-# Collapsible menu shared between top and bottom navigation
- if User.session
.navbar-collapse.watchlist-collapse.navbar-dark
.navbar-dark.bg-gray-800.navbar-collapse.watchlist-collapse
= render WatchlistComponent.new(user: User.session!,
bs_request: @bs_request,
package: @package,
Expand Down

0 comments on commit f26fc84

Please sign in to comment.