search

bakkeby / dmenu-flexipatch

A dmenu build with preprocessor directives to decide which patches to include during build time
MIT License
183 stars 78 forks source link

Add highlight for mouse hover #30

Closed happyzleaf closed 1 month ago

happyzleaf commented 1 month ago

To my surprise the mouse support patch does not highlight an entry when hovering by default.

I applied this patch from the same author that brings back motion support.

Also wanted to thank you for all these projects, you made ricing much more enjoyable.

happyzleaf commented 1 month ago

When I was proof-reading this PR I figured that there's a new patch that I somehow missed.

I'm not sure why but it was working before and is working now. For the sake of it I have uploaded the new patch instead.

bakkeby commented 1 month ago

Thanks, I must have missed that variant. I am going to make a few changes to see if I can make this work with grid as well.

bakkeby commented 1 month ago

I pushed https://github.com/bakkeby/dmenu-flexipatch/commit/4a8b71c19126fd2bd7e8945585efbc2d83a00235 with some patch cross-compatibility changes, primarily with grid and the symbols patch.

The mouse support patch didn't actually support grid either, so I added that. It helped that I had already integrated this in my personal build of dmenu.

I liked the simplicity of the original implementation (horizontal vs vertical).