From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by yocto-www.yoctoproject.org (Postfix, from userid 118) id 9A182E009A3; Tue, 21 Apr 2015 05:34:46 -0700 (PDT) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on yocto-www.yoctoproject.org X-Spam-Level: X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_LOW autolearn=ham version=3.3.1 X-Spam-HAM-Report: * -0.7 RCVD_IN_DNSWL_LOW RBL: Sender listed at http://www.dnswl.org/, low * trust * [209.85.212.176 listed in list.dnswl.org] * -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% * [score: 0.0000] Received: from mail-wi0-f176.google.com (mail-wi0-f176.google.com [209.85.212.176]) by yocto-www.yoctoproject.org (Postfix) with ESMTP id 88105E00998 for ; Tue, 21 Apr 2015 05:34:44 -0700 (PDT) Received: by wiax7 with SMTP id x7so106290290wia.0 for ; Tue, 21 Apr 2015 05:34:43 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:message-id:date:from:user-agent:mime-version:to :subject:references:in-reply-to:content-type :content-transfer-encoding; bh=v8UIKOFn8sHI7M2z+gBPIOk21BhP4AG8Haik+029z7E=; b=FoSy6nTBvrHAvErmiAruButo16rr/CKDn+XPC7cOaz8RRFSoysF45EYl5UAogcBpga fx3CpFAgz7aoEpfvYxs8OmAvWp3mVv0CoyCm5348pp3WsMSYmQQqOgeUjxaql52/ZzZr d0b7KHQXlF8Fm9RkPXBahuIXlkVkEWt+ROuIDIetOAElFbCLZkFHHMNmm+7JwndwTp7n ou1YPcZgEOABhAHk3+CJoPpIv4umrXvYCZnfl0wWCaTPU8bYFghwVRA/i6yqhN5Cc7Xe ZCaHhINnRWjgMvfeWVqSaGYpm3Bu1F9fKxZ+vi7YOS5/9hVK1yqJAdtkzhX5U0RZVdye E7SA== X-Gm-Message-State: ALoCoQm00QFLCv1XneGZzyuxzISH7/3l3QQVU0KAOYykIjf7yEBp398F82S1efCidehAFOIFOFHd X-Received: by 10.194.211.8 with SMTP id my8mr41342776wjc.90.1429619683557; Tue, 21 Apr 2015 05:34:43 -0700 (PDT) Received: from [192.168.2.119] ([83.217.123.106]) by mx.google.com with ESMTPSA id la10sm2439591wjb.36.2015.04.21.05.34.42 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 21 Apr 2015 05:34:42 -0700 (PDT) Message-ID: <553643E1.5050500@intel.com> Date: Tue, 21 Apr 2015 13:34:41 +0100 From: Michael Wood User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: "Barros Pena, Belen" , "toaster@yoctoproject.org" References: <552D22BB.3060607@intel.com> In-Reply-To: Subject: Re: [review-request] michaelw/toaster-frontend-cleanups 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: Tue, 21 Apr 2015 12:34:46 -0000 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 8bit Version 2 pushed with more refactoring. I found out there were a few more places which had custom code for handling add/remove layer which was causing the differing behaviour. Also fixed the heading, a typo in the help text a number of jshint issues and the inline notifications for added layers. Summary: 18 files changed, 439 insertions(+), 731 deletions(-) .../lib/toaster/toastergui/static/css/default.css | 2 +- .../toastergui/static/html/layer_deps_modal.html | 17 ++ bitbake/lib/toaster/toastergui/static/js/base.js | 39 ++-- .../toaster/toastergui/static/js/importlayer.js | 20 +- .../lib/toaster/toastergui/static/js/layerBtn.js | 71 +++++++ .../toaster/toastergui/static/js/layerDepsModal.js | 90 +++++++++ .../toaster/toastergui/static/js/layerdetails.js | 99 ++------- .../lib/toaster/toastergui/static/js/libtoaster.js | 95 ++++++++- .../lib/toaster/toastergui/static/js/machines.js | 97 --------- .../lib/toaster/toastergui/static/js/projectapp.js | 31 ++- bitbake/lib/toaster/toastergui/templates/base.html | 19 +- .../toaster/toastergui/templates/importlayer.html | 6 +- .../toaster/toastergui/templates/layerdetails.html | 9 +- .../lib/toaster/toastergui/templates/layers.html | 224 +++------------------ .../toastergui/templates/layers_dep_modal.html | 99 --------- .../lib/toaster/toastergui/templates/machines.html | 35 ++-- .../lib/toaster/toastergui/templates/targets.html | 215 ++++---------------- bitbake/lib/toaster/toastergui/views.py | 2 +- commit a4ed04911d428944824f5b20f9ee642fdf5feea2 Author: Michael Wood Date: Tue Apr 21 11:59:37 2015 +0100 toaster: bitbake: Refactor and expand layer add remove mechanism We have multiple pages which have buttons to add and remove layers this patch adds functionality to libtoaster to abstract this and implements it in the pages affected. We handle loading and showing the dependencies dialog here too and generating the notification messages. Also implemented is using the selectmachine api from the projectapp to avoid having to handle this in each page that allows selecting machines. A small number of jshint issues, help text and the machine page name have also been fixed. Signed-off-by: Michael Wood commit 507fd79963318e7788aef057fe5f1c699032dd14 Author: Michael Wood Date: Tue Apr 21 11:38:03 2015 +0100 bitbake: toaster: projectapp Implement machine select command Use the project page to select the machine rather than setting it and then redirecting to the project page. This will also avoid having to have a special handler in the machines page it's self. Signed-off-by: Michael Wood commit 927c40e686736f70a0557f93bd42009dfbdefae8 Author: Michael Wood Date: Fri Apr 10 18:15:03 2015 +0100 bitbake: toaster: Move project context variables to common scope We have a bunch of context data which are used in multiple pages so it makes more sense to have this in a single place libtoaster.ctx that's accessible from each page rather than request it from every page. Signed-off-by: Michael Wood Michael On 16/04/15 15:11, Barros Pena, Belen wrote: > Hi Michael, > > On 14/04/2015 15:22, "Michael Wood" wrote: > >> poky-contrib michaelw/toaster-frontend-cleanups > I've checked what might be a v2 of this branch > (fd14503957379243e6ec34aea5f92b79dd2aefa8), and I have some comments: > > * Can we change the plural case in the notification show in the 'all > compatible' pages to match the one we use in the project page? The 'all > compatible' pages are using 'layer(s)' and the 'project' page says > 'layers'. In the first, the '(s)' bit is not really needed because the > word appears in singular when you only add one layer. Screenshots attached > > * The add layer and delete layer buttons are not working in the 'all > compatible layers' page, in the 'all compatible machines' page and in the > layer details page > > * The 'edit columns' menu seems to be broken in the 'all compatible > machines' page. All columns are shown by default, but when you check the > 'edit columns' menu content some of the checkboxes are not selected > > * The h1 in the 'all compatible machines' page says 'all machines' instead > of 'all compatible machines' > > Thanks! > > Belén > > > > >> >> commit 310409d14c1af131ac39121936a3bb66741c39dc >> Author: Michael Wood >> Date: Tue Apr 14 15:12:35 2015 +0100 >> >> bitbake: toaster: Create and use a single common layer add/remove >> mechanism >> >> This adds a common add/remove layer mechanism that will also handle >> showing >> the modal dialog if there are dependencies for the layer which also >> need >> adding. To help bring consistency this also adds a layer >> added/removed >> alert utility function to generate the alert message. This touches >> all >> places where layers can be added or removed to port them to this new >> common mechanism. >> >> Signed-off-by: Michael Wood >> >> commit c1c7a3435dcb70d82c2f9c710140c038cc9d205b >> Author: Michael Wood >> Date: Fri Apr 10 18:15:03 2015 +0100 >> >> bitbake: toaster: Move project context variables to common scope >> >> We have a bunch of context data which are used in multiple pages so >> it >> makes more sense to have this in a single place libtoaster.ctx that's >> accessible from each page rather than request it from every page. >> >> Signed-off-by: Michael Wood >> >> -- >> _______________________________________________ >> toaster mailing list >> toaster@yoctoproject.org >> https://lists.yoctoproject.org/listinfo/toaster