mokasin / apw Goto Github PK

Hi I am using awesome on Debian stable:

$ awesome -version
awesome debian/3.4.13-1 (Octopus)
 • Build: Jul 16 2012 13:57:50 for x86_64 by gcc version 4.7.1 (@keller)
 • D-Bus support: ✔

running

require("apw/widget") 

gives

widget.lua:44: attempt to index field 'layout' (a nil value)

Did the awesome API change at some point? What can I do to get this working on my installation? Thanks!

The issue can be reproduced with the following (Awesome 4.1):

• Start out from the default Awesome rc.lua.
• Put the APW widget on the wibar (I tried it right after mykeyboardlayout).
• Set show_text = true (or beautiful.apw_show_text = true).

Without show_text, the widget works fine. If show_text is enabled, then it occupies almost the whole screen and does not update.

Hi,

first off, your widget is awesome (… haha -.-)! I could think of one enhancement though, which is the ability to control the volume of multiple sinks at once — possibly and for the ease of use, all at once? I’m using a USB-DAC for HiFi listening, and my mainboards S/PDIF output for all other everyday tasks — and it would be really cool if the keybinds to APW would just control both simultaneously, so i don’t have to spawn pavucontrol for one …

Thanks!
-NebuK!

instead of

APWTimer:connect_signal("timeout", AWP.Update)

following will work

APWTimer:connect_signal("timeout", APW.Update)

lua 5.3 introduces a new integer subtype for numbers [0] and string.format() will only work with ints. Volume is used as a float and passed to string.format() --Line no.95 in pulseaudio.lua--, while this works fine prior to lua 5.3, it fails with 5.3.

[0] http://www.lua.org/manual/5.3/manual.html#8.1

Hi, I've added another little feature to the widget. But I actually committed it before I pulled from the new Beautiful integration. So how do I submit a pull request now? Should I merge it first or just send a pull request and let you merge it manually?