search

nyyManni / dmenu-wayland

dmenu for wayland-compositors
MIT License
197 stars 22 forks source link

Fixed suggestions not showing #5

Closed janlucaklees closed 5 years ago

janlucaklees commented 5 years ago

The suggestions for which executable to run were not showing, requiring the user to type in the exact name of the executable. This depreciated the user experience when using the dmenu. In this commit I fixed just that. The suggestions are showing again and you can select them with the arrowkeys, just as you would expect them to.

The problem was a naming issue in the dmenu-wl_run script. There the executable for finding all executables was named dmenu_path, while in the meson.build file this executable was named dmenu-wl_path. All I needed to do was adapting the name in the script.

nyyManni commented 5 years ago

Thanks! The dmenu-wl_run was accidentally using the path from the original dmenu, which of course fails if it is not installed.

janlucaklees commented 5 years ago

You're welcome. Thank you for the quick response and merge!