Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

how to decrease the spacing/padding in the contextual menu? #30

Closed
mooms06 opened this issue Jun 2, 2021 · 6 comments
Closed

how to decrease the spacing/padding in the contextual menu? #30

mooms06 opened this issue Jun 2, 2021 · 6 comments
Labels
Class::Padding Padding and margin related layout Component::Context-Menu Only for context menu Issue::Question Further information is requested

Comments

@mooms06
Copy link

mooms06 commented Jun 2, 2021

Especially the spelling suggestions (see image)

image

@black7375 black7375 added Class::Padding Padding and margin related layout Component::Context-Menu Only for context menu Issue::Question Further information is requested labels Jun 2, 2021
@black7375
Copy link
Owner

Adjust this.

/** Menu - Reduce Padding ***************************************************/
:root:not([uidensity=touch]) menupopup > menuitem, menupopup > menu {
padding-block: 0.35em !important; /* Original: 0.5em */
}
:root:not([uidensity=touch]) .menu-text, .menu-iconic-text {
padding-inline-end: 0 !important; /* Original: 2px */
}
:root:not([uidensity=touch]) .menupopup-arrowscrollbox {
padding-block: 1px !important; /* Original: 4px*/
}
:root:not([uidensity=touch]) #context-navigation:not([hidden]) {
padding: 0 0 1px !important; /* Original: 0 0 4px*/
}
:root:not([uidensity=touch]) .menu-right {
margin-right: 6px !important; /* Original: 12px */
}
:root[uidensity=compact] menupopup > menuitem, menupopup > menu {
padding-block: 0.25em !important; /* Original: 0.5em */
}

@mooms06
Copy link
Author

mooms06 commented Jun 2, 2021

Which one should I modify for normal density ?

image

@black7375
Copy link
Owner

Targets with :root:not([uidensity=touch]) in front of them.

@mooms06
Copy link
Author

mooms06 commented Jun 2, 2021

That's perfect, thank you very much !

I've settled for this, seems good to me:

  /** Menu - Reduce Padding ***************************************************/
  :root:not([uidensity=touch]) menupopup > menuitem, menupopup > menu {
    padding-block: 0.05em !important; /* Original: 0.5em */
  }
  :root:not([uidensity=touch]) .menu-text, .menu-iconic-text {
    padding-inline-end: 0 !important; /* Original: 2px */
  }
  :root:not([uidensity=touch]) .menupopup-arrowscrollbox {
    padding-block: 1px !important;    /* Original: 4px*/
  }
  :root:not([uidensity=touch]) #context-navigation:not([hidden]) {
    padding: 0 0 1px !important;      /* Original: 0 0 4px*/
  }
  :root:not([uidensity=touch]) .menu-right {
    margin-right: 1px !important;     /* Original: 12px */
  }

Result:
image

@mooms06 mooms06 closed this as completed Jun 2, 2021
@black7375
Copy link
Owner

I think it would be good to apply it to compact mode later.

@mooms06
Copy link
Author

mooms06 commented Jun 2, 2021

I don't plan to use compact mode, but to me it should be set to 0.10em or 0.15em on default mode, 0.35 is still too much IMO.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Class::Padding Padding and margin related layout Component::Context-Menu Only for context menu Issue::Question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants