Review Request 129904: fix when sort indicator is shown in list header

[Carlo Vanini]

> On Jan. 31, 2017, 11:06 p.m., Aleix Pol Gonzalez wrote:
> > src/PackageModel/PackageWidget.cpp, line 411
> > <https://git.reviewboard.kde.org/r/129904/diff/1/?file=490900#file490900line411>
> >
> >     Can you remove the argument then?

Yes, it can be removed. Also, this slot could be moved to the PackageViewHeader class.
If you agree I would move it and remove the argument in another commit, please see /r/129908


- Carlo


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129904/#review102332
-----------------------------------------------------------


On Feb. 1, 2017, 1:20 p.m., Carlo Vanini wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129904/
> -----------------------------------------------------------
> 
> (Updated Feb. 1, 2017, 1:20 p.m.)
> 
> 
> Review request for Kubuntu and Muon Package Management Suite.
> 
> 
> Bugs: 313596
>     http://bugs.kde.org/show_bug.cgi?id=313596
> 
> 
> Repository: muon
> 
> 
> Description
> -------
> 
> When starting a new search in the package list, and the list is sorted by name, then the results are sorted by relevancy and the indicator is hidden. Clicking on a column header changes the order wrt the selected column.
> 
> 
> Diffs
> -----
> 
>   src/PackageModel/PackageProxyModel.h fea49b7f4ff24202b405c377213677f5fc81cc9c 
>   src/PackageModel/PackageProxyModel.cpp 807ad07216e787b9d4c03d8783dd82afde5770f1 
>   src/PackageModel/PackageViewHeader.h ada54986cb006d8221938e4a0bd5bbeb458913a0 
>   src/PackageModel/PackageViewHeader.cpp e9557ec9ef7792dd55c7e96f423c999e9fd5057c 
>   src/PackageModel/PackageWidget.cpp e70e4d3b143769073e706312133375630944f17b 
> 
> Diff: https://git.reviewboard.kde.org/r/129904/diff/
> 
> 
> Testing
> -------
> 
> Test 1
> ------
> 
> * Click on 'Status' header
> * Search for 'fri' (gives meaningful result to see the order)
> * Result is sorted by status and then by name
> 
> Test 2
> ------
> 
> * Clear search string, click on 'Package' header
> * Search for 'fri'
> * Sort indicator is hidden, result is sorted by relevancy
> * Click on a header
> * Result is order by selected column, indicator is shown
> * Continue writing in search string, 'fritz'
> * Result is still ordered by column
> * Clearing search string and fill it again
> * Search result is sorted by relevancy
> 
> 
> Thanks,
> 
> Carlo Vanini
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/kubuntu-devel/attachments/20170201/477811d6/attachment-0001.html>