From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by yocto-www.yoctoproject.org (Postfix) with ESMTP id 5AFACE00BC6 for ; Fri, 28 Mar 2014 11:02:10 -0700 (PDT) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga101.jf.intel.com with ESMTP; 28 Mar 2014 11:02:10 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.97,752,1389772800"; d="scan'208";a="482277194" Received: from irsmsx103.ger.corp.intel.com ([163.33.3.157]) by orsmga001.jf.intel.com with ESMTP; 28 Mar 2014 11:01:13 -0700 Received: from irsmsx152.ger.corp.intel.com (163.33.192.66) by IRSMSX103.ger.corp.intel.com (163.33.3.157) with Microsoft SMTP Server (TLS) id 14.3.123.3; Fri, 28 Mar 2014 18:01:11 +0000 Received: from irsmsx106.ger.corp.intel.com ([169.254.8.58]) by IRSMSX152.ger.corp.intel.com ([169.254.6.180]) with mapi id 14.03.0123.003; Fri, 28 Mar 2014 18:01:11 +0000 From: "Barros Pena, Belen" To: "Wymore, Farrell W (Wind River)" Thread-Topic: [review requested] column sort order Thread-Index: Ac9J+30KPcqtaOA/R9Oj8H9J++ol1gAfX5GAAArKy0AAAuL7gA== Date: Fri, 28 Mar 2014 18:01:10 +0000 Message-ID: References: <09A9B72E8E1E64419C95E3946F8A6709B13F93@ALA-MBA.corp.ad.wrs.com> <09A9B72E8E1E64419C95E3946F8A6709B14008@ALA-MBA.corp.ad.wrs.com> In-Reply-To: <09A9B72E8E1E64419C95E3946F8A6709B14008@ALA-MBA.corp.ad.wrs.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Microsoft-MacOutlook/14.3.9.131030 x-originating-ip: [10.237.224.30] MIME-Version: 1.0 Cc: "toaster@yoctoproject.org" Subject: Re: [review requested] column sort order X-BeenThere: toaster@yoctoproject.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Web based interface for BitBake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 28 Mar 2014 18:02:11 -0000 Content-Language: en-US Content-Type: text/plain; charset="iso-8859-1" Content-ID: Content-Transfer-Encoding: quoted-printable Hi Farrell, Thanks for this. With your changes, the following works correctly: * Sort the packages table by Size * Open the Edit columns menu * Deselect the Size column * Close the Edit columns menu When the menu closes, the Package sorting is applied. Nice! :) But if I try to do more than one thing in the Edit columns menu, the page refresh is triggered before I dismiss the menu. Try this: * Go to the Packages table * Sort it by Size * Open the Edit columns menu * Deselect the Size column * Select the Size column again * Then close the "Edit columns" menu The resorting is triggered before I dismiss the menu. So, I am starting to think this might be a bit too much work for an edge case. Would it be easier if we simply disable the checkbox for whichever column has the .sorted class applied? This means that when I sort by a certain column, if the checkbox for that column is not disabled by default, we disable it (add the disabled attribute to the input tag and the .muted class to the label tag). That might be an easy thing to do with jQuery.=20 Let me know what you think. Thanks!! Bel=E9n On 28/03/2014 16:42, "Wymore, Farrell" wrote: >Hi Belen, > >I have amended this branch to have the indicated behavior - no action >taken until the pop-up is closed by the user - it passes the test below. > > - fw > >-----Original Message----- >From: Barros Pena, Belen [mailto:belen.barros.pena@intel.com] >Sent: Friday, March 28, 2014 4:30 AM >To: Wymore, Farrell >Cc: toaster@yoctoproject.org >Subject: Re: [review requested] column sort order > > > >On 27/03/2014 20:33, "Wymore, Farrell" >wrote: > >>Hi Belen, >> >>Please review the correction for the column sort order issue #5919. > >Hi Farrell, > >This is behaving a bit funny. The sorting change should trigger only when >the user closes the "Edit columns" menu, and the "Edit columns" menu >should only close when the user closes it. > >I should be able to do something like: > >* Go to the Packages table >* Sort it by Size >* Open the Edit columns menu >* Deselect the Size column >* Select the Size column again >* Then close the "Edit columns" menu > >And the table should still be sorted by Size. > >Right now, when I deselect the column with the sorting applied, the "Edit >columns" menu closes by itself after a certain amount of time: that >should never happen. That menu should only ever be dismissed by clicking >the button or outside the menu area when the menu is open. Dismissing it >should trigger the sorting change when needed. > >Cheers > >Bel=E9n > >>I have applied this change to all displays that have tables with >>sortable/optional columns. >> >> >>Thanks in advance - fw >> >> > >