andersju / zzzfoo

Full-text desktop search with Rofi+Recoll
GNU General Public License v2.0
121 stars 8 forks source link

fixed option parsing, added message parameters #9

Closed ickerwx closed 4 years ago

ickerwx commented 4 years ago

The way you simply split() the rofi_options doesn't work for more complex parameters that contain spaces, but shlex.split() fixes that. For example, you can't use --rofi-options="-mesg 'This is some text'", but you can with my fix. Additionally, I wanted to display text in the rofi message UI element. Using --search-mesg "foo bar" will result in -mesg "foo bar" being passed to the search rofi call, same with the --result-mesg parameter for the result rofi call.

andersju commented 4 years ago

Looks good - thanks for the fix and the feature!