All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michael Wood <michael.g.wood@intel.com>
To: "Barros Pena, Belen" <belen.barros.pena@intel.com>
Cc: "toaster@yoctoproject.org" <toaster@yoctoproject.org>
Subject: Re: [review-request] michaelw/toaster/new-build-button-v2
Date: Tue, 18 Nov 2014 14:42:13 +0000	[thread overview]
Message-ID: <546B5AC5.4030005@intel.com> (raw)
In-Reply-To: <D090EAD3.4E512%belen.barros.pena@intel.com>

Thanks. I've pushed that fix for that to the same branch.

On 18/11/14 12:29, Barros Pena, Belen wrote:
> Hi Michael,
>
> This is working really nicely. I just found a couple of tiny issues with
> the layout. There is a patch here
>
> http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/commit/?h=bbarrosp/n
> ew-build-button2&id=dbd175cf9d74ce57382444baab0d516562397ab6
>
> I've also found a small problem with the project input field: the save
> button should be disabled whenever the field is empty, which is not the
> case right now. This is how I tested it: in the new build dialog, click on
> the 'change' icon, select a different project and then, without clicking
> 'save', delete the project name from the input field. The 'save' button is
> still enabled (it should be disabled) and, if you click it, it will set
> the project to the project name you just deleted from the input field,
> which is a bit weird. We can avoid this kind of thing (and also having to
> show error messages) by disabling the buttons when their input fields are
> empty.
>
> Thanks!!
>
> Belén
>
>
>
> On 17/11/2014 15:07, "Michael Wood" <michael.g.wood@intel.com> wrote:
>
>> poky-contrib michaelw/toaster/new-build-button-v2
>>
>> bitbake: toaster: project page remove additional tooltip for Change pencil
>> bitbake: toaster: libtoaster: fix missing tooltip hover on pencil icon
>> bitbake: toaster: Add New Build Button feature
>> bitbake: toaster: libtoaster: Add getProjectInfo utility function
>> bitbake: toaster: Create libtoaster.js with some utility functions
>>
>> Thanks,
>>
>> Michael
>> -- 
>> _______________________________________________
>> toaster mailing list
>> toaster@yoctoproject.org
>> https://lists.yoctoproject.org/listinfo/toaster



  reply	other threads:[~2014-11-18 14:42 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-17 15:07 [review-request] michaelw/toaster/new-build-button-v2 Michael Wood
2014-11-18 12:29 ` Barros Pena, Belen
2014-11-18 14:42   ` Michael Wood [this message]
2014-11-18 18:03     ` Barros Pena, Belen
2014-11-20 10:47       ` Damian, Alexandru

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=546B5AC5.4030005@intel.com \
    --to=michael.g.wood@intel.com \
    --cc=belen.barros.pena@intel.com \
    --cc=toaster@yoctoproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.