Hello,
Thanks for taking a look.
On 05/05/12 08:57, Uli Schlachter wrote:
> + table.insert(shownitems, { name = "Run query: " .. query, cmdline =
> query,
> icon = nil })
>
> Why "Run query: gimp"? I find the term "query" here a little confusing. What
> would you think about just "Run: gimp"?
Will do.
>
> Patch 4:
> Would merge, but merging this without patch 3 is bad. Also, I call hack for
> injecting the terminal prefix, but whatever. Leave this patch as-is.
How should we solve that? I am happy to do it the right way, but I could
not think of anything smart. We could and try have some more logics and
do it in the perform action function, but I am not sure if that is too a
good idea.
>
> Could you write a new patch which adds some luadoc which explains the
> usage/keybindings?
OK, I will try and write some docs. Should that be in HTML, like the
awful.rules documentation is written?
> Patch 5:
>
> - awful.prompt.run({ prompt = "Run app: " },
> + awful.prompt.run({ prompt = ": " },
Ooops, that slipped in unintentionally. I just had an issue, that if
there is only one choice left (i.e. the Run: <your_cmd_here>), then you
have the same text twice, which I was not a great fan of. Maybe I should
just make that choice invisible if only it is left?
All best,
Ignas
--
To unsubscribe, send mail to awesome-devel-unsubscr...@naquadah.org.
