mzur / gnome-shell-wsmatrix

GNOME shell extension to arrange workspaces in a two-dimensional grid with workspace thumbnails
GNU General Public License v3.0
464 stars 59 forks source link

Labels are very small #140

Closed toobaz closed 3 years ago

toobaz commented 3 years ago

First, thanks for this great extension.

Unfortunately, in the popup labels (when enabled) are very small:

Schermata da 2021-01-14 08-13-59

I didn't find a way to rescale them.

(gnome-shell 3.38.2-1 in Debian testing)

(By the way: it would be great to be able to show both the preview AND the label - related to #41 )

mzur commented 3 years ago

The label font size should be more or less the same than the text in the panel at the top. Is this too small for you, too? Maybe this is a GNOME 3.38 issue (which is still not fully supported by this extension #125). You could change the font size with your theme and the .ws-switcher-label selector. Or you would have to implement a new settings option for this extension which applies a new style: 'font-size: XXpx;' here.

toobaz commented 3 years ago

The label font size should be more or less the same than the text in the panel at the top. Is this too small for you, too?

I confirm they look the same size... which seems small to me. Maybe because I'm used to Workspace-Grid, where they would take most of the available empty space, making them easier to quickly look up: https://www.worldofgnome.org/workspace-grid-shell-extension-for-gnome-3-8

My opinion would be that since the label is shown in an empty box, nobody will complain if you just make the label fill more of that box, so I wouldn't see a new setting option as needed.

I did not understand - but I will investigate - your suggestion of using the .ws-switcher-label selector.

mzur commented 3 years ago

I agree that the label display could be improved. At least the labels could be centered in the workspace indicators. But when you say "make the label fill more of that box", how to determine the size? Should the label fill the whole width of the indicator? This would produce different sizes for different labels which looks bad (imo). So which size to pick? The only solution I can think of is to let the theme decide or to provide a settings option. Maybe you can take a look at how Workspace Grid did this. Feel free to send a pull request with a solution.

toobaz commented 3 years ago

Maybe you can take a look at how Workspace Grid did this.

It seems like it used the ws-switcher-label class:

https://github.com/zakkak/workspace-grid/blob/3.30/src/gridWorkspaceSwitcherPopup.js#L268

... that defines the size as 2em:

https://github.com/zakkak/workspace-grid/blob/3.30/src/stylesheet.css#L74

I see your worry that some long names might be truncated... but it never happened to me, and if it happened, that would be a minor concern... and in principle, it could happen even now.

I can try a PR if you like the solution... although it would be my first javascript PR ever, not really sure of the result.

(By the way: providing a settings option is probably the perfect fix... but there I'm really clueless implementation-wise)

mzur commented 3 years ago

Feel free to send a PR if you like. Ideally I'd like the following solution:

ebeem commented 3 years ago

implemented and tested in #152, please let's discuss any issue with the current implementation (if any) in the PR.