[gnome-shell/wip/fmuellner/notification-redux+sass: 86/145] status: Stop using .panel-status-button



commit 08099ea3b01e4b6a92e3fd6b9dc94640f155f654
Author: Florian Müllner <fmuellner gnome org>
Date:   Wed Feb 18 00:01:09 2015 +0100

    status: Stop using .panel-status-button
    
    This used to be the style-class for status icons (i.e. icon-only
    top bar items). It got unused a while ago when the class used
    by status icons stopped using it - except for the keyboard indicator,
    which set the class manually to appear as status icon despite not
    being a real icon.
    Now that the button highlight is provided by the .panel-button class
    on a parent, the obsolete class results in a double border on the
    keyboard indicator when active - just drop it from there as well
    to fix.

 data/theme/_common.scss                  |    8 --------
 data/theme/gnome-shell-high-contrast.css |    3 ---
 data/theme/gnome-shell.css               |    3 ---
 js/ui/status/keyboard.js                 |    1 -
 4 files changed, 0 insertions(+), 15 deletions(-)
---
diff --git a/data/theme/_common.scss b/data/theme/_common.scss
index c5de4e3..f56e272 100644
--- a/data/theme/_common.scss
+++ b/data/theme/_common.scss
@@ -684,14 +684,6 @@ StScrollBar {
     }
   }
   
-  .panel-status-button {
-  
-    &:active, &:overview, &:focus, &:checked {
-      border-color: lighten($selected_bg_color,5%);
-      border-bottom-width: 2px;
-    }
-  }
-  
   .panel-status-indicators-box,
   .panel-status-menu-box {
     spacing: 2px;
diff --git a/data/theme/gnome-shell-high-contrast.css b/data/theme/gnome-shell-high-contrast.css
index 588f79d..48ee64d 100644
--- a/data/theme/gnome-shell-high-contrast.css
+++ b/data/theme/gnome-shell-high-contrast.css
@@ -579,9 +579,6 @@ StScrollBar {
       color: white; }
       .unlock-screen #panel .panel-button:focus, .unlock-screen #panel .panel-button:hover, .unlock-screen 
#panel .panel-button:active, .login-screen #panel .panel-button:focus, .login-screen #panel 
.panel-button:hover, .login-screen #panel .panel-button:active, .lock-screen #panel .panel-button:focus, 
.lock-screen #panel .panel-button:hover, .lock-screen #panel .panel-button:active {
         color: white; }
-  #panel .panel-status-button:active, #panel .panel-status-button:overview, #panel 
.panel-status-button:focus, #panel .panel-status-button:checked {
-    border-color: #256ab1;
-    border-bottom-width: 2px; }
   #panel .panel-status-indicators-box,
   #panel .panel-status-menu-box {
     spacing: 2px; }
diff --git a/data/theme/gnome-shell.css b/data/theme/gnome-shell.css
index f390028..0431e97 100644
--- a/data/theme/gnome-shell.css
+++ b/data/theme/gnome-shell.css
@@ -579,9 +579,6 @@ StScrollBar {
       color: white; }
       .unlock-screen #panel .panel-button:focus, .unlock-screen #panel .panel-button:hover, .unlock-screen 
#panel .panel-button:active, .login-screen #panel .panel-button:focus, .login-screen #panel 
.panel-button:hover, .login-screen #panel .panel-button:active, .lock-screen #panel .panel-button:focus, 
.lock-screen #panel .panel-button:hover, .lock-screen #panel .panel-button:active {
         color: white; }
-  #panel .panel-status-button:active, #panel .panel-status-button:overview, #panel 
.panel-status-button:focus, #panel .panel-status-button:checked {
-    border-color: #256ab1;
-    border-bottom-width: 2px; }
   #panel .panel-status-indicators-box,
   #panel .panel-status-menu-box {
     spacing: 2px; }
diff --git a/js/ui/status/keyboard.js b/js/ui/status/keyboard.js
index 3595a39..e3938d5 100644
--- a/js/ui/status/keyboard.js
+++ b/js/ui/status/keyboard.js
@@ -589,7 +589,6 @@ const InputSourceIndicator = new Lang.Class({
         this._hbox.add_child(PopupMenu.arrowIcon(St.Side.BOTTOM));
 
         this.actor.add_child(this._hbox);
-        this.actor.add_style_class_name('panel-status-button');
 
         this._propSeparator = new PopupMenu.PopupSeparatorMenuItem();
         this.menu.addMenuItem(this._propSeparator);


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]