Bug 5919 - Interaction between sorting and "Edit columns" menu not working as designed
Summary: Interaction between sorting and "Edit columns" menu not working as designed
Status: RESOLVED FIXED
Alias: None
Product: Toaster
Classification: Build System, Metadata & Runtime
Component: toaster (show other bugs)
Version: 1.6.1
Hardware: All Multiple
: Medium normal
Target Milestone: 1.7 M3
Assignee: Farrell Wymore
QA Contact: Alexandru Roman
URL:
Whiteboard:
: 5987 (view as bug list)
Depends on:
Blocks: 6291
  Show dependency tree
 
Reported: 2014-03-06 15:39 UTC by Belen Barros Pena
Modified: 2016-02-23 14:11 UTC (History)
7 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Belen Barros Pena 2014-03-06 15:39:08 UTC 
From the design spec:

If you use the "Edit columns" menu to hide the column with the applied sorting, we revert the sorting to the default sorting (i.e. "Order"). The default sorting always uses one of the core columns, which cannot be hidden using the "Edit columns" menu.

This is not happening: if I hide the column with the sorting applied, the table does not revert to the default sorting.

toaster/master:213fef6e87cbf60d773e28d31653e3c626d4058e
Comment 1 Belen Barros Pena 2014-03-17 15:07:55 UTC 
*** Bug 5987 has been marked as a duplicate of this bug. ***
Comment 2 Farrell Wymore 2014-03-27 20:56:27 UTC 
The default sort order is used when the current sorted column is hidden.
Comment 3 Belen Barros Pena 2014-07-31 16:32:58 UTC 
A fix for this was submitted for review to the Toaster mailing list

http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/log/?h=fwymore/sortcol-5919

but was never merged to master. Was it ever submitted to bitbake-devel?
Comment 4 Farrell Wymore 2014-07-31 16:43:57 UTC 
I submitted this twice; once for the original bug which Belen approved and once when asked to rebase.
Comment 5 Alexandru Damian 2014-08-27 17:07:38 UTC 
This is merged.
Comment 6 Belen Barros Pena 2014-09-04 09:13:29 UTC 
(In reply to comment #5)
> This is merged.

Well, if it's merged, it's not working :/
Comment 7 Belen Barros Pena 2014-09-19 14:12:35 UTC 
(In reply to comment #6)
> (In reply to comment #5)
> > This is merged.
> 
> Well, if it's merged, it's not working :/

Sorry, my bad. This seems to be working on master/280b6d0011f101e7a8edabd34c265b37a2f791ed