X-Git-Url: https://git.madduck.net/etc/awesome.git/blobdiff_plain/7a4d6f5ffa594603ef83bc8b0e9945d12e082490..f7aa28919588d307083b4b5b1c010f26cbc63cde:/widgets/alsabar.lua

diff --git a/widgets/alsabar.lua b/widgets/alsabar.lua
index 0421f5c..d8f1731 100644
--- a/widgets/alsabar.lua
+++ b/widgets/alsabar.lua
@@ -4,16 +4,20 @@
      Licensed under GNU General Public License v2 
       * (c) 2013, Luke Bonham                     
       * (c) 2013, Rman                            
+                                                  
 --]]
 
+local newtimer     = require("lain.helpers").newtimer
+
 local awful        = require("awful")
 local beautiful    = require("beautiful")
 local naughty      = require("naughty")
 
-local io           = io
-local math         = { modf  = math.modf }
-local string       = { match = string.match,
-                       rep   = string.rep }
+local io           = { popen  = io.popen }
+local math         = { modf   = math.modf }
+local string       = { format = string.format,
+                       match  = string.match,
+                       rep    = string.rep }
 local tonumber     = tonumber
 
 local setmetatable = setmetatable
@@ -23,68 +27,84 @@ local setmetatable = setmetatable
 local alsabar =
 {
   channel = "Master",
-  step = "5%",
+  step    = "5%",
 
   colors =
   {
      background = beautiful.bg_normal,
-     mute   = "#EB8F8F",
-     unmute = "#A4CE8A"
+     mute       = "#EB8F8F",
+     unmute     = "#A4CE8A"
   },
 
-  mixer = terminal .. " -e alsamixer",
+  terminal = terminal or "xterm",
+  mixer    = terminal .. " -e alsamixer",
 
   notifications =
   {
-     font = beautiful.font:sub(beautiful.font:find(""), beautiful.font:find(" ")),
+     font      = beautiful.font:sub(beautiful.font:find(""), beautiful.font:find(" ")),
      font_size = "11",
-     bar_size = 18 -- Awesome default
+     color     = beautiful.fg_focus,
+     bar_size  = 18
   },
 
   _current_level = 0,
-  _muted = false
+  _muted         = false
 }
 
-function alsabar:notify()
+function alsabar.notify()
+  alsabar.update()
+
 	local preset =
 	{
-      title = "", text = "",
-      timeout = 3,
-      font = alsabar.notifications.font .. " " .. alsabar.notifications.font_size,
-      fg = beautiful.fg_focus
+      title   = "",
+      text    = "",
+      timeout = 4,
+      font    = alsabar.notifications.font .. " " ..
+                alsabar.notifications.font_size,
+      fg      = alsabar.notifications.color
 	}
 
-	if alsabar._muted then
+	if alsabar._muted
+  then
 		preset.title = alsabar.channel .. " - Muted"
 	else
 		preset.title = alsabar.channel .. " - " .. alsabar._current_level * 100 .. "%"
 	end
 
-  local int = math.modf(alsabar._current_level * alsabar.notifications.bar_size)
-  preset.text = "[" .. string.rep("|", int)
-                .. string.rep(" ", alsabar.notifications.bar_size - int) .. "]"
+  int = math.modf(alsabar._current_level * alsabar.notifications.bar_size)
+  preset.text = "["
+                .. string.rep("|", int)
+                .. string.rep(" ", alsabar.notifications.bar_size - int)
+                .. "]"
 
   if alsabar._notify ~= nil then
-		alsabar._notify = naughty.notify ({ replaces_id = alsabar._notify.id,
-			                               preset = preset })
+		alsabar._notify = naughty.notify ({
+        replaces_id = alsabar._notify.id,
+			  preset      = preset
+    })
 	else
-		alsabar._notify = naughty.notify ({ preset = preset })
+		alsabar._notify = naughty.notify ({
+        preset = preset
+    })
 	end
 end
 
-function worker(args)
+local function worker(args)
     local args = args or {}
+    local timeout = args.timeout or 4
     local width = args.width or 63
     local height = args.heigth or 1
     local ticks = args.ticks or true
     local ticks_size = args.ticks_size or 7
     local vertical = args.vertical or false
+
     alsabar.channel = args.channel or alsabar.channel
     alsabar.step = args.step or alsabar.step
     alsabar.colors = args.colors or alsabar.colors
     alsabar.notifications = args.notifications or alsabar.notifications
 
     alsabar.bar = awful.widget.progressbar()
+
     alsabar.bar:set_background_color(alsabar.colors.background)
     alsabar.bar:set_color(alsabar.colors.unmute)
     alsabar.tooltip = awful.tooltip({ objects = { alsabar.bar } })
@@ -93,11 +113,9 @@ function worker(args)
     alsabar.bar:set_ticks(ticks)
     alsabar.bar:set_ticks_size(ticks_size)
 
-    if vertical then
-        alsabar.bar:set_vertical(true)
-    end
+    if vertical then alsabar.bar:set_vertical(true) end
 
-    local myvolumebarupdate = function()
+    function alsabar.update()
         -- Get mixer control contents
         local f = io.popen("amixer get " .. alsabar.channel)
         local mixer = f:read("*all")
@@ -105,60 +123,48 @@ function worker(args)
 
         -- Capture mixer control state:          [5%] ... ... [on]
         local volu, mute = string.match(mixer, "([%d]+)%%.*%[([%l]*)")
-        -- Handle mixers without data
+
         if volu == nil then
-           volu = 0
-           mute = "off"
+            volu = 0
+            mute = "off"
         end
 
         alsabar._current_level = tonumber(volu) / 100
         alsabar.bar:set_value(alsabar._current_level)
 
-        if mute == "" and volu == "0" or mute == "off"
+        if not mute and tonumber(volu) == 0 or mute == "off"
         then
             alsabar._muted = true
             alsabar.tooltip:set_text (" [Muted] ")
             alsabar.bar:set_color(alsabar.colors.mute)
         else
             alsabar._muted = false
-            alsabar.tooltip:set_text(" " .. alsabar.channel .. ": " .. volu .. "% ")
+            alsabar.tooltip:set_text(string.format(" %s:%s ", alsabar.channel, volu))
             alsabar.bar:set_color(alsabar.colors.unmute)
         end
     end
 
-    local myvolumebartimer = timer({ timeout = 5 })
-    myvolumebartimer:connect_signal("timeout", myvolumebarupdate)
-    myvolumebartimer:start()
-    myvolumebartimer:emit_signal("timeout")
+    newtimer("alsabar", timeout, alsabar.update)
 
     alsabar.bar:buttons (awful.util.table.join (
           awful.button ({}, 1, function()
             awful.util.spawn(alsabar.mixer)
           end),
           awful.button ({}, 3, function()
-            awful.util.spawn("amixer sset " .. alsabar.channel .. " toggle")
-            myvolumebarupdate()
+            awful.util.spawn(string.format("amixer set %s toggle", alsabar.channel))
+            alsabar.update()
           end),
           awful.button ({}, 4, function()
-            awful.util.spawn("amixer sset " .. alsabar.channel .. " "
-                              .. alsabar.step .. "+")
-            myvolumebarupdate()
+            awful.util.spawn(string.format("amixer set %s %s+", alsabar.channel, alsabar.step))
+            alsabar.update()
           end),
           awful.button ({}, 5, function()
-            awful.util.spawn("amixer sset " .. alsabar.channel .. " "
-                              .. alsabar.step .. "-")
-            myvolumebarupdate()
+            awful.util.spawn(string.format("amixer set %s %s-", alsabar.channel, alsabar.step))
+            alsabar.update()
           end)
     ))
 
-    return { widget = alsabar.bar,
-             channel = alsabar.channel, 
-             step = alsabar.step,
-             notify = function() 
-                         myvolumebarupdate()
-                         alsabar.notify()
-                       end
-           }
+    return alsabar
 end
 
 return setmetatable(alsabar, { __call = function(_, ...) return worker(...) end })