Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

make xhr.cache optionally synchronous #318

Closed
wants to merge 1 commit into from

Conversation

IgorMinar
Copy link
Contributor

  • add sync flag xhr.cache
  • change ng:include to use the sync flag
  • change ng:view to use the sync flag

The end result is that there are fewer repaints in the browser,
which means less "blinking" that user sees.

- add `sync` flag xhr.cache
- change ng:include to use the sync flag
- change ng:view to use the sync flag

The end result is that there are fewer repaints in the browser,
which means less "blinking" that user sees.
@IgorMinar
Copy link
Contributor Author

angular.js: master Igor Minar * c06c5a3 (4 files in 4 dirs): make xhr.cache optionally synchronous ...

@IgorMinar IgorMinar closed this Mar 30, 2011
@guillaume86
Copy link

Do you guys have a solution for the performance hit of ngInclude because of this?
I use ngInclude inside big tables (one include by cell, 40*15 cells more or less) and I get +1s degradation in chrome on a i7 3635QM when using ngInclude (chrome timeline shows (start timer/end timer/parseHTML) * (cell count).

As a workaround I use a ng-include-cached directive, but it would be nice to have an official solution.
Include could have another level of cache with the compiled template.

@guillaume86
Copy link

Does the comments on closed stuff get read by anyone?

@caitp
Copy link
Contributor

caitp commented Feb 20, 2014

Yes, they get do

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants