[systemsettings] [Bug 325837] New: Mouse cursor theme kcm shows wrong resize cursor in preview for some themes

[Wolfgang Bauer]

https://bugs.kde.org/show_bug.cgi?id=325837

            Bug ID: 325837
           Summary: Mouse cursor theme kcm shows wrong resize cursor in
                    preview for some themes
    Classification: Unclassified
           Product: systemsettings
           Version: 4.11.2
          Platform: openSUSE RPMs
               URL: https://bugs.kde.org/show_bug.cgi?id=325763#c15
                OS: Linux
            Status: UNCONFIRMED
          Severity: minor
          Priority: NOR
         Component: kcm_mouse
          Assignee: unassigned-bugs at kde.org
          Reporter: wbauer at tmo.at
                CC: unassigned-bugs at kde.org

The mouse cursor theme kcm has a preview area for the selected cursor theme.
Normally (for the Oxygen themes e.g.) the resize cursor shown there is the one
with arrows from top-left to bottom-right ("size_fdiag").
For themes that don't provide the file "size_fdiag", the other one is shown
(arrows from bottom-left to top-right, which would be "size_bdiag" in Oxygen).

Presumably, this also causes the glitch that the wrong shape is shown as resize
hint in the current session after applying such a theme (see bug#325763 for
more details).

Reproducible: Always

Steps to Reproduce:
1. Open systemsettings and enter the "Cursor Theme" module
2. Select a theme that doesn't provide "size_fdiag", examples include:
crystalwhite (and variations like crystalblue), DMZ, Adwaita, redglass,
whiteglass and handhelds

Actual Results:  
Preview shows arrows from bottom-left to top-right for the resize cursor

Expected Results:  
Preview should show arrows from top-left to bottom-right for the resize cursor,
as is done with themes that do provide the file "size_fdiag" (Oxygen and
KDE_classic f.e.)

Apparently in XCursorTheme::findAlternative() (file
kcontrol/input/xcursor/xcursortheme.cpp) the alternatives for "size_bdiag" and
"size_fdiag" are swapped.

I will create a review request to fix this shortly, but want to test it first.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.