Skip to content

Conversation

@oyepez003
Copy link
Contributor

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? n/a
Fixed tickets n/a
License MIT
Doc PR n/a

The change removes the animation in the "sf-toolbar-block-ajax" when the request state was "ok".

…-ajax

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | n/a
| Fixed tickets | n/a
| License       | MIT
| Doc PR        | n/a

The change removes the animation in the "sf-toolbar-block-ajax" when the request state was "ok".
@javiereguiluz
Copy link
Member

👍 I've just tested this code and it solves the mentioned issue. Thanks @oyepez003.

@fabpot
Copy link
Member

fabpot commented Sep 9, 2015

Shouldn't it be fixed in 2.7?

@oyepez003
Copy link
Contributor Author

This issue only happens in sf 2.8 (the new toolbar), for the current toolbar ( in sf 2.7.* ) the "highlight" animation works fine.

@fabpot
Copy link
Member

fabpot commented Sep 9, 2015

Thank you @oyepez003.

@fabpot fabpot merged commit 8497ad7 into symfony:2.8 Sep 9, 2015
fabpot added a commit that referenced this pull request Sep 9, 2015
…block-ajax (oyepez003)

This PR was merged into the 2.8 branch.

Discussion
----------

[WebProfilerBundle ] Removes ajax animation in sf-toolbar-block-ajax

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | n/a
| Fixed tickets | n/a
| License       | MIT
| Doc PR        | n/a

The change removes the animation in the "sf-toolbar-block-ajax" when the request state was "ok".

Commits
-------

8497ad7 [WebProfilerBundle ] Removes the ajax animation in | sf-toolbar-block-ajax
@oyepez003 oyepez003 deleted the oyepez003-WebProfilerBundle-ajax-animaion branch September 9, 2015 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants