[Toaster] V2: Commit for recipe detail view ready for review

Barros Pena, Belen belen.barros.pena at intel.com
Tue Jan 21 02:21:48 PST 2014


Sorry, David. Just came across a Homepage: unknown (in the recipe
update-rc.d-native). The recipe has no HOMEPAGE variable set, so the About
section should show no homepage information (only summary, description,
section and license).

Cheers

Belén

On 21/01/2014 10:15, "Barros Pena, Belen" <belen.barros.pena at intel.com>
wrote:

>dreyna/recipe-detail-view-v2 is looking good. All issues expect the layout
>seem fixed. Thanks David!
>
>
>I'll speak to AlexD about the layout of basebuilddetailpage.html
>
>There is one other minor thing. The openembedded-core layer is shown as
>"meta" in Toater. Could we show the Layer name as openembedded-core
>instead? The layer directory should still be /poky/meta/
>
>Cheers
>
>Belén
>
>On 21/01/2014 07:26, "Reyna, David" <david.reyna at windriver.com> wrote:
>
>>Hi Belen,
>> 
>>I will look for you tonight, to see if we can resolve the alignment
>>problem that I do not yet see.
>>
>> 
>>Thanks!
>>David
>> 
>>From: Damian, Alexandru [mailto:alexandru.damian at intel.com]
>>
>>Sent: Monday, January 20, 2014 4:32 AM
>>To: Reyna, David
>>Cc: Barros Pena, Belen; toaster at yoctoproject.org
>>Subject: Re: [Toaster] V2: Commit for recipe detail view ready for review
>>
>>
>> 
>>David,
>>
>>I took in the patch
>>
>>> Implementation of base build details page
>>
>>with the title prefixed with "bitbake: toaster: "
>>
>>For the
>>
>>> Implementation of recipe detail views
>>
>>please:
>>
>>* update the title to be prefixed with  "bitbake: toaster: "
>>
>>* fix the end-of-line whitespace
>>
>>* sort out the alignment issues reported by Belen :)
>>
>>Cheers,
>>Alex
>>
>>
>> 
>>On Sat, Jan 18, 2014 at 1:13 AM, Reyna, David <david.reyna at windriver.com>
>>wrote:
>>Hi all,
>>
>>* I have pushed my Version 2 commit (dreyna/recipe-detail-view-v2) for
>>the recipe details page for review. There are two separate commits for
>>tracking purposes. (I tried to modify my V1 push, but it did not take).
>>
>>* I have fixed the issues identified by Belen, except that I have a
>>question with the issue "small layout issue that makes the heading and
>>the tabs not to align properly".
>>
>>I do not see this issue. Moreover, here is the HTML layout as reflected
>>by Firebug:
>>
>>  HTML file             Schema
>>  -------------------- --------------------
>>  base                |    <body>
>>  base                |        <div class="navbar
>>navbar-static-top"></div>
>>  base                |        <div class="container-fluid">
>>  basebuilddetailpage |          <div class="">
>>  basebuilddetailpage |            <div class="section"></div>
>>  basebuilddetailpage |            <div class="row-fluid">
>>  recipe              |                <div class="row span11"></div>
>>  recipe              |                <div class="row span7
>>tabbable"></div>
>>  recipe              |                <div class="row span4 well"></div>
>>  basebuilddetailpage |            </div>
>>  basebuilddetailpage |          </div>
>>  base                |        </div>
>>  base                |    </body>
>>
>>The page follows Belen's advised layout, with slight variations in
>>"basebuilddetailpage.html", but that file follows the same layout as
>>"basebuilddetailpage.html" which I inherited.
>>
>>Belen, what alignment artifacts do you see?
>>
>>- David
>>
>>> -----Original Message-----
>>> From: Barros Pena, Belen [mailto:belen.barros.pena at intel.com]
>>> Sent: Friday, January 17, 2014 8:14 AM
>>> To: Reyna, David
>>> Cc: toaster at yoctoproject.org
>>> Subject: Re: Commit for recipe detail view ready for review
>>>
>>> Hi David,
>>>
>>> Thanks for pushing this: it¹s looking great :)
>>>
>>> From a UI standpoint, the page seems to have a small layout issue that
>>> makes the heading and the tabs not to align properly. To fix it, the
>>> body
>>> structure should look like this:
>>>
>>> <body>
>>>     <div class=³navbar navbar-static-top²></div>
>>>     <div class=³container-fluid²>
>>>         <div class=³row-fluid²>
>>>           <div class=³section²></div>
>>>           <div class=³row span11²></div>
>>>           <div class=³row span7 tabbable²></div>
>>>           <div class=³row span4 well²></div>
>>>         </div>
>>>     </div>
>>> </body>
>>>
>>> You need to add the class row-fluid to the child of container-fluid,
>>> and
>>> remove the div with class row-fluid that is a sibling of the div with
>>> class section. That will align all content nicely.
>>>
>>> Only 3 other things:
>>>
>>> 1. Next to the layer name you are showing the blue icon that links to
>>> the
>>> layer index. We currently have an issue with this that we still don¹t
>>> know
>>> how to solve (see 5192 in Bugzilla). Can we take the icon out for the
>>> moment until we find the right solution?
>>>
>>> 2. I am finding recipes that report no tasks in their tasks table (see
>>> attached image). This should not happen, and in fact when you go to the
>>> tasks list there are tasks for those recipes.
>>>
>>> 3. The Bugtracker value seems to have a trailing / (see attached
>>> image).
>>> So if there is no bugtracker specified for the recipe, the / shows. If
>>> any
>>> of the variables used to populate the information about the recipe is
>>> not
>>> specified in the recipe file, we simply don¹t show them. For example,
>>> if
>>> there is no bugtracker, we show only summary, description, homepage,
>>> section and license.
>>>
>>> Any questions, let me know.
>>>
>>> Cheers
>>>
>>> Belén
>>>
>>>
>>_______________________________________________
>>toaster mailing list
>>toaster at yoctoproject.org
>>https://lists.yoctoproject.org/listinfo/toaster
>>
>>
>>
>>
>>-- 
>>Alex Damian
>>Yocto Project
>>
>>SSG / OTC 
>>
>>
>>
>>
>>
>
>_______________________________________________
>toaster mailing list
>toaster at yoctoproject.org
>https://lists.yoctoproject.org/listinfo/toaster



More information about the toaster mailing list