Re: [PATCH 106/106] toaster: Unique project names to Boostrap 3

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

Phew! great work, thank you!!

So on my TODO list the next steps are

- Fix the indentation of patches #77, #80 , #81, 82, #91. #92, #93, #94, 
#96, #98, #102
- Drop #59
- Fixup/squash patches which rewrite the same parts.
- Fix the authorship of the typeahead patch - somehow it's been 
attributed to me
- Revert the bootstrap-toaster theme related stuff
- Fix the tests which probably will have broken

Then we'll merge it into toaster-next, get the build analysis stuff 
done, let it settle for a bit and then submit it upstream.

Michael

On 24/05/16 12:40, Belen Barros Pena wrote:
> We recently added functionality to enforce unique project names. Such
> functionality was marked up for Boostrap 2, and some changes are
> required to ensure it works with Bootstrap 3.
>
> Signed-off-by: Belen Barros Pena <belen.barros.pena@linux.intel.com>
> ---
>   bitbake/lib/toaster/toastergui/static/js/libtoaster.js | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/bitbake/lib/toaster/toastergui/static/js/libtoaster.js b/bitbake/lib/toaster/toastergui/static/js/libtoaster.js
> index a56e84a..e4e4f6c 100644
> --- a/bitbake/lib/toaster/toastergui/static/js/libtoaster.js
> +++ b/bitbake/lib/toaster/toastergui/static/js/libtoaster.js
> @@ -393,11 +393,11 @@ var libtoaster = (function () {
>                     data.results[0].name === projectName) {
>                   // This project name exists hence show the error and disable
>                   // the save button
> -                ctrlGrpValidateProjectName.addClass('control-group error');
> +                ctrlGrpValidateProjectName.addClass('has-error');
>                   hintError.show();
>                   enableOrDisableBtn.attr('disabled', 'disabled');
>                 } else {
> -                ctrlGrpValidateProjectName.removeClass('control-group error');
> +                ctrlGrpValidateProjectName.removeClass('has-error');
>                   hintError.hide();
>                   enableOrDisableBtn.removeAttr('disabled');
>                 }