Skip to content
This repository was archived by the owner on May 29, 2019. It is now read-only.

docs(readme): add clarity on angular requirement #4278

Closed
wants to merge 1 commit into from

Conversation

icfantv
Copy link
Contributor

@icfantv icfantv commented Aug 26, 2015

  • add a note indicating that due to critical animation fixes, UI Bootstrap
    0.13.3 and higher requires Angular 1.3.18 or higher.

addresses #4277

@wesleycho
Copy link
Contributor

Maybe it's worth changing the comments to highly recommend using 1.3.18? I'm an advocate of letting people play with fire if they like, especially since that is sort of what we have seen to have enough of a presence in the community.

@wesleycho wesleycho added this to the 0.13.4 (Performance) milestone Aug 26, 2015
@icfantv
Copy link
Contributor Author

icfantv commented Aug 26, 2015

I'm cool with that. @Foxandxss, I'll leave this as you're currently reworking the docs?

* add a note indicating that due to critical animation fixes, UI Bootstrap
  0.13.3 and higher requires Angular 1.3.18 or higher.

addresses angular-ui#4277

docs(readme): add clarity on angular requirement

* add note strongly recommending the use of Angular 1.3.18 or higher for UI
  Bootstrap 0.13.3 and higher due to animation fixes.

addresses angular-ui#4277
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants