Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Initial CanvasController refactor #707

Closed
n8rzz opened this issue Sep 18, 2017 · 0 comments
Closed

Initial CanvasController refactor #707

n8rzz opened this issue Sep 18, 2017 · 0 comments
Assignees
@n8rzz
Labels
enhancement Notable improvement or expansion upon an EXISTING feature refactor Code changes that neither fix a defect nor add functionality
Milestone
v5.5.0

Comments

@n8rzz
Copy link
Member

n8rzz commented Sep 18, 2017 •
edited by erikquinn

this is a precursor to existing issue #236. There are several 'quick wins' that we can do to increase performance.

  • refactor:
    • $.each()
    • _forEach()
  • consolidate math operations
  • move some logic out of render cycle
  • create separate canvases for moving elements (aircraft) and non-moving elements (everything else)
@n8rzz n8rzz added enhancement Notable improvement or expansion upon an EXISTING feature refactor Code changes that neither fix a defect nor add functionality labels Sep 18, 2017
@n8rzz n8rzz added this to the v5.5.0 milestone Sep 18, 2017
@n8rzz n8rzz self-assigned this Sep 18, 2017
@n8rzz n8rzz mentioned this issue Sep 19, 2017
Merged
@erikquinn erikquinn added the has pull request Add this label when an issue has a PR which will resolve it label Sep 20, 2017
@n8rzz n8rzz mentioned this issue Sep 25, 2017
Closed
@n8rzz n8rzz removed the has pull request Add this label when an issue has a PR which will resolve it label Sep 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@n8rzz n8rzz

Labels
enhancement Notable improvement or expansion upon an EXISTING feature refactor Code changes that neither fix a defect nor add functionality
Projects
None yet
Milestone
v5.5.0
Development

No branches or pull requests
2 participants
@n8rzz @erikquinn