Re: [review-request] bbarrosp/nav-project-changes

[Michael Wood <michael.g.wood@intel.com>]

On 03/08/15 11:23, Barros Pena, Belen wrote:
> Some tiny tweaks to the project pages after Michael's sweeping changes.
>
> http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/commit/?h=bbarrosp/n
> av-project-changes
>
> bitbake: toastergui: spacing and top nav style change
>
> 03d81dd9114073037207a30a9cb896ad06e50f95
>
>
> bitbake: toastergui: breadcrumb fixes
>
> 6dd1d2325ffb70754a1b4cd5b7006176d148ba85

To improve this code further you can use single quotes instead of having 
to escape the double quotes.

e.g.

.append('<span class="divider">&rarr;</span>');

>
> bitbake: toastergui: change link styles in alert-info
>
> acfc0dfda16bf9edcefb2dfefaea2f44cc7e635d
>
> bitbake: toastergui: remove project name from layer change notification
>
> c3606313e53c7614a09a065b9f4463363053d62a

If we don't need the project name and link can you also remove where the 
project name and link are set i.e. these two lines:

alertMsg.children("#project-affected-name").text(libtoaster.ctx.projectName);
     alertMsg.children("#project-affected-name").attr("href", 
libtoaster.ctx.projectPageUrl);



>
> bitbake: toastergui: fix spacing in the import layers page
>
> f5d3d554d20eaaa36e4a9e4e281567f82ed66c17
>
> bitbake: toastergui: add action to empty "most built" recipes
>
> 79ce966108ecea344e29a5ccece8f7a09f23d047
>
> Thanks!
>
> Belén

Everything else looks fine, Thanks!


Michael