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.

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

[WIP] Infinite canvas #3623

Merged
merged 9 commits into from
Apr 4, 2018
Merged

[WIP] Infinite canvas #3623

merged 9 commits into from
Apr 4, 2018

Conversation

fjakobs
Copy link
Contributor

@fjakobs fjakobs commented Mar 23, 2018

Use an infinite canvas div to render lines. Offloading scrolling almost entirely to the GPU.

@fjakobs fjakobs changed the title Infinite canvas [WIP] Infinite canvas Mar 23, 2018
@codecov
Copy link

codecov bot commented Mar 24, 2018

Codecov Report

Merging #3623 into master will increase coverage by 1.59%.
The diff coverage is 73.76%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3623      +/-   ##
==========================================
+ Coverage   64.82%   66.41%   +1.59%     
==========================================
  Files         471      472       +1     
  Lines       47303    47451     +148     
  Branches     9023     9040      +17     
==========================================
+ Hits        30663    31515     +852     
+ Misses      16640    15936     -704
Impacted Files Coverage Δ
lib/ace/edit_session.js 81.84% <ø> (+1.03%) ⬆️
lib/ace/mouse/default_handlers.js 52.38% <ø> (+3.16%) ⬆️
lib/ace/renderloop.js 100% <ø> (+4.76%) ⬆️
lib/ace/scrollbar.js 81.31% <ø> (+16.48%) ⬆️
lib/ace/keyboard/vim_test.js 95.94% <100%> (ø) ⬆️
lib/ace/mouse/multi_select_handler.js 78.99% <100%> (+64.58%) ⬆️
lib/ace/mouse/mouse_handler_test.js 98.52% <100%> (+2.37%) ⬆️
lib/ace/virtual_renderer_test.js 97.93% <100%> (+3.25%) ⬆️
lib/ace/lib/fixoldbrowsers.js 66.66% <33.33%> (-33.34%) ⬇️
lib/ace/layer/text.js 69.03% <54.28%> (+16.94%) ⬆️
... and 31 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 77b932d...8f61adc. Read the comment docs.

@lennartcl
Copy link
Contributor

Please look into code coverage for this one:

Merging #3623 into master will decrease coverage by 37.58%.

@fjakobs fjakobs force-pushed the infinite-canvas branch 7 times, most recently from 09ae2c8 to 3ba0b2c Compare March 28, 2018 11:52
@nightwing nightwing force-pushed the infinite-canvas branch 2 times, most recently from 7627883 to 3c6bc8e Compare April 1, 2018 08:23
position: absolute;
top: 0;
left: 0;
right: 0;
}

.ace_text-layer .ace_line_group .ace_line {
position: relative;
@media (-webkit-min-device-pixel-ratio: 1.5), (min-resolution: 1.5dppx) {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this mean we only get hw acceleration on retina screens?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, do we need to enable it on mac even if screen is not retina?

Copy link
Contributor Author

@fjakobs fjakobs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is ready to merge

@nightwing nightwing merged commit 1f8333b into master Apr 4, 2018
@nightwing nightwing deleted the infinite-canvas branch April 4, 2018 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants