Mouse support

[tboox]

Mouse support

[waruqi]

Great, thank you very much for your contribution, I will review this patch.

[waruqi]

I have published v2.1 and upload it to luarocks.

[laelnasan]

So... reading through the src files of pdcurses I found out it was only missing a definition to support ncurses' mouse API. I've committed a single line that theoretically would solve it. I still don't have a windows machine set up, so if you would kindly do the testing it would be awesome. Sorry for the delay, I just didn't have the time till now.

[waruqi]

ok, I will test it.

[laelnasan]

Hi, me again
After some work I finally got the mouse working with all view objects (AFAIK) and tests
Since this is an optional feature I thought it would be fit to present it with a customization system.
The ideia is to fool the Lua's loading system by placing a ltui.lua file in a higher priority directory as, for example, the directory where the tests are called. It may also be nice for newcomers (like me) to have an example on how to customize the package to attend individual needs.

https://gist.github.com/laelnasan/92129fecd1e231c6492cd3babd3a0f29

[waruqi]

Great! 👍

Can you help submit pr to directly modify these views of ltui instead of customizing the views.

We can add options.mouseable = true/false to the init() function of each view to selectively enable the mouse feature.

[laelnasan]

That would be a bit of a grind, because every call to an init() of a child of panel would have to be changed, specially the instantiation of the objects inside dialogs would have to be conditioned to the property.
In the solution I presented, I change the behaviour of the classes at loading time so that all children inherit the changes automatically.
Not saying it cant be done, but I can't see an easy way to do it

[laelnasan]

I think I may have found a workaround... I just don't know if it fits the coding style: to use a metatable to dynamically load the ltui/* modules and set a global ltui.options

Then one could write something like:

local ltui = require('ltui')
ltui.options = {mouseable = true}

local app   = ltui.application
local label = ltui.label
...

[waruqi]

ltui.options = {mouseable = true}

like this:

ltui/src/ltui/textarea.lua

Lines 37 to 41 in 987b002

	-- mark as selectable
	self:option_set("selectable", true)
	-- enable progress
	self:option_set("progress", true)

[laelnasan]

yes, but that is a option of the instantiated object, right? not the class per se. and how do we identify when the entire application is mouseable or not?

[laelnasan]

ow, I think I get it now

[waruqi]

yes, but that is a option of the instantiated object, right? not the class per se. and how do we identify when the entire application is mouseable or not?

Each view component should allow the user to control whether it can enable the mouse if this view support mouse.

[laelnasan]

but if the default behaviour doesn't take a global property then the user must modify every single object instantiated to get it working, is that it? I am not trying to convince you, just making sure I understand the idea.

[waruqi]

but if the default behaviour doesn't take a global property then the user must modify every single object instantiated to get it working, is that it? I am not trying to convince you, just making sure I understand the idea.

We can also use application:option_set() and view:application():option() to set/get global default option to all sub-views.