bug symfony#17283 [WebProfilerBundle] Remove loading status from AJAX toolbar after error (kucharovic)

fabpot · fabpot · commit ba75435a2175 · 2016-01-07T14:22:28.000+01:00
This PR was submitted for the master branch but it was merged into the 2.8 branch instead (closes symfony#17283). Discussion ---------- [WebProfilerBundle] Remove loading status from AJAX toolbar after error Q | A ---|--- Bug fix? | no New feature? | no BC breaks? | no Deprecations? | no Tests pass? | yes Fixed tickets | License | MIT Doc PR | Commits ------- 079ab73 [WebProfilerBundle] Remove loading status from AJAX toolbar after error
diff --git a/src/Symfony/Bundle/WebProfilerBundle/Resources/views/Profiler/base_js.html.twig b/src/Symfony/Bundle/WebProfilerBundle/Resources/views/Profiler/base_js.html.twig
@@ -188,6 +188,7 @@
                     Sfjs.removeClass(ajaxToolbarPanel, 'sf-toolbar-status-red');
                 } else if (state == 'error') {
                     Sfjs.addClass(ajaxToolbarPanel, 'sf-toolbar-status-red');
+                    Sfjs.removeClass(ajaxToolbarPanel, 'sf-ajax-request-loading');
                 } else {
                     Sfjs.addClass(ajaxToolbarPanel, 'sf-ajax-request-loading');
                 }

Original file line number	Diff line number	Diff line change
`@@ -188,6 +188,7 @@`
`188`	`188`	`Sfjs.removeClass(ajaxToolbarPanel, 'sf-toolbar-status-red');`
`189`	`189`	`} else if (state == 'error') {`
`190`	`190`	`Sfjs.addClass(ajaxToolbarPanel, 'sf-toolbar-status-red');`
	`191`	`+ Sfjs.removeClass(ajaxToolbarPanel, 'sf-ajax-request-loading');`
`191`	`192`	`} else {`
`192`	`193`	`Sfjs.addClass(ajaxToolbarPanel, 'sf-ajax-request-loading');`
`193`	`194`	`}`