Re: [Linuxwacom-devel] [PATCH libwacom] data: set the 22HD(T) to allow for touch strip modes

2016-10-28 Thread Bastien Nocera
On Fri, 2016-10-28 at 16:23 -0700, Ping Cheng wrote:
> On Thu, Oct 27, 2016 at 2:53 AM, Bastien Nocera 
> wrote:
> > On Thu, 2016-10-27 at 13:39 +1000, Peter Hutterer wrote:
> > > There is no physical LED to show the current mode, but the tablet
> > > was
> > > clearly
> > > intended to be used that way [1] and we already reserve two
> > > buttons
> > > as mode
> > > switch buttons anyway. Declare it correctly and let the userspace
> > > stack worry
> > > about displaying mode switches (it's already responsible for
> > > changing
> > > LEDs
> > > anyway).
> > > 
> > > [1] "The ambidextrous design of the Cintiq 22HD touch features a
> > > pair
> > > of
> > > rear-mounted Touch Strips, along with accompanying Touch Strip
> > > Toggle
> > > buttons.
> > > Each controls up to four application-specific functions, such as
> > > brush size,
> > > zooming, scrolling and on-screen canvas rotation."
> > > https://buywacom.com.au/cintiq-22hd-touch.html
> > 
> > Looks like this would need user-space changes to show an OSD when
> > the
> > feature is being used. In the meanwhile, I'd hold off on making a
> > change until at least one front-end implemented the functionality.
> 
> Maybe it's not a bad idea to make the change now so front-end can
> rely
> on it when user-space is ready, as long as the change won't break the
> existing user-space implementation. Does that make sense?

If you made that change without the corresponding user-space changes,
you'd have a button that used to work as simply a button, and that now
would change modes without any feedback as to which mode it's using.

I would at least expect a bug filed explaining the intent (for GNOME I
guess), so that Carlos has a chance to look into how we'd hook the UI
for this.

It might be a one-liner once a lot of the calibration and setup UI has
moved into the shell but we won't know until we're there.

Cheers

--
The Command Line: Reinvented for Modern Developers
Did the resurgence of CLI tooling catch you by surprise?
Reconnect with the command line and become more productive. 
Learn the new .NET and ASP.NET CLI. Get your free copy!
http://sdm.link/telerik
___
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel


Re: [Linuxwacom-devel] [PATCH libwacom] data: set the 22HD(T) to allow for touch strip modes

2016-10-28 Thread Ping Cheng
On Thu, Oct 27, 2016 at 2:53 AM, Bastien Nocera  wrote:
> On Thu, 2016-10-27 at 13:39 +1000, Peter Hutterer wrote:
>> There is no physical LED to show the current mode, but the tablet was
>> clearly
>> intended to be used that way [1] and we already reserve two buttons
>> as mode
>> switch buttons anyway. Declare it correctly and let the userspace
>> stack worry
>> about displaying mode switches (it's already responsible for changing
>> LEDs
>> anyway).
>>
>> [1] "The ambidextrous design of the Cintiq 22HD touch features a pair
>> of
>> rear-mounted Touch Strips, along with accompanying Touch Strip Toggle
>> buttons.
>> Each controls up to four application-specific functions, such as
>> brush size,
>> zooming, scrolling and on-screen canvas rotation."
>> https://buywacom.com.au/cintiq-22hd-touch.html
>
> Looks like this would need user-space changes to show an OSD when the
> feature is being used. In the meanwhile, I'd hold off on making a
> change until at least one front-end implemented the functionality.

Maybe it's not a bad idea to make the change now so front-end can rely
on it when user-space is ready, as long as the change won't break the
existing user-space implementation. Does that make sense?

Ping

--
The Command Line: Reinvented for Modern Developers
Did the resurgence of CLI tooling catch you by surprise?
Reconnect with the command line and become more productive. 
Learn the new .NET and ASP.NET CLI. Get your free copy!
http://sdm.link/telerik
___
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel