[xcursor-transparent-theme][Patch] Add several cursors

[xcursor-transparent-theme][Patch] Add several cursors

[Johannes Pointner]

This patch adds several cursor to the transparent-theme which are
used by Qt for example.

Signed-off-by: Johannes Pointner <johannes.pointner@gmail.com>
---
 cursors/Makefile.am | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/cursors/Makefile.am b/cursors/Makefile.am
index d699942..587f4e5 100644
--- a/cursors/Makefile.am
+++ b/cursors/Makefile.am
@@ -26,10 +26,17 @@ CURSOR_NAMES = \
 	bottom_tee \
 	center_ptr \
 	circle \
+	closedhand \
+	color-picker \
+	copy \
 	cross \
 	cross_reverse \
 	crossed_circle \
 	crosshair \
+	dnd-copy \
+	dnd-move \
+	dnd-no-drop \
+	dnd-none \
 	dot \
 	dot_box_mask \
 	dotbox \
@@ -40,21 +47,27 @@ CURSOR_NAMES = \
 	exchange     \
 	fd_double_arrow \
 	fleur \
+	forbidden \
 	gumby \
 	h_double_arrow \
 	hand \
 	hand1 \
 	hand2 \
+	ibeam \
 	left_ptr \
 	left_ptr_watch \
 	left_side \
 	left_tee  \
+	link \
 	ll_angle  \
 	lr_angle  \
 	move      \
+	openhand \
 	pencil    \
 	pirate    \
 	plus      \
+	pointing_hand \
+	progress \
 	question_arrow \
 	right_ptr      \
 	right_side     \
@@ -67,9 +80,17 @@ CURSOR_NAMES = \
 	sb_up_arrow \
 	sb_v_double_arrow \
 	shuttle \
+	size_all \
+	size_bdiag \
+	size_fdiag \
+	size_hor \
+	size_ver \
 	sizing  \
+	split_h \
+	split_v \
 	target  \
 	tcross  \
+	text \
 	top_left_arrow  \
 	top_left_corner \
 	top_right_corner \
@@ -77,9 +98,12 @@ CURSOR_NAMES = \
 	top_tee \
 	trek \
 	ul_angle \
+	up_arrow \
 	ur_angle \
 	v_double_arrow \
+	wait \
 	watch \
+	whats_this \
 	xterm  
 
 CURSOR_DIR = $(datadir)/icons/xcursor-transparent/cursors
-- 
2.7.2

Re: [xcursor-transparent-theme][Patch] Add several cursors

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 2571 bytes --]

Hi Johannes,

This and the other patch have been merged, thanks.

However I just realised that we don't actually need this anymore and should
instead be passing -nocursor when starting X11.

Ross


On 1 March 2016 at 23:52, Johannes Pointner <h4nn35.work@gmail.com> wrote:

> This patch adds several cursor to the transparent-theme which are
> used by Qt for example.
>
> Signed-off-by: Johannes Pointner <johannes.pointner@gmail.com>
> ---
>  cursors/Makefile.am | 24 ++++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
>
> diff --git a/cursors/Makefile.am b/cursors/Makefile.am
> index d699942..587f4e5 100644
> --- a/cursors/Makefile.am
> +++ b/cursors/Makefile.am
> @@ -26,10 +26,17 @@ CURSOR_NAMES = \
>         bottom_tee \
>         center_ptr \
>         circle \
> +       closedhand \
> +       color-picker \
> +       copy \
>         cross \
>         cross_reverse \
>         crossed_circle \
>         crosshair \
> +       dnd-copy \
> +       dnd-move \
> +       dnd-no-drop \
> +       dnd-none \
>         dot \
>         dot_box_mask \
>         dotbox \
> @@ -40,21 +47,27 @@ CURSOR_NAMES = \
>         exchange     \
>         fd_double_arrow \
>         fleur \
> +       forbidden \
>         gumby \
>         h_double_arrow \
>         hand \
>         hand1 \
>         hand2 \
> +       ibeam \
>         left_ptr \
>         left_ptr_watch \
>         left_side \
>         left_tee  \
> +       link \
>         ll_angle  \
>         lr_angle  \
>         move      \
> +       openhand \
>         pencil    \
>         pirate    \
>         plus      \
> +       pointing_hand \
> +       progress \
>         question_arrow \
>         right_ptr      \
>         right_side     \
> @@ -67,9 +80,17 @@ CURSOR_NAMES = \
>         sb_up_arrow \
>         sb_v_double_arrow \
>         shuttle \
> +       size_all \
> +       size_bdiag \
> +       size_fdiag \
> +       size_hor \
> +       size_ver \
>         sizing  \
> +       split_h \
> +       split_v \
>         target  \
>         tcross  \
> +       text \
>         top_left_arrow  \
>         top_left_corner \
>         top_right_corner \
> @@ -77,9 +98,12 @@ CURSOR_NAMES = \
>         top_tee \
>         trek \
>         ul_angle \
> +       up_arrow \
>         ur_angle \
>         v_double_arrow \
> +       wait \
>         watch \
> +       whats_this \
>         xterm
>
>  CURSOR_DIR = $(datadir)/icons/xcursor-transparent/cursors
> --
> 2.7.2
>
>

[-- Attachment #2: Type: text/html, Size: 3555 bytes --]

Re: [xcursor-transparent-theme][Patch] Add several cursors

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 351 bytes --]

Hi Johannes,

On 2 March 2016 at 12:19, Burton, Ross <ross.burton@intel.com> wrote:

> This and the other patch have been merged, thanks.
>

I've also merged in the patches that oe-core was carrying and did a bit of
a spring clean, so if you can test and submit a srcrev bump and patch
removal for oe-core that would be appreciated.

Ross

[-- Attachment #2: Type: text/html, Size: 749 bytes --]

Re: [xcursor-transparent-theme][Patch] Add several cursors

[Johannes Pointner]

Hello Ross,

thanks.

I didn't know this option.
But I think there is still a use case for the
xcursor-transparent-theme. Because the cursor theme can be set before
starting a new application without restarting X11.


2016-03-02 13:19 GMT+01:00 Burton, Ross <ross.burton@intel.com>:
> Hi Johannes,
>
> This and the other patch have been merged, thanks.
>
> However I just realised that we don't actually need this anymore and should
> instead be passing -nocursor when starting X11.
>
> Ross
>
>
> On 1 March 2016 at 23:52, Johannes Pointner <h4nn35.work@gmail.com> wrote:
>>
>> This patch adds several cursor to the transparent-theme which are
>> used by Qt for example.
>>
>> Signed-off-by: Johannes Pointner <johannes.pointner@gmail.com>
>> ---
>>  cursors/Makefile.am | 24 ++++++++++++++++++++++++
>>  1 file changed, 24 insertions(+)
>>
>> diff --git a/cursors/Makefile.am b/cursors/Makefile.am
>> index d699942..587f4e5 100644
>> --- a/cursors/Makefile.am
>> +++ b/cursors/Makefile.am
>> @@ -26,10 +26,17 @@ CURSOR_NAMES = \
>>         bottom_tee \
>>         center_ptr \
>>         circle \
>> +       closedhand \
>> +       color-picker \
>> +       copy \
>>         cross \
>>         cross_reverse \
>>         crossed_circle \
>>         crosshair \
>> +       dnd-copy \
>> +       dnd-move \
>> +       dnd-no-drop \
>> +       dnd-none \
>>         dot \
>>         dot_box_mask \
>>         dotbox \
>> @@ -40,21 +47,27 @@ CURSOR_NAMES = \
>>         exchange     \
>>         fd_double_arrow \
>>         fleur \
>> +       forbidden \
>>         gumby \
>>         h_double_arrow \
>>         hand \
>>         hand1 \
>>         hand2 \
>> +       ibeam \
>>         left_ptr \
>>         left_ptr_watch \
>>         left_side \
>>         left_tee  \
>> +       link \
>>         ll_angle  \
>>         lr_angle  \
>>         move      \
>> +       openhand \
>>         pencil    \
>>         pirate    \
>>         plus      \
>> +       pointing_hand \
>> +       progress \
>>         question_arrow \
>>         right_ptr      \
>>         right_side     \
>> @@ -67,9 +80,17 @@ CURSOR_NAMES = \
>>         sb_up_arrow \
>>         sb_v_double_arrow \
>>         shuttle \
>> +       size_all \
>> +       size_bdiag \
>> +       size_fdiag \
>> +       size_hor \
>> +       size_ver \
>>         sizing  \
>> +       split_h \
>> +       split_v \
>>         target  \
>>         tcross  \
>> +       text \
>>         top_left_arrow  \
>>         top_left_corner \
>>         top_right_corner \
>> @@ -77,9 +98,12 @@ CURSOR_NAMES = \
>>         top_tee \
>>         trek \
>>         ul_angle \
>> +       up_arrow \
>>         ur_angle \
>>         v_double_arrow \
>> +       wait \
>>         watch \
>> +       whats_this \
>>         xterm
>>
>>  CURSOR_DIR = $(datadir)/icons/xcursor-transparent/cursors
>> --
>> 2.7.2
>>
>

Re: [xcursor-transparent-theme][Patch] Add several cursors

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 679 bytes --]

On 2 March 2016 at 12:45, Johannes Pointner <h4nn35.work@gmail.com> wrote:

> I didn't know this option.
> But I think there is still a use case for the
> xcursor-transparent-theme. Because the cursor theme can be set before
> starting a new application without restarting X11.
>

How do you handle users switching between applications in that case? Or the
application crashing and leaving you without a cursor?  If a particular
application should hide the cursor them it should just hide the cursor, if
the entire UI shouldn't have a cursor then use -nocursor.

Don't worry, I won't be deleting the transparent theme just yet, but I do
think it's a hack.

Ross

[-- Attachment #2: Type: text/html, Size: 1168 bytes --]

Re: [xcursor-transparent-theme][Patch] Add several cursors

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 483 bytes --]

On 2 March 2016 at 12:48, Burton, Ross <ross.burton@intel.com> wrote:

> How do you handle users switching between applications in that case? Or
> the application crashing and leaving you without a cursor?  If a particular
> application should hide the cursor them it should just hide the cursor, if
> the entire UI shouldn't have a cursor then use -nocursor.
>

Or if you're using matchbox-window-manager, pass -use_cursor no, which will
also hide it system-wide.

Ross

[-- Attachment #2: Type: text/html, Size: 891 bytes --]

Re: [xcursor-transparent-theme][Patch] Add several cursors

[Johannes Pointner]

Of course its a hack :)
I will test the changes and send the patches.

Thanks,
Hannes

2016-03-02 13:50 GMT+01:00 Burton, Ross <ross.burton@intel.com>:
>
> On 2 March 2016 at 12:48, Burton, Ross <ross.burton@intel.com> wrote:
>>
>> How do you handle users switching between applications in that case? Or
>> the application crashing and leaving you without a cursor?  If a particular
>> application should hide the cursor them it should just hide the cursor, if
>> the entire UI shouldn't have a cursor then use -nocursor.
>
>
> Or if you're using matchbox-window-manager, pass -use_cursor no, which will
> also hide it system-wide.
>
> Ross

Re: [xcursor-transparent-theme][Patch] Add several cursors

[William Mills]

On 03/02/2016 07:48 AM, Burton, Ross wrote:
> 
> On 2 March 2016 at 12:45, Johannes Pointner <h4nn35.work@gmail.com
> <mailto:h4nn35.work@gmail.com>> wrote:
> 
>     I didn't know this option.
>     But I think there is still a use case for the
>     xcursor-transparent-theme. Because the cursor theme can be set before
>     starting a new application without restarting X11.
> 
> 
> How do you handle users switching between applications in that case? Or
> the application crashing and leaving you without a cursor?  If a
> particular application should hide the cursor them it should just hide
> the cursor, if the entire UI shouldn't have a cursor then use -nocursor.
>

How about transparent when last user input was touch-screen but normal
when last input was mouse/touchpad?

This would be done on a per DISPLAY basis not a per app basis.

Of course I don't know where to put that code and if I were doing it
today I would put it into a wayland based arch but ...

> Don't worry, I won't be deleting the transparent theme just yet, but I
> do think it's a hack.
> 
> Ross
> 
> 

Re: [xcursor-transparent-theme][Patch] Add several cursors

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 486 bytes --]

On 2 March 2016 at 15:36, William Mills <wmills@ti.com> wrote:

> How about transparent when last user input was touch-screen but normal
> when last input was mouse/touchpad?
>
> This would be done on a per DISPLAY basis not a per app basis.
>
> Of course I don't know where to put that code and if I were doing it
> today I would put it into a wayland based arch but ...
>

I guess you could do this in the window manager, calling XFixesShowCursor /
HideCursor().

Ross

[-- Attachment #2: Type: text/html, Size: 899 bytes --]