Evening,
On 05.05.2012 23:04, Ignas Anikevicius wrote:
[...]
>>
>> 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.
Just leave the patch as-is, I guess this is better than any of the alternatives.
>>
>> 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?
Uhm, no strong opinion on that, but yeah, "like awful.rules" is fine with me.
>> 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?
Sounds like one more patch that we will be getting. :-)
So "Run app: Run: gimp" with "Run: gimp" being highlighted...
Perhaps "Execute: gimp" instead of "Run: gimp"? Or just no prefix at all?
Dunno.
Uli
--
Bitte nicht mit dem verbleibenden Auge in den Laser gucken.
- Vincent Ebert
--
To unsubscribe, send mail to [email protected].
