change codename

Signed-off-by: Uli Schlachter <psychon@znc.in>

Menubar: Expose the wm_name attribute

This commit makes it possible to change the wm_name attribute, which
means that programs can be shown/hidden according to the wishes of the
user.

Possible usecase: A user is using Gnome and Awesome together, but sinc
wm_name by default is awesome, all the apps shipped with GNOME DE are
hidden by default. Changing wm_name attribute in this case would solve
the issue.

Signed-off-by: Ignas Anikevicius (gns_ank) <anikevicius@gmail.com>
Signed-off-by: Uli Schlachter <psychon@znc.in>

screen: Ignore RANDR displays in clone mode (FS#1046)

This just makes the code for ignoring screens on Xinerama apply to RANDR screens
as well. The code now also applies to protocol screens, but there is only ever a
single one of those, so nothing should change here.

Signed-off-by: Uli Schlachter <psychon@znc.in>

dbus: Fix event handling

The port to glib failed to actually watch the file descriptor for events and
thus awesome silently ignored all dbus messages. My bad.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Use glib instead of libev

This commit ports awesome from libev to the glib main loop. This means that
awesome has fewer dependencies, because we were already depending on glib before
and now no longer need glib.

However, the main reason for this change is that, thanks to lgi, we have glib
bindings for lua. This means that lua code can add all kinds of event sources to
the main loop (timeouts, fd watchers, SIGCHLD watchers, ....). Yay

Signed-off-by: Uli Schlachter <psychon@znc.in>

luadoc: Fix a typo for awesome.release

Signed-off-by: Uli Schlachter <psychon@znc.in>

awful.client.toggletag: Use tag.getscreen()

Thanks to muni for reporting this bug.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Fix warnings from LDoc

Signed-off-by: Uli Schlachter <psychon@znc.in>

Use LDoc instead of Luadoc (FS#1006)

Since we are no longer using module(), luadoc no longer understands our modules.
With this commit we use LDoc instead which seems more modern.

Signed-off-by: Uli Schlachter <psychon@znc.in>

luadoc/: Remove luadoc hacks

LDoc doesn't need any of this and actually breaks due to it.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Lots of random documentation fixes

This is mostly about mis-named parameters, but also other small things.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Also make gears.object more object-y

Signed-off-by: Uli Schlachter <psychon@znc.in>

wibox: Make the functions a little more object-y

This replaces lots of function foo.bar(this, ...) with function foo:bar(...).
There should be no other changes in this commit.

The point is to make it easier for api documentation tools to figure out that
these are methods on objects.

Signed-off-by: Uli Schlachter <psychon@znc.in>

naughty: Correctly handle rowstride on icons

The notification spec allow sending icons as data with a dbus message. The
rowstride for this can be set which means there can be bytes after each row of
icon data which must be ignored.

Before this commit, naughty wasn't properly ignoring these garbage bytes which
resulted in weird notifications.

Thanks to dadrc for reporting that gmusicbrowser's notifications didn't work.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Fixed bounding shape size to take into account border size.

Signed-off-by: Uli Schlachter <psychon@znc.in>

fix awful.util.table.join if first arg is null (#1052)

Signed-off-by: Uli Schlachter <psychon@znc.in>

Fixed window gaps in the "fair" tiling mode.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Fix a bug when switching fullscreen mode

The code assumed that a window gets resized when it enters/leaves the fullscreen
mode. This was needed because the code for resizing a window hides and shows the
titlebars.

However, it is possible for a window to enter/leave fullscreen mode without
getting resized, because it already has the correct geometry. In this case,
titlebars were not handled correctly.

This commit fixes that by forcing a resize to the current geometry when the
fullscreen flag is toggled.

Signed-off-by: Uli Schlachter <psychon@znc.in>

client: Re-add shape support

Same reasoning as for the recent commit which adds these to drawins.

Signed-off-by: Uli Schlachter <psychon@znc.in>

drawin: Correctly set window shape

Thanks to "rnoway" (aka Arnaud?) for noticing my typo.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Drawin: Re-add shape support

Commit 03e0ee53d2b1f2e7bac removed window shapes, because at the time I was too
lazy to port them from the old image system to oocairo.

This commit re-adds them, but for now only as a way to set the shape.

Signed-off-by: Uli Schlachter <psychon@znc.in>

awful.tooltip: Update geometry in set_text (FS#956)

Signed-off-by: Uli Schlachter <psychon@znc.in>

Fix fullscreen windows

Fullscreen windows only worked on a screen which had x and y coordinates 0.
Otherwise, the window inside the frame window was completely misplaced.

Signed-off-by: Uli Schlachter <psychon@znc.in>

wibox.drawables: Fix a memleak

Drawables could never be garbage-collected, because of the "wallpaper_changed"
signal. The callback function for this signal kept a strong reference to the
drawable.

Fix this by putting all drawable's draw() methods into a weakly keyed tabled so
that the strong reference to them disappears.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Hide titlebars on fullscreen clients

Fullscreen clients shouldn't be decorated, thus hide titlebars on those.

This commits assumes that clients get resized when they enter or leave the
fullscreen state. I don't think that this assumption is necessarily true, but
let's just wait for bug reports. :-)

Signed-off-by: Uli Schlachter <psychon@znc.in>

Enlarge clients when adding titlebars

Previously, the area of the window that the client could draw to got smaller
when titlebars are added. This causes all sorts of odd-looking effects, so
instead this should enlarge the window suitably.

Signed-off-by: Uli Schlachter <psychon@znc.in>

awful.icccm: Handle titlebars

Size hints should be applied to the "real" client geometry. That means the area
taken by titlebars should be ignored.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Remove unused signal property::widgets

Signed-off-by: Uli Schlachter <psychon@znc.in>

awful.client: Convert another t.screen to awful.tag.getscreen()

Signed-off-by: Uli Schlachter <psychon@znc.in>

lib/wibox: Make signals from drawables available

Signed-off-by: Uli Schlachter <psychon@znc.in>

Send correct geometry in ConfigureNotifies

The code was sending out ConfigureNotify events which contained the size of the
frame window. Thus, the client assumed that it is was larger than it actually
was.

Fix this by subtracting the size of the titlebars from the geometry for the
event.

Signed-off-by: Uli Schlachter <psychon@znc.in>

rc.lua: Resize clients via the titlebar

Signed-off-by: Uli Schlachter <psychon@znc.in>

drawin: Update the drawable's geometry on move

We only need to update a drawin's drawing related resources when its size
changes. Thus, when a drawin is just moved, drawin_update_drawing() is not
called. However, this function was used for telling the drawin's drawable about
its geometry. This means that the drawable had an outdated idea about what its
geometry was.

Fix this by making sure to always call drawable_set_geometry().

Thanks to Javafant for reporting this bug on IRC.

Signed-off-by: Uli Schlachter <psychon@znc.in>

capi.screen[scr]:tags() doesn't exist anymore

awful.tag.gettags(scr) does now the job

Signed-off-by: Uli Schlachter <psychon@znc.in>

tag.screen doesn't exist anymore

awful.tag.getscreen(tag) returns now the tag's screen index

Signed-off-by: Uli Schlachter <psychon@znc.in>

Fix a bug when using in a rule the property 'tag'.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Add "-colorspace rgb" to our ImageMagick calls

The old PNGs that were generated caused a warning from libpng when they were
read:

    incorrect gamma=(0/100000)
    Ignoring incorrect gAMA value when sRGB is also present

This changes the look of the resulting icons by making them darker. However,
right now I just want to get rid of that stupid warning. Feel free to suggest
better options.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Don't let Xlib own the event queue

Since commit 531f8b415c65e97, we are using Xlib for our X11 connection. However,
we still use XCB for event processing. This means we actually have to tell Xlib
to leave all events alone.

This might fix FS#1047. Perhaps.

Signed-off-by: Uli Schlachter <psychon@znc.in>

awful.titlebars: Implement

This commits adds the necessary lua code so that we finally can have titlebars.
As the baseline for the needed functionality, the titlebar code in awesome 3.4
and a quick poll on the mailing list were used.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Client: Properly get rid of titlebars on unmanage

Signed-off-by: Uli Schlachter <psychon@znc.in>

Tags: Remove screen property

A tag's screen is now implemented purely in lua and it is no longer C's
business.

Signed-off-by: Uli Schlachter <psychon@znc.in>

awful.tag.attached_connect_signal: Simplify

Signals are emitted on individual objects and on the class for a while now. This
can be used to simplify this function a lot.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Added a functions for getting all tags on a screen

Signed-off-by: Uli Schlachter <psychon@znc.in>

drawable: Add property::surface

This new property is used for fixing some missing redraws that the old code had.
Those could be seen via awful.menu. Open and close a submenu repeatedly and the
submenu will appear black.

Signed-off-by: Uli Schlachter <psychon@znc.in>

luaA_window_get_opacity: Always return a number

What's the point of distinguishing between "opacity 1" and "no opacity set"?
This commit makes awesome handle both cases identically.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Use gdk-pixbuf instead of Imlib2 (FS#1028)

There are two reasons for this switch:

- gdk-pixbuf is smaller
- gdk-pixbuf supports SVGs

Signed-off-by: Uli Schlachter <psychon@znc.in>

Add titlebars on the C side

This commit makes it possible to add titlebars to a client. These titlebars are
drawables.

The drawin's input handling is moved to the drawable. This allows it to use the
same code for drawin and titlebar input handling, although there are lots of
differences between the two on the C side.

On the lua side, a new wibox.drawable module is created which handles all the
drawable-specific magic and which can also be used for titlebars.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Introduce drawables

A drawable is something that you can draw to, just like a drawin. However, a
drawable isn't necessarily its own windows. This will later on be used to
implement titlebars where the titlebars are drawables.

Signed-off-by: Uli Schlachter <psychon@znc.in>

draw.h: Include lua.h

This header uses lua_State*, but doesn't actually include lua.h

Signed-off-by: Uli Schlachter <psychon@znc.in>

gears.object: Give better error messages

It helps a lot to know which signal does not exist. That should make it a lot
easier to look for the "guily" code without having to resort to the backtrace.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Add x11-xcb to the pkg-config checks

This is needed for XGetXCBConnection().

Signed-off-by: Uli Schlachter <psychon@znc.in>

Added initial support for Xlib cursor themes

I hope this time i got all right with git format-patch.

Signed-off-by: Tumin Alexander <iamtakingiteasy@eientei.org>
Signed-off-by: Uli Schlachter <psychon@znc.in>

Fixes module namespace issues in screen.lua and client.lua

The wrong module names were introduced in commits:
0e2960ebf372507017d6dba4e573c28dbd028478 and
d799ac76aa9d182abc4d80810e4c552e6e4d7e17.

Once fixed, client.lua and screen.lua mutually require each other, so we must
use a trick, and load the modules inside the functions that need them.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Makes awful.screen.focus keep the cursor position on screen

When changing focus to an other screen, awful.screen.focus keeps relative
position of the cursor, instead of moving to the top left corner. Does not
trigger mouse:enter and mouse:leave signals.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Functions to change client focus by direction across screens

Added functions awful.client.focus.global_bydirection and
awful.client.swap.global_bydirection, that change focus and swap clients,
crossing screen boundaries.

Also modified awful.client.movetoscreen. Now calls awful.screen.focus.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Added function to change screen focus by direction

The function awful.screen.focus_bydirection changes the screen focus
according to physical position. The code is based on
awful.client.focus.bydirection.

Signed-off-by: Uli Schlachter <psychon@znc.in>

make naughty catch appname from DBUS messages

Signed-off-by: Uli Schlachter <psychon@znc.in>

Makes awful.client.restore return the restored client

If some client is restored, returns it, otherwise returns nil.

Signed-off-by: Uli Schlachter <psychon@znc.in>

menubar: Make prompt configurable

Add a parameter to configure menubar's prompt.

Signed-off-by: Alexander Yakushev <yakushev.alex@gmail.com>
Signed-off-by: Uli Schlachter <psychon@znc.in>

menubar: Make some parameters configurable again

After the rewrite from modules they ended up being local, but should
be available to user externally.
Also remove unnecessary local context bindings.

Signed-off-by: Alexander Yakushev <yakushev.alex@gmail.com>
Signed-off-by: Uli Schlachter <psychon@znc.in>

Have only FreeBSD require dynamic linking (FS#743)

Signed-off-by: Steven Oliver <oliver.steven@gmail.com>
Signed-off-by: Uli Schlachter <psychon@znc.in>

Fix debug.dump

dump_return is no longer local, so debug prefix must be provided

Signed-off-by: Alexander Yakushev <yakushev.alex@gmail.com>
Signed-off-by: Julien Danjou <julien@danjou.info>

Remove incorrect comment.

The FindLua module from CMake now displays the correct Lua version.

Signed-off-by: Steven Oliver <oliver.steven@gmail.com>
Signed-off-by: Julien Danjou <julien@danjou.info>

layout: Remove duplicate capi definition (FS#1032)

Signed-off-by: Uli Schlachter <psychon@znc.in>

Don't strip RPATH on Solaris (FS#744)

Signed-off-by: Steven Oliver <oliver.steven@gmail.com>
Signed-off-by: Uli Schlachter <psychon@znc.in>

Add NoDisplay to .desktop file (FS#1031)

Signed-off-by: Uli Schlachter <psychon@znc.in>

Add mousegrabber and keygrabber isrunning luadoc

Signed-off-by: Sébastien Luttringer <seblu@seblu.net>
Signed-off-by: Uli Schlachter <psychon@znc.in>

Add awful.client.setmaster

This function is the counterpart of awful.client.setslave.

Windows is moved to the first position in the stack.

Signed-off-by: Sébastien Luttringer <seblu@seblu.net>
Signed-off-by: Uli Schlachter <psychon@znc.in>

run_or_raise: fix for lua 5.2 upgrade

Signed-off-by: Anurag Priyam <anurag08priyam@gmail.com>
Signed-off-by: Uli Schlachter <psychon@znc.in>

awful.client: Add a missing "local"

Reported by | and his strict.lua.

Signed-off-by: Uli Schlachter <psychon@znc.in>

common/util.h: dodgy non-__GNUC__ p_delete

I assume nobody have tried to compile Awesome with GNU uncompatible
compiler for ages and thus non-__GNUC__ p_delete version got
overlooked for quite some time.
First of all, a problem I see is that it assigns void** to a variable
of type void* and then dereferences the same void* variable.
None of the compilers I am aware of will let you go through this
without an error.
And second of all, lets have one portable p_delete.

Signed-off-by: Arvydas Sidorenko <asido4@gmail.com>
Signed-off-by: Uli Schlachter <psychon@znc.in>

Re-add wallpapers to the themes

The commit "Remove all traces of awsetbg and wallpaper setters" removed these.
This commit adds them back, but they are now applied a little different. The
main reason is that this makes it really easy for the user to ignore the
wallpaper that a theme specifies.

Signed-off-by: Uli Schlachter <psychon@znc.in>

gears.wallpaper: Import

This is a lua module for managing the wallpaper.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Remove all traces of awsetbg and wallpaper setters

Signed-off-by: Uli Schlachter <psychon@znc.in>

root: Add a wallpaper setter

This allows lua code to set a wallpaper directly instead of having to spawn some
external tools which possibly aren't installed.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Move some helper functions to draw.h

Signed-off-by: Uli Schlachter <psychon@znc.in>

Handle execv() errors

Previously, awesome would just crash when execv() fails, because it already
destroyed all of its internal state, but then tries to do another main loop
iteration. Whoops.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Print signal number while dying

Signed-off-by: Uli Schlachter <psychon@znc.in>

Unminimize before checking for tag switch (FS#1020)

For a minimized client, c:isvisible() is always false which is not what we want
here. So instead, unminimize the client before checking if it's visible.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Ignore (Un)Grab focus notifies

When a keyboard grab activate/deactives, the input focus jumps to the window
which does the grab. These FocusIn events sometimes managed to confuse awesome.
The symptom was that a newly mapped/started client didn't receive the input
focus:

- You press your key-combo to start a terminal. This activates a passive grab
  and the input focus jumps to the root window
- The terminal opens and the "manage" rule does client.focus = c
- This doesn't set the focus yet, but instead causes
  globalconf.focus.need_update = true and .client = new_terminal
- Before the focus is updated, the key combo is released and the focus jumps
  back to the previously focused client
- The FocusIn with mode == Ungrab causes awesome to think that the previously
  focused client got focused again and thus globalconf.client.focus gets re-set
  to what it was before
- Finally the focus is refreshed and the previously focused client gets focused

The fix is simple: We don't need the FocusIn events which are generated when a
grab activates or deactivates, so we can just ignore them.

Thanks to Majic for reporting this to me.

Signed-off-by: Uli Schlachter <psychon@znc.in>

rc.lua: Call awful.tag.viewnext/prev with a screen number

This is necessary because of commit fa36bd0a5964a3a5da31084541759666e
"tag.viewidx should take a screen index".

Signed-off-by: Uli Schlachter <psychon@znc.in>

awful.rules: Implement callbacks for individual properties

When a property is now set to a function, the function's return value will be
used for the value of the property. The function gets the new client as its only
argument.

There is no property which accepts a function as its value and thus this change
can't break anything (yeah, famous last words...).

This should fix half of FS#1011. Panels now don't get focused by awful.rules.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Ignore re-focusing the focused client

When something gives the input focus to the client which already has the input
focus, bad things can happen. Normally, you'd expect nothing to happen in this
case, but X11 is not that simple.

When updating the input focus and the focused client has the nofocus hint set,
we are actually taking away the focus from this client.

Hopefully this fixes FS#973.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Don't change focus in response to FocusIn events

Previously, when we received a FocusIn event, we would update the input focus,
because client_focus_update set globalconf.focus.need_update to true.

However, this is wrong for clients following the globally active focus model,
because in this case its the client which controls which window has the input
focus. It could happen that we thus took away the focus from the client which
just gave itself the focus.

Signed-off-by: Uli Schlachter <psychon@znc.in>

rotate: Use the draw_widget() function

Without this, input handling is broken (draw_widget() registers each widget with
its position for input handling).

This also removes some unnecessary save()/restore() calls. Our caller (again
draw_widget()) already does that for us.

Signed-off-by: Uli Schlachter <psychon@znc.in>

Fix spelling in doc string

Signed-off-by: Felix Bier <flx.bier@googlemail.com>
Signed-off-by: Uli Schlachter <psychon@znc.in>

tag.viewidx should take a screen index

This makes tag.viewidx take a screen index rather than a screen object,
complying with its doc string. This fixes #963.

Signed-off-by: Felix Bier <flx.bier@googlemail.com>
Signed-off-by: Uli Schlachter <psychon@znc.in>

Menubar: some more prefixes

Signed-off-by: Ignas Anikevicius (gns_ank) <anikevicius@gmail.com>
Signed-off-by: Uli Schlachter <psychon@znc.in>

Yet another lua call without prefixed module

Signed-off-by: Arvydas Sidorenko <asido4@gmail.com>
Signed-off-by: Julien Danjou <julien@danjou.info>

gears.surface: Update required lgi version

Signed-off-by: Uli Schlachter <psychon@znc.in>

Broken lgi version checking

Currently it takes `require('lgi.version')`, converts it blindly to
number and compares with another value.
I am using lgi v0.6.1 and that is not convertable to number value,
thus tonumber() returns nil and crashes Awesome.
The fix is to take major and minor numbers only ignoring the rest.

Signed-off-by: Arvydas Sidorenko <asido4@gmail.com>
Signed-off-by: Uli Schlachter <psychon@znc.in>

Do not attempt to call global destroy

Signed-off-by: Felix Bier <flx.bier@googlemail.com>
Signed-off-by: Uli Schlachter <psychon@znc.in>

awful.client: Add a missing module table

Signed-off-by: Uli Schlachter <psychon@znc.in>

More fixes for the recent module() removal

Signed-off-by: Uli Schlachter <psychon@znc.in>

Calling naughty.notify without specifing module

From 55eb706b698c3b0c59559eff342d6ee59ec9e3f3 Mon Sep 17 00:00:00 2001
From: Arvydas Sidorenko <asido4@gmail.com>
Date: Sun, 17 Jun 2012 15:11:01 +0200
Subject: [PATCH] Calling naughty.notify without specifing module

Lua 5.2 port bug, which shows up on notify-send

Signed-off-by: Arvydas Sidorenko <asido4@gmail.com>
Signed-off-by: Uli Schlachter <psychon@znc.in>

Fix various minor errors after the lua 5.2 conversion

Signed-off-by: Uli Schlachter <psychon@znc.in>

Portable way to use loadstring

Signed-off-by: Arvydas Sidorenko <asido4@gmail.com>
Signed-off-by: Uli Schlachter <psychon@znc.in>

Fixed bug with global `tags` in rc.lua

awful.wibox.tasklist was overwritting `tags` variable breaking all the
functionality which uses it.

Signed-off-by: Arvydas Sidorenko <asido4@gmail.com>
Signed-off-by: Uli Schlachter <psychon@znc.in>

Ported awful.layout.suit to lua 5.2

Tested with lua 5.1: all good

Signed-off-by: Arvydas Sidorenko <asido4@gmail.com>
Signed-off-by: Uli Schlachter <psychon@znc.in>

Ported awful.layout to lua 5.2

Tested with lua 5.1: all good

Signed-off-by: Arvydas Sidorenko <asido4@gmail.com>
Signed-off-by: Uli Schlachter <psychon@znc.in>