[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