Question about wibox.widget.textbox:set_font(font) and other similar methods

[Sorky]

@psychon

What do you (and others) think of...

function textbox:set_font(font)
    self._private.layout:set_font_description(beautiful.get_font(font))
    self:emit_signal("widget::redraw_needed")
    self:emit_signal("widget::layout_changed")
 -- vvvvv
    return self  
 -- ^^^^^
end

To allow...

local textbox = wibox.widget.textbox( "Hi" ):set_font( font )

PS: This method doesn't seem to appear in the doco

[psychon]

I don't mind that.

PS: This method doesn't seem to appear in the doco

It is documented as a property and not a method: https://awesomewm.org/doc/api/classes/wibox.widget.textbox.html#wibox.widget.textbox.font

[Sorky]

I was able to do local textbox = wibox.widget.textbox( "Hi" ) textbox.font = font and local textbox = wibox.widget.textbox( "Hi" ) textbox:set_font( font )

Same? Surely the method is additional in this case? Causes a refresh? Which is better?

[Ryuno-Ki]

Reminds me on jQuery way of writing …

[Sorky]

Just like lua string methods

[Elv13]

Just like lua string methods

No. Lua string return a new string, not a reference to the old string.

[Sorky]

I see it as an object to work with - new/old/reference/whatever