Aris-t2 / CustomCSSforFx

Custom CSS tweaks for Firefox
GNU General Public License v3.0
1.9k stars 182 forks source link

popup_items_hover_appearance_aero.css needs a little correct #674

Closed ArkadiuszMichalski closed 1 month ago

ArkadiuszMichalski commented 3 months ago

When we use @import "./css/generalui/popup_items_hover_appearance_aero.css"; then text in hover but disabled menuitem unnecessarily gets black color. This makes it look like it's not disabled and clickable.

image

I suggest that for hovered blocked items:

image

I use: CustomCSSforFx 4.5.5 Firefox 129.0.2 (64-bit)

Aris-t2 commented 3 months ago

Replace the code inside popup_items_hover_appearance_aero.css with this:

/* Firefox userChrome.css tweaks ********************************************************/
/* Github: https://github.com/aris-t2/customcssforfx ************************************/
/****************************************************************************************/

/* override color set by context_bfrsb_labels_with_icons.css */
@media (-moz-platform: windows) {
  #context-navigation :is(#context-back,#context-forward,#context-reload,#context-stop,#context-bookmarkpage):hover {
    background-color: unset !important;
  }
}

:is(popup,menupopup,panel,panelmultiview,.panel-subview-body) :is(menuitem,menu,.subviewbutton,toolbarbutton):not(#appMenu-fxa-status2):hover {
  background: transparent !important;
  background-color: transparent !important;
  background-image:
    linear-gradient(to right, rgba(255,255,255,0.4) 0px, transparent 0px),
    linear-gradient(to left, rgba(255,255,255,0.4) 0px, transparent 0px),
    linear-gradient(to top, rgba(255,255,255,0.3) 0px, transparent 0px),
    linear-gradient(to bottom, rgba(163,196,247,0.2), rgba(122,180,246,0.2)) !important;
  border-radius: 6px !important;
  outline: 1px solid rgba(124,163,206,0.7) !important;
  outline-radius: 3px !important;
  outline-offset: -1px !important;
}

:is(popup,menupopup,panel,panelmultiview,.panel-subview-body) :is(menuitem,menu,.subviewbutton,toolbarbutton):not(#appMenu-fxa-status2):not([disabled]):hover {
  color: inherit !important;
}

:is(popup,menupopup,panel,panelmultiview,.panel-subview-body) :is(menuitem,menu,.subviewbutton,toolbarbutton):not(#appMenu-fxa-status2)[disabled]:hover {
  filter: grayscale(75%) !important;
}
ArkadiuszMichalski commented 3 months ago

@Aris-t2 It's correct now. Will this fix be added in the next version of CustomCSSforFx?

image

Aris-t2 commented 3 months ago

Yes, it will be in next release.