ChrisRM / material-theme-jetbrains

JetBrains theme of Material Theme
Other
7.83k stars 450 forks source link

Vertical padding in menus is too large #1844

Closed rparree closed 2 years ago

rparree commented 2 years ago

Summary

The vertical padding in my menus are too large.

image

I did change my font size (File | Settings | Appearance & Behavior | Appearance) to size 20

Environment

IntelliJ IDEA 2021.3 Beta (Ultimate Edition)
Build #IU-213.5744.18, built on November 10, 2021
IntelliJ IDEA EAP User
Expiration date: December 10, 2021
Runtime version: 11.0.13+7-b1751.19 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Linux 5.14.17-201.fc34.x86_64
GC: G1 Young Generation, G1 Old Generation
Memory: 2048M
Cores: 16
Registry:
    ide.balloon.shadow.size=0
    scala.erase.compiler.process.jdk.once=false

Non-Bundled Plugins:
    org.toml.lang (213.5744.3)
    org.nik.presentation-assistant (1.0.9)
    org.jetbrains.plugins.go-template (213.5744.3)
    org.jetbrains.plugins.go (213.5744.18)
    name.kropp.intellij.makefile (213.5744.3)
    in.thekalinga.snippet.intellij (6.1.0)
    ideanginx9 (0.1.9)
    org.jetbrains.plugins.vagrant (213.5744.9)
    com.jetbrains.plugins.ini4idea (213.5744.3)
    org.jetbrains.plugins.hocon (2021.1.0)
    intellij.haskell (1.0.0-beta88)
    String Manipulation (8.22.203.000.1)
    mobi.hsz.idea.gitignore (4.3.0)
    net.ashald.envfile (3.2.2)
    org.rust.lang (0.4.159.4232-213)
    com.laravel_idea.tailwind (1.3.1)
    com.mallowigi (50.2.0)
    Dummy Text Generator (1.2.0)
    com.intellij.kubernetes (213.5744.3)
    com.intellij.plugins.html.instantEditing (213.5744.3)
    izhangzhihao.rainbow.brackets (6.21)
    org.intellij.scala (2021.3.491)

Kotlin: 213-1.5.10-release-949-IJ5744.18
Current Desktop: KDE

My VM Settings

-Xms128m
-Xmx2048m
-XX:ReservedCodeCacheSize=512m
-XX:+UseG1GC
-XX:SoftRefLRUPolicyMSPerMB=50
-XX:CICompilerCount=2
-XX:+HeapDumpOnOutOfMemoryError
-XX:-OmitStackTraceInFastThrow
-ea
-Dsun.io.useCanonCaches=false
-Djdk.http.auth.tunneling.disabledSchemes=""
-Djdk.attach.allowAttachSelf=true
-Djdk.module.illegalAccess.silent=true
-Dkotlinx.coroutines.debug=off
-Dsun.tools.attach.tmp.only=true
-Dide.no.platform.update=true
-XX:+IgnoreUnrecognizedVMOptions
-Dtoolbox.notification.token=Qnh6ZxNjKzH4aIk8hEIdSmXneG-OQ0iE-8XVifasedk=
-Dtoolbox.notification.portFile=/home/rparree/.local/share/JetBrains/Toolbox/apps/WebStorm/ch-0/213.5605.12.vmoptions.port
-Didea.plugins.path=/home/rparree/.local/share/JetBrains/Toolbox/apps/WebStorm/ch-0/213.5605.12.plugins
mallowigi commented 2 years ago

There's a setting for that. Please read the doc.

rparree commented 2 years ago

Before posing, I looked everywhere in the settings and searched for known issues. Tried again, but can't find it. Was only able to find disabling the padding all together.

mallowigi commented 2 years ago

Really? https://material-theme.com/docs/configuration/compact-settings/#compact-menus

rparree commented 2 years ago

As i said, that disables the padding all together. When padding is enabled, the padding is too large.

Is it proportional to the font size? (that would explain why mine is too large).

tx.,

mallowigi commented 2 years ago

I don't think so, this is the regular size for non padded settings. I can take a look but as far as I remember they are not related. However it is indeed related to the scale, but from the screenshots they look like regular padded menus (all things considered I do think they are too large though, I'll try to add a setting to fine tune it so that people can select their own padding size)