Bug 356305

Summary: Ascending/Descending sort arrow reversed
Product: [Plasma] Breeze Reporter: Unknown <null>
Component: QStyleAssignee: Hugo Pereira Da Costa <hugo.pereira.da.costa>
Status: RESOLVED FIXED    
Severity: normal CC: hugo.pereira.da.costa
Priority: NOR    
Version: 5.4.3   
Target Milestone: ---   
Platform: OpenSUSE   
OS: Linux   
Latest Commit: Version Fixed In:

Description Unknown 2015-12-05 11:16:26 UTC
It seems to me the arrow of the sorting interface should be the other way. It is pointing downward when things are sorted in ascending order.

Reproducible: Always

Steps to Reproduce:
1. Open Dolphin (or another app with sorting capabilities)
2. Sort files ascending
3. The arrow of the interface is pointing down

(and vice versa)

Actual Results:  
The arrow of the interface is pointing in the other direction, counter intuitive

Expected Results:  
Correct direction

Not sure if this is mean to work this way. Just noticed it a few times and decided to write.
Comment 1 Christoph Feck 2015-12-05 14:45:06 UTC

*** This bug has been marked as a duplicate of bug 297807 ***
Comment 2 Christoph Feck 2015-12-05 14:51:44 UTC
See also bug 349608.
Comment 3 Hugo Pereira Da Costa 2015-12-05 15:50:05 UTC
Re-oppening/reassigning.
Thing is, I have not ported the infamous option to breeze.
Also maybe we should move it upstream to kstyle 
(mmm but then that would break oxygen installation).
Comment 4 Hugo Pereira Da Costa 2015-12-05 17:10:26 UTC
Git commit c327915a8d864501ff0c7f701693225d457a8369 by Hugo Pereira Da Costa.
Committed on 05/12/2015 at 17:09.
Pushed by hpereiradacosta into branch 'master'.

added ViewInvertSortIndicator hidden option (to be added to $HOME/.config5/breezerc) in order to changed sort
order arrow orientation.
(copied from oxygen)

M  +4    -0    kstyle/breeze.kcfg
M  +3    -0    kstyle/breezestyle.cpp

http://commits.kde.org/breeze/c327915a8d864501ff0c7f701693225d457a8369
Comment 5 Christoph Feck 2015-12-06 03:18:18 UTC
Oh sorry, I had falsely assumed that Breeze was built on Oxygen code, and had the option already implemented.