Bug 1352244

Summary: aisleriot can't find all icons in theme under wayland but fine under gtk
Product: [Fedora] Fedora Reporter: Trever Adams <trever>
Component: aisleriotAssignee: Matthias Clasen <mclasen>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 24CC: lsatenstein, mclasen, rstrode
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-05-09 02:13:48 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Don't try non-existant names
none
Better patch in that it also fixes the can drop icon none

Description Trever Adams 2016-07-02 23:01:24 UTC 
Description of problem:
(sol:5613): Gdk-WARNING **: gdkcursor-wayland.c:170: Unable to load openhand from the cursor theme

(sol:5613): Gdk-WARNING **: gdkcursor-wayland.c:170: Unable to load closedhand from the cursor theme


Version-Release number of selected component (if applicable):
aisleriot-3.20.2-1.fc24.x86_64
Comment 1 Trever Adams 2016-07-02 23:50:32 UTC 
It appears it is loading hand1 and grabbing instead. The problem is, in Wayland setups these are not displayed.
Comment 2 Trever Adams 2016-07-03 00:49:16 UTC 
Created attachment 1175444 [details]
Don't try non-existant names

gdk_cursor_new_from_name is now preferred over gdk_cursor_new_for_display.
It appears that gdk_cursor_new_from name breaks if the name is not found when used under Wayland, so reorder things.
Comment 3 Trever Adams 2016-07-03 02:09:44 UTC 
Created attachment 1175445 [details]
Better patch in that it also fixes the can drop icon
Comment 4 Trever Adams 2016-10-17 10:36:38 UTC 
https://bugzilla.gnome.org/show_bug.cgi?id=773077 should fix part of this problem. The rest are nice to have, but not critical to the functioning of the app.
Comment 5 Christian Persch 2016-10-17 16:29:08 UTC 
I've pushed the 2nd path in the upstream git.
Comment 6 Trever Adams 2016-10-17 19:15:51 UTC 
Thank you!
Comment 7 Leslie Satenstein 2016-11-26 20:14:29 UTC 
Month 7 t'il now, and so far, it has not made it into the fixes for Fedora 25.

Please investigate why this bug has not made it to Fedora 25 release version
Comment 8 Leslie Satenstein 2017-05-08 14:26:10 UTC 
Updates have corrected the session
Thank you

Please close
Comment 9 Trever Adams 2017-05-09 02:13:48 UTC 
It has been fixed.