Giter VIP home page Giter VIP logo

Comments (6)

raphlinus avatar raphlinus commented on June 20, 2024

This is known, but it's good to have a bug for it. The underlying problem is that opening a new tab is not sending a scroll RPC to the core, letting the core know the window size.

This might be because viewDidAppear is not getting called, or it's possible that there's an ordering problem so it's being called before a valid scroll notification can be sent.

from xi-mac.

raphlinus avatar raphlinus commented on June 20, 2024

A looked into this a little, and found that repro was inconsistent (it was particularly hard to repro when I was looking closely). Also, a quick scan of the code shows that it's trying to do the right thing - if the RPC isn't set up, the scroll request goes into a pending queue. So there's something deeper going on here.

from xi-mac.

cmyr avatar cmyr commented on June 20, 2024

Just realized what the bug is, here while working on something related:

When we open in an existing tab, the logic that does the initial line fetch doesn't happen because it is called from the view's viewDidAppear, which happens when we first open the tab but before we open the file.

Not sure what the best solution is, but should be trivial.

from xi-mac.

cmyr avatar cmyr commented on June 20, 2024

@lennylxx Would you like to check and see if you can still reproduce this on master? I believe it's fixed, but I had trouble reproducing consistently.

from xi-mac.

lennylxx avatar lennylxx commented on June 20, 2024

@cmyr Build on top of 4603667 and xi-editor/xi-editor@53f4267, it can't be reproduced.
I think it's fixed. Thank you for the work!

from xi-mac.

cmyr avatar cmyr commented on June 20, 2024

@lennylxx great, thanks for the follow-up!

from xi-mac.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.