[PATCH conductor] BZ797208: Enable push all button on autoupdate
Petr Blaho
pblaho at redhat.com
Mon Mar 5 14:23:45 UTC 2012
On Monday, March 05, 2012 02:00:23 PM ifarkas at redhat.com wrote:
> From: Imre Farkas <ifarkas at redhat.com>
>
> https://bugzilla.redhat.com/show_bug.cgi?id=797208
> ---
> src/app/stylesheets/layout.scss | 5 ++++-
> src/app/views/images/show.html.haml | 13 +++++++++++--
> src/public/javascripts/backbone/views.js | 14 ++++++++++++++
> 3 files changed, 29 insertions(+), 3 deletions(-)
>
> diff --git a/src/app/stylesheets/layout.scss b/src/app/stylesheets/layout.scss
> index c58d93a..da3f9d9 100644
> --- a/src/app/stylesheets/layout.scss
> +++ b/src/app/stylesheets/layout.scss
> @@ -2920,6 +2920,10 @@ ul.catalog_pretty {
> /********************************************* Image show view */
> /************************************************************************** */
>
> +#push-all-btn {
> + display: inline;
> +}
> +
> ul.image_builds{
> border: 1px solid #ccc;
> -webkit-border-radius: 7px 7px 7px 7px; -moz-border-radius: 7px 7px 7px 7px; border-radius: 7px 7px 7px 7px;
> @@ -2972,7 +2976,6 @@ ul.image_builds{
> }
> }
>
> -
> /* ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> Flat Table Component -- v.0.0.1 [flat_table] (flat_table.scss)
> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ */
> diff --git a/src/app/views/images/show.html.haml b/src/app/views/images/show.html.haml
> index fe9faaa..38e34cf 100644
> --- a/src/app/views/images/show.html.haml
> +++ b/src/app/views/images/show.html.haml
> @@ -45,8 +45,10 @@
> - if @account_groups.size > 1
> - build_text << '_all'
> = button_to t(build_text), rebuild_all_image_path(@image.id), :class => 'button pill'
> - - if @build and @build.id == @latest_build
> - = button_to t('.push_all'), push_all_image_path(@image.id, :build_id => @build.id), :class => 'button pill'
> + #push-all-btn
> + - if @build and @build.id == @latest_build
> + = button_to t('.push_all'), push_all_image_path(@image.id, :build_id => @build.id), :class => 'button pill'
> +
> .content
> %ul.image_builds
> - @account_groups.each do |driver, group|
> @@ -257,3 +259,10 @@
> </li>
> {{/if}}
> {{/each}}
> +
> +%script#pushAllButtonTemplate{ :type => 'text/x-jquery-tmpl' }
> + :plain
> + #{path = push_all_image_path('replace_image_id', :build_id => 'replace_build_id')
> + path = path.sub('replace_image_id', '${image[\'uuid\']}')
> + path = path.sub('replace_build_id', '${build[\'uuid\']}')
> + button_to(t('images.show.push_all'), path, :class => 'button pill')}
> diff --git a/src/public/javascripts/backbone/views.js b/src/public/javascripts/backbone/views.js
> index 2d6da0d..6a1d823 100644
> --- a/src/public/javascripts/backbone/views.js
> +++ b/src/public/javascripts/backbone/views.js
> @@ -208,5 +208,19 @@ Conductor.Views.ImagesShow = Backbone.View.extend({
> $builds.empty();
>
> $('#imageBuildsTemplate').tmpl(this.model.toJSON()).appendTo($builds);
> +
> + // Enable/Disable Push All button
> + var $buttonTemplate = $('#pushAllButtonTemplate');
> + var $pushAllBtn = $('#push-all-btn');
> +
> + var buildId = this.model.get("build") ? this.model.get("build")['uuid'] : null;
> + var latestBuildId = this.model.get("latest_build_id");
> +
> + if (buildId && buildId == latestBuildId) {
> + $pushAllBtn.html($buttonTemplate.tmpl(this.model.toJSON()));
> + }
> + else {
> + $pushAllBtn.empty();
> + }
> }
> });
>
Ack.
--
With regards
Petr Blaho
More information about the aeolus-devel
mailing list