Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Install grab and wait cursors #2453

Merged
merged 1 commit into from Oct 24, 2020
Merged

Install grab and wait cursors #2453

merged 1 commit into from Oct 24, 2020

Conversation

seb128
Copy link
Contributor

@seb128 seb128 commented Oct 21, 2020

Closes #2451

@seb128 seb128 mentioned this pull request Oct 21, 2020
@seb128
Copy link
Contributor Author

seb128 commented Oct 21, 2020

Tentative fix for #2451 , I don't really understand the build system for cursors though so it's just guess work. Does the win32 build need to be updated the same way (and how does one test the result on windows?)

@seb128
Copy link
Contributor Author

seb128 commented Oct 21, 2020

hum, also if not symlinking the watch.ani probably does need to be copied to wait.ani but that isn't in the same file?
(just doing a cp on disk without the .ani makes the testcase works so that might be enough)

@Feichtmeier
Copy link
Member

Thank you very much @seb128

CC @ubuntujaggers @Jupiter007-43 please help @seb128 😺

@Jupi007
Copy link
Member

Jupi007 commented Oct 21, 2020

I think @ubuntujaggers will be much more helpful than me. I had a hard time with my last cursors PR. 😥

@ubuntujaggers
Copy link
Contributor

Argh, cursors 😁

I will look at this as soon as I can 👍

@Feichtmeier
Copy link
Member

Feichtmeier commented Oct 23, 2020

Hi @seb128

Trying this command you've posted

python3 -c "from gi.repository import Gtk,Gdk,GObject; window = Gtk.Window(); window.show_all(); window.get_window().set_cursor(Gdk.Cursor.new_from_name(Gdk.Display.get_default(), 'wait')); Gtk.main()"

which pops out the window and the wait cursor. however, with yaru master the wait cursor shows up in 20.10 with yaru master.
when I do the same on my 20.04 host the adwaita cursor shows up.

Edit: and the same with Baobab - 20.10 yaru -> yaru cursor, 20.04 -> adwaita cursor

Trying chromium snap currently has no theming at all. It has that fuzzy x cursor and no yaru theme :|

Any other idea how I could test this?
Edit 2: I want to make sure I can confirm your issue before installing from your branch

@seb128
Copy link
Contributor Author

seb128 commented Oct 23, 2020

with yaru master the wait cursor shows up in 20.10 with yaru master.

Hum, maybe the issue is fixed in master? I'm trying on 20.10 ... is there any package from master I could try (or any other way to opt in for a trunk build?)

@Feichtmeier
Copy link
Member

you could

sudo apt install libgtk-3-dev git meson sassc
git clone https://github.com/ubuntu/yaru.git
cd yaru
meson build
cd build
sudo ninja install

@seb128
Copy link
Contributor Author

seb128 commented Oct 23, 2020

Ok, I don't understand it, the testcase works also in a VM for me, but also on a focal xubuntu live session when I just tried... need some more investigation

@seb128
Copy link
Contributor Author

seb128 commented Oct 23, 2020

seems like gtk has some mapping table
https://gitlab.gnome.org/GNOME/gtk/-/blob/gtk-3-24/gdk/x11/gdkcursor-x11.c#L623

the extra files might still be required for other toolkit or chromium though?

@Feichtmeier
Copy link
Member

Sure thing, we can merge it anyways thanks for the work
Maybe they changed the name in newer versions :) If you tested this yourself and it resolved the missing cursor let's merge it

@Feichtmeier
Copy link
Member

Okay, let's merge it, eventually we could include this in the yaru SRU to backport it for focal @seb128 ?

@Feichtmeier Feichtmeier merged commit 2c0e4d0 into ubuntu:master Oct 24, 2020
@seb128
Copy link
Contributor Author

seb128 commented Dec 1, 2020

Sorry for not replying earlier, including in one of the next SRUs should be fine yes, at the same time the main complain we received was with chromium which uses themes from the snap so that's probably low priority

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cursor icons missing
4 participants