diff --git a/.travis.yml b/.travis.yml index 55014ebb3..c50c925c7 100644 --- a/.travis.yml +++ b/.travis.yml @@ -86,7 +86,7 @@ install: # Install busted for "make check-unit". - travis_retry sudo luarocks install busted # Install luacheck for "make check-qa". - - if [ "$DO_QUECKQA" = 1 ]; then travis_retry sudo luarocks install luacheck; fi + - if [ "$DO_CHECKQA" = 1 ]; then travis_retry sudo luarocks install luacheck; fi # Install ldoc for building docs. - travis_retry sudo luarocks install ldoc 1.4.4 diff --git a/tests/_multi_screen.lua b/tests/_multi_screen.lua index d7757f10c..c333db872 100644 --- a/tests/_multi_screen.lua +++ b/tests/_multi_screen.lua @@ -1,3 +1,6 @@ +-- FIXME See commit 7901a1c6479aea +local skip_wallpapers_due_to_weird_slowdowns = true + -- Helper module to replicate a set of steps across multiple screen scenarios -- Ideas for improvements: -- * Export an iterator so the suits can call the screen change step by step @@ -348,58 +351,60 @@ local colors = { -- Paint it black local function clear_screen() --- for s in screen do --FIXME very, very slow on Travis --- local sur = surface.widget_to_surface( --- wibox.widget { --- bg = "#000000", --- widget = wibox.container.background --- }, --- s.geometry.width, --- s.geometry.height --- ) --- wallpaper.fit(sur, s, "#000000") --- end + if skip_wallpapers_due_to_weird_slowdowns then + return + end + for s in screen do + local sur = surface.widget_to_surface( + wibox.widget { + bg = "#000000", + widget = wibox.container.background + }, + s.geometry.width, + s.geometry.height + ) + wallpaper.fit(sur, s, "#000000") + end end -- Make it easier to debug the tests by showing the screen geometry when the -- tests are executed. local function show_screens() - --FIXME very, very slow on Travis --- wallpaper.set(color("#000000")) -- Should this clear the wallpaper? It doesn't --- --- -- Add a wallpaper on each screen --- for i=1, screen.count() do --- local s = screen[i] --- --- local w = wibox.widget { --- { --- text = table.concat{ --- "Screen: ",i,"\n", --- s.geometry.width,"x",s.geometry.height, --- "+",s.geometry.x,",",s.geometry.y --- }, --- valign = "center", --- align = "center", --- widget = wibox.widget.textbox, --- }, --- bg = colors[i], --- fg = "#ffffff", --- shape_border_color = "#ff0000", --- shape_border_width = 1, --- shape = shape.rectangle, --- widget = wibox.container.background --- } --- local sur = surface.widget_to_surface( --- w, --- s.geometry.width, --- s.geometry.height --- ) --- wallpaper.fit(sur, s) --- end + if skip_wallpapers_due_to_weird_slowdowns then + wallpaper.maximized = function() end + return + end + wallpaper.set(color("#000000")) -- Should this clear the wallpaper? It doesn't - --TODO remove this line, all it does it saving some time in the tests until - -- the bug is fixed. - wallpaper.maximized = function() end + -- Add a wallpaper on each screen + for i=1, screen.count() do + local s = screen[i] + + local w = wibox.widget { + { + text = table.concat{ + "Screen: ",i,"\n", + s.geometry.width,"x",s.geometry.height, + "+",s.geometry.x,",",s.geometry.y + }, + valign = "center", + align = "center", + widget = wibox.widget.textbox, + }, + bg = colors[i], + fg = "#ffffff", + shape_border_color = "#ff0000", + shape_border_width = 1, + shape = shape.rectangle, + widget = wibox.container.background + } + local sur = surface.widget_to_surface( + w, + s.geometry.width, + s.geometry.height + ) + wallpaper.fit(sur, s) + end end local function add_steps(real_steps, new_steps) diff --git a/tests/examples/wibox/logo/logo.lua b/tests/examples/wibox/logo/logo.lua index 7d9009fa6..734ad2b89 100644 --- a/tests/examples/wibox/logo/logo.lua +++ b/tests/examples/wibox/logo/logo.lua @@ -1,10 +1,6 @@ local parent = ... --DOC_HIDE local wibox = require( "wibox" ) --DOC_HIDE -local beautiful = require( "beautiful" ) --DOC_HIDE local assets = require( "xresources.assets" ) --DOC_HIDE -local color = require( "gears.color" ) --DOC_HIDE - -local size = 128 --DOC_HIDE parent:add( --DOC_HIDE diff --git a/tests/examples/wibox/logo/logo_and_name.lua b/tests/examples/wibox/logo/logo_and_name.lua index ef160856e..126d7dbe4 100644 --- a/tests/examples/wibox/logo/logo_and_name.lua +++ b/tests/examples/wibox/logo/logo_and_name.lua @@ -1,10 +1,6 @@ local parent = ... --DOC_HIDE local wibox = require( "wibox" ) --DOC_HIDE -local beautiful = require( "beautiful" ) --DOC_HIDE local assets = require( "xresources.assets" ) --DOC_HIDE -local color = require( "gears.color" ) --DOC_HIDE - -local size = 128 --DOC_HIDE parent:add( --DOC_HIDE @@ -13,7 +9,7 @@ wibox.widget { draw = function(_, _, cr) assets.gen_logo(cr, 128, 128, nil, "#535d6c") cr:translate(128 + 128/16, 0) - assets.gen_awesome_name(cr, 0, 158, nil, "#535d6c", alt_fg, true) + assets.gen_awesome_name(cr, 158, nil, "#535d6c", nil) end, widget = wibox.widget.base.make_widget } diff --git a/themes/xresources/assets.lua b/themes/xresources/assets.lua index 3c7f4e8a6..9f6bbfa0e 100644 --- a/themes/xresources/assets.lua +++ b/themes/xresources/assets.lua @@ -61,7 +61,7 @@ local function make_letter(cr, n, lines, size, bg, fg, alt_fg) end lines = lines or {} - color = alt_fg or fg + local color = alt_fg or fg cr:set_source(gears.color(color)) cr:rectangle( 0, (size+letter_gap)*n, @@ -83,7 +83,7 @@ local function make_letter(cr, n, lines, size, bg, fg, alt_fg) cr:set_operator(cairo.Operator.OVER) end -function theme_assets.gen_awesome_name(cr, width, height, bg, fg, alt_fg, vertical) +function theme_assets.gen_awesome_name(cr, height, bg, fg, alt_fg) local ls = height/10 -- letter_size local letter_line = ls/18 @@ -178,7 +178,7 @@ function theme_assets.wallpaper(bg, fg, alt_fg, s) cr:set_source(gears.color(bg)) cr:paint() - theme_assets.gen_awesome_name(cr, width, height, bg, fg, alt_fg, true) + theme_assets.gen_awesome_name(cr, height, bg, fg, alt_fg) return img end