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
Replacing fields results in an invalid tree #78
Comments
Lines 412 to 415 in b5ec298
I'm working on this one, and I'm not sure why do you think it is not feasible ? Line 48 in b5ec298
Was there a reason that you put an immutable array inside Json::Field ?
Is this sort of related to #11 |
Oh hang on - I'm also working on this 😅 - at least I'm doing some more huge refactoring, which will eventually fix this. The thing that made me say this might be hard is that just fixing it for JSON objects is not really enough - I'd like to actually fix the problem that causes this. Unless I've missed something? - it's entirely possible that I'm making a mountain out of a molehill here.
Yes - it's needed because
Perhaps? I don't think it has that much connection. |
Looking forward to your refactoring. Kind of want to get a master degree in CS. This is so fun. |
Go for it! I've no idea how the US university(/college?) system works, but it must be possible to do another degree. Fair warning, though - degree level CS is just as much about learning logic and maths as it is about programming - but knowing the theory is incredibly useful, even if learning it is less fun than just writing code all the time 😆. |
Actually I'm close to get a master degree in math. I learned some algorithm, and data structure from math department, but I have almost forgotten all of them. I definitely will get a degree in CS in near future. |
Oh cool! I also enjoyed the course we had on algorithms and datastructures - until I got to an exam and discovered that the exams were just really really hard 😆. But I think that was just the person who wrote the exam being overoptimistic about how good the students were. |
|
Absolutely 😆! This wasn't the worst module, actually - we had one exam where one of the examiner's reports said something along the lines of 'even compared to other years, this group did badly'... |
Haha, actually, it was not you guys did badly in exam. It was the Prof didn't pay much attention in teaching. |
That's often true, I'll admit. But I think this was the opposite - the lecturer put so much effort into teaching that he expected the students to do really well in his crazily hard exam... |
Suppose that the cursor is over a field in an object (cursor in
<>
):{ <"key": null> }
If we type
rt
at this point will result in the following invalid object:{ <true> }
AFAIK, this is not an easy fix (at least, no easy fix that actually solves the cause of the problem). The main issue is that objects implicitly create fields to contain their children, a very nice UX feature that causes some jank with tree validation.
The text was updated successfully, but these errors were encountered: