Jakub Mikulas

Results 9 comments of Jakub Mikulas

Hey, I was thinking about it, and for me it seems that the most natural implementation would be to be able to initialize vuex for each scenario like this: ```js...

Yea, had a local branch where I was playing with it, but word diff is really good when you are just reading it. Once you start editing it, you get...

Could you elaborate on the use case? I'm thinking what's the scenario here and whether do you need to show them AND edit them? Or merge them? Thinking that adding...

That's not the error I'm seeing on the link you've shared: Don't see ace-builds being included there. If you include it (globally 2nd line or with `options`), it seems to...

Yup, was also thinking about collapsing unchanged parts (~like GitHub does it with PRs). I think that it will have 2 parts: 1. add a public API to scroll changes...

Yea, the change happened here https://github.com/JackuB/diff-match-patch/pull/13/files#diff-168726dbe96b3ce427e7fedce31bb0bcR76 It's emulating array for backward compatibility. But the destructing use case is not taken care of. 1. We could add Symbol.iterator to the `Diff`...

Released 1.0.4 with a rollback https://github.com/JackuB/diff-match-patch/releases/tag/1.0.4 Will hold on with releasing v2 as there is a chance they will rollback the API change

Seeing there is an `'extra' does not exist in evaluation environment` in the stack trace. Might be related to this issue: https://github.com/getsentry/sentry/issues/3143 and a setuptools version?

You could create this logic on your own using [`import()/require.ensure`](https://webpack.js.org/guides/code-splitting/) and then change Ace mode in callback. ```js // pseudocode solution select.on('change', () => { import('brace/mode/javascript').then(() => editor.getSession().setMode('ace/mode/javascript')) }) ```...