[PATCH conductor] Fix the filter view toggle after converge-ui layout changes

Matt Wagner matt.wagner at redhat.com
Thu Jun 14 20:49:10 UTC 2012


On Wed, Jun 13, 2012 at 05:21:23PM +0200, ifarkas at redhat.com wrote:
> From: Imre Farkas <ifarkas at redhat.com>
> 
> ---
>  src/public/javascripts/application.js |    8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/src/public/javascripts/application.js b/src/public/javascripts/application.js
> index bfaaec2..2162880 100644
> --- a/src/public/javascripts/application.js
> +++ b/src/public/javascripts/application.js
> @@ -82,12 +82,12 @@ $.extend(Conductor, {
>      });
>    },
>  
> -  bind_pretty_toggle: function() {
> +  bindPrettyToggle: function() {
>      Conductor.nicelyHookAjaxClick($("#pretty_view"), function() {
>        var link_element = this;
>        $('#content .toggle-view').html('<div class="loading_tabs"></div>');
>        $.get($(this).attr("href"), $(this).serialize(), function(result) {
> -        $('#content .toggle-view').html(result);
> +        $('.toggle-view').html(result);
>          $(link_element).addClass('active');
>          $("#filter_view").removeClass('active');
>        });
> @@ -96,7 +96,7 @@ $.extend(Conductor, {
>        var link_element = this;
>        $('#content .toggle-view').html('<div class="loading_tabs"></div>');
>        $.get($(this).attr("href"), $(this).serialize(), function(result) {
> -        $('#content .toggle-view').html(result);
> +        $('.toggle-view').html(result);
>          $('#details-selected').hide();
>          $('#details-view').tabs();
>          $(link_element).addClass('active');
> @@ -310,7 +310,7 @@ $(document).ready(function () {
>    $("#notification").enhanceInteraction();
>    Conductor.enhanceListView();
>    Conductor.enhanceDetailsTabs();
> -  Conductor.bind_pretty_toggle();
> +  Conductor.bindPrettyToggle();
>    Conductor.multiActionValidation();
>    Conductor.closeNotification();
>    Conductor.toggleCollapsible();
> -- 
> 1.7.10.2

ACK, fixes the issue.

I touched up the commit message to include the BZ number per your other
comment, and pushed to master.

-- Matt



More information about the aeolus-devel mailing list