Skip to content

Commit

Permalink
Implement editing, find/replace in streaming model (#316)
Browse files Browse the repository at this point in the history
* wip

* fixup, get replacement editing working

* unsaved ranges and go to offset

* chore: remove dead code

* feat: implement virtual scroll

* wip

* refactor: add comments and clean up math

* find/replace

* feat: undo/redo

* fix: focus issues with go to offset quickpick

* fix: focus back to the byte when hiding find widget

* fix: binary in search, cap results by default for perf, progress

* fix: don't change selected toolbar background on hover

* fix: issues with length changes and large find/replace

* feat: add replace toggle

* feat: show better validation for find widget errors

* feat: wip on string search with placeholders

This approach will not work, it's wildcards and not placeholders. But committing it if we need it in the future...

* feat: support placeholders in search

* feat: persist webview state information

* fix: make revert work
  • Loading branch information
connor4312 authored Nov 29, 2021
1 parent e7b8e35 commit f91ed84
Show file tree
Hide file tree
Showing 47 changed files with 2,930 additions and 2,517 deletions.
2 changes: 2 additions & 0 deletions .esbuild.config.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
const esbuild = require('esbuild');
const svgr = require('esbuild-plugin-svgr');
const linaria = require('@linaria/esbuild');

const watch = process.argv.includes('--watch');
Expand Down Expand Up @@ -67,6 +68,7 @@ esbuild.build({
platform: 'browser',
outfile: 'dist/editor.js',
plugins: [
svgr(),
linaria.default({ sourceMap: watch }),
],
}).catch(() => process.exit(1))
2 changes: 1 addition & 1 deletion .vscode/launch.json
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@
"runtimeExecutable": "${execPath}",
"args": [
"--extensionDevelopmentPath=${workspaceFolder}",
"--extensionTestsPath=${workspaceFolder}/dist/test"
"--extensionTestsPath=${workspaceFolder}/dist/test.js"
],
"outFiles": [
"${workspaceFolder}/dist/**/*.js"
Expand Down
59 changes: 0 additions & 59 deletions media/editor/byteData.ts

This file was deleted.

172 changes: 0 additions & 172 deletions media/editor/chunkHandler.ts

This file was deleted.

Loading

0 comments on commit f91ed84

Please sign in to comment.