Bug 5919

Summary:	Interaction between sorting and "Edit columns" menu not working as designed
Product:	[Build System, Metadata & Runtime] Toaster	Reporter:	Belen Barros Pena <belen.barros.pena>
Component:	toaster	Assignee:	Farrell Wymore <farrell.wymore>
Status:	RESOLVED FIXED	QA Contact:	Alexandru Roman <alexandru.costinx.roman>
Severity:	normal
Priority:	Medium	CC:	alexandru.damian, belen.barros.pena, cristiana.voicu, david.reyna, jessica.zhang, valentinx.hangan, Yuan.Sun2
Version:	1.6.1
Target Milestone:	1.7 M3
Hardware:	All
OS:	Multiple
Whiteboard:
OS type for building Yocto:	---	Type of Regression:	---
Verified:		Documentation change:	No (bug/feature does not impact docs)
Bug Depends on:
Bug Blocks:	6291

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