Skip to content

Conversation

@anaximeno
Copy link
Contributor

@anaximeno anaximeno commented Jan 10, 2026

closes #11540
solves https://github.com/orgs/linuxmint/discussions/82

Example of how should work

2026-01-21.17-26-02.mp4

@Initu-Castilhos
Copy link

This feature is great, it's similar to dash to dock and dash to panel. Hopefully it will be implemented in the stable version.

@JosephMcc
Copy link
Contributor

Didn't actually try it yet, but it looks pretty slick. Wonder if you could use vfade when the slider button shows up as a bit more of a visual cue.

…orkspaces

To find the new app position in the container instead of using actor.x we are summing the size of each preceding appGroup until the current app to focus because actor.x would report 0 some times.
This was triggering GLib Critical error in the logs.
…uttons visibility

This avoids issues with improper representation of the current allocation after the buttons are hidden.
@anaximeno anaximeno force-pushed the gwl/add-support-for-slide-on-overflow branch from c3016dd to 21564ff Compare January 11, 2026 03:45
@anaximeno anaximeno force-pushed the gwl/add-support-for-slide-on-overflow branch from 21564ff to 5da320c Compare January 11, 2026 03:54
@anaximeno
Copy link
Contributor Author

Wonder if you could use vfade when the slider button shows up as a bit more of a visual cue.

I'll give it a try and see if I can get it to work with it.

@anaximeno anaximeno marked this pull request as ready for review January 11, 2026 20:34
@anaximeno anaximeno changed the title [WIP] gwl: Add overflow support through sliding gwl: Add overflow support through sliding Jan 21, 2026
@anaximeno anaximeno changed the title gwl: Add overflow support through sliding [NEXT] gwl: Add overflow support through sliding Jan 21, 2026
@anaximeno
Copy link
Contributor Author

@JosephMcc I've updated the implementation to use the native St.ScrollView instead of a direct container translation to handle the scroll, which goes with how scroll is being implemented accross Cinnamon. I wanted to use it initially as well but was having some issues with the scroll not taking effect in the gwl and instead the same stretch behavior was persisting, however I found the reason for it and it seems it was that since there was no overflow method before, a change was introduced to set the min_size of the AppGroup.actor around 1 (which meant instead of the container knowing what's the proper size to represent the full content it was considering that it should just stretch almost indefinetely, so I reverted that behavior in https://github.com/linuxmint/cinnamon/pull/13352/changes#diff-ff87e15fbe5d4a462aeef966bf403d5543648b31289a55f4b835cdcfcb32f362.

Thanks to that and getting St.ScrollView to work, I can also use the vfade/hfade in the edges instead of the slider buttons (as you suggested), I've updated the video demonstration above to represent that.

In general I think this PR is ready for testing and consideration, and I'll be continuing to test and it sending adjustments if necessary until the next merge window.

Regards.

@anaximeno anaximeno changed the title [NEXT] gwl: Add overflow support through sliding [NEXT] gwl: Add overflow support through scroll Jan 21, 2026
@clefebvre clefebvre changed the title [NEXT] gwl: Add overflow support through scroll gwl: Add overflow support through scroll Jan 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Better approach when running out of space in the panel

3 participants