[PATCH] BZ 790824 hide build select unless build exists

Matt Wagner matt.wagner at redhat.com
Mon Feb 20 21:15:13 UTC 2012


On Mon, Feb 20, 2012 at 03:33:11PM -0500, Tzu-Mainn Chen wrote:
> ---
>  src/app/views/images/show.html.haml |   13 +++++++------
>  1 files changed, 7 insertions(+), 6 deletions(-)
> 
> diff --git a/src/app/views/images/show.html.haml b/src/app/views/images/show.html.haml
> index c4ad8d7..edcd57a 100644
> --- a/src/app/views/images/show.html.haml
> +++ b/src/app/views/images/show.html.haml
> @@ -19,10 +19,11 @@
>      %h3= t('images.environment', :environment => @image.environment)
>      %h2= t('.provider_images')
>      .section-controls
> -      = t'.view_build'
> -      = form_tag image_path(@image.id), :method => :get do
> -        = select_tag :build, options_for_build_select(@builds, @build, @latest_build)
> -        = submit_tag t('.select_build'), :id => 'seletect_build_button'
> +      - if @builds.any?
> +        = t'.view_build'
> +        = form_tag image_path(@image.id), :method => :get do
> +          = select_tag :build, options_for_build_select(@builds, @build, @latest_build)
> +          = submit_tag t('.select_build'), :id => 'select_build_button'
>        - if check_privilege(Privilege::USE, PoolFamily) and @environment
>          - if @image.imported?
>            = t('.can_not_build_imported_image')
> @@ -96,9 +97,9 @@
>  
>  :javascript
>    $(document).ready(function(){
> -    $("#seletect_build_button").hide();
> +    $("#select_build_button").hide();
>      $("#build").change(function() {
> -      $("#seletect_build_button").click();
> +      $("#select_build_button").click();
>      });
>    });
>  
> -- 
> 1.7.6.5

ACK. And thanks for fixing the "seletect" typo while you were in there.
:)

-- Matt



More information about the aeolus-devel mailing list