-
Notifications
You must be signed in to change notification settings - Fork 30.3k
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
Notebook exploration #86632
Merged
Merged
Notebook exploration #86632
Changes from 250 commits
Commits
Show all changes
319 commits
Select commit
Hold shift + click to select a range
55d2140
dirty state update
rebornix 77b395f
Update notebook test extension for new API.
rebornix 7d47821
default langauges
rebornix 42b45bb
Delete cell
rebornix a0d8ac1
jpeg rendering
rebornix 0385552
Update line breaks
rebornix 7d2c56a
Editor resize with view line change
rebornix 7100934
css update for markdown
rebornix 3cfaf1b
Support markdown in output
rebornix ed12c87
Typing latency: avoid unncessary dirty change (leads to 80ms per typi…
rebornix f4938e3
destory notebook should clear both core and ext host.
rebornix dcc4ed5
Merge remote-tracking branch 'origin/master' into rebornix/notebook
rebornix 38dfdc9
Edit cell with enter
rebornix 9375c9a
Focus cell
rebornix eb43b41
Esacpe a markdown cell should exit preview mode
rebornix 97fd062
Uri.file for cross os.
rebornix 1d71d49
Merge remote-tracking branch 'origin/master' into rebornix/notebook
rebornix f2e0c33
Adopt content size event from monaco editor.
rebornix 86fd51d
event listener for statefull markdown
rebornix ddb1795
move markdown cell renderer out
rebornix 330f6f7
CellViewModel
rebornix 3368403
Cell renderer with right disposals
rebornix 7445e35
Code refactoring
rebornix 1fea3ae
💄
rebornix 8a77280
💄
rebornix 700dfa7
setter/getter for dynamic height
rebornix f46d2fc
doc update for typings
rebornix c78a869
Merge remote-tracking branch 'origin/master' into rebornix/notebook
rebornix 50751af
Remove initial dimension update
rebornix 90ab42e
remove script tag check in webview
rebornix 88fd5ab
update scroll handler
rebornix 54a5a6e
application/javascript
rebornix 6bbdfe0
svg
rebornix f53b0eb
json editor
rebornix 2790c0b
Merge remote-tracking branch 'origin/master' into rebornix/notebook
rebornix aec1335
Refactoring and latex support.
rebornix df477bc
contributed output renderer first cut
rebornix 64a3d6c
Nteract renderers.
rebornix 612a935
Merge remote-tracking branch 'origin/master' into rebornix/notebook
rebornix 7d12ce5
list view set row height option
rebornix 298a5d0
Move typings to common.
rebornix 20f4cbf
Update preload scripts for notebook renderer
rebornix 2d99b7d
output chagne event
rebornix 7faf159
nteract renderer
rebornix 1adc63c
Refactoring: IoC.
rebornix 7de343c
Move notebookActions to separate module.
rebornix 6f8fd62
Errorless
rebornix 81cfb89
public by default
rebornix 55d34d7
No ambiguous naming.
rebornix 7e74dc8
handler -> notebookEditor
rebornix 059527f
add ICellDto, use UriComponents
jrieken 6395ca1
use registerAction2-todo
jrieken ebb2e25
add CellContentProvider, more injection
jrieken ae304b3
authority is not case sensitive.
rebornix ac800b7
Update notebook cells
rebornix 1417090
extract code cell updates
rebornix 49ebc84
No latex handler in core.
rebornix 7f28579
💄
rebornix a88978b
Display Order first cut.
rebornix 18cadef
check for scheme when parsing cell-uri
jrieken fc46070
towards code navigation...
jrieken 6db3625
add ICell#uri and expose INotebook#viewType
jrieken 25fc954
add IEditorNavigation, remove custom ICodeEditorService because that …
jrieken 796c2ba
Diff for cells and outputs in exthost.
rebornix 4bced15
notebook cell update with renderer updates.
rebornix c46f92e
Cell insertion/deletion in notebook editor list view.
rebornix e485c82
No unused args.
rebornix a4e9073
registerAction2
rebornix de5e408
🌩static height code cell with relayout.
rebornix 2db9e68
unused import
jrieken 1541d57
Merge branch 'master' into rebornix/notebook
jrieken 6bc2ba7
adopt workbench changes
jrieken 315b240
use ICompositeCodeEditor instead IEditorNavigation
jrieken 594dd0b
command handler honor composite code editor
jrieken a8d8035
forward list events as editor change events
jrieken bc4b343
don't check for composite code editor in commands handler
jrieken 5c32f22
forward active (nested) editor event from composite code editors, mak…
jrieken 673d983
Merge remote-tracking branch 'origin' into rebornix/notebook
rebornix 604cee1
size/offset trackers for individual output
rebornix afd83b5
ignore empty splice events.
rebornix 0c3e12a
remove webview change.
rebornix 6f88d16
Merge remote-tracking branch 'origin/master' into rebornix/notebook
rebornix fc97748
No webview workarounds.
rebornix 0fcff50
wire notebook editor restore logic but keep it disabled due to notebo…
jrieken 24a5860
revert markdown change
rebornix fb12bf9
Merge branch 'master' into rebornix/notebook
rebornix f524bea
Merge remote-tracking branch 'origin/master' into rebornix/notebook
rebornix 5a21f02
Merge branch 'master' into rebornix/notebook
rebornix 33f008e
adopt scrollable element event
rebornix 3d53af1
move more notebook list logic out
rebornix 83a30d7
more list
rebornix 6579aba
document why domFocus in notebook is different from normal list
rebornix 61c5456
Merge branch 'master' into rebornix/notebook
rebornix 679a008
empty line
rebornix cc14e23
typings for notebook settings.
rebornix 88fe192
fix file saving
rebornix 0ec2804
we still need input checkbox
rebornix 7ff6e76
Fix enter keybinding for notebook list
rebornix b01035a
Focus still not working actually.
rebornix 62f68fe
hide console log
rebornix ff5e746
activate extension by notebook view type
rebornix b5804fa
Updating cell state when destorying it is not good
rebornix 33fe974
make sure view cells are disposed
jrieken ee7bb4f
Merge branch 'master' into rebornix/notebook
jrieken 214e4df
remove commented type
jrieken d911177
Focus editor when creating new cells
rebornix 09a8143
No additional event triggering for crud from core.
rebornix d00f117
Mimetype picker first cut.
rebornix bb8216b
some padding for output
rebornix 01426ff
resizeobserver does not include padding.
rebornix ab1a981
Tokenization.
rebornix 8e6f63b
regex obj is statefull ;(
rebornix 22ffffb
disposables
rebornix c9789ee
spacing around the editor
rebornix f5ad712
fix transformed output rendering for nteract.
rebornix 298461e
cell layout
rebornix cef3c06
find widget attempt
rebornix dc0ed1a
Update element height
rebornix b42d1d5
remove unused list hack
rebornix 1ff083c
container dom node
rebornix 3856d31
no scroll height update hack
rebornix 352bbf2
list view update height refactoring
rebornix ae075e9
public is default
rebornix ad43402
we don't need dynamic height change anymore
rebornix fa0fef8
isRendering is not needed if we push everything to next frame
rebornix f3780d9
Merge remote-tracking branch 'origin' into rebornix/notebook
rebornix 76e1c59
no isrendering
rebornix 8dbf9b2
Find Widget typings
rebornix 1dd89fc
Merge remote-tracking branch 'origin/rebornix/listview-item-heightcha…
rebornix 9502753
update update element height callers
rebornix e0107f4
not test folder in core anymore
rebornix eed9a7b
Merge remote-tracking branch 'origin' into rebornix/notebook
rebornix 285c61d
add renderer extension folder to webview allowed resource list
rebornix 3f4fccc
Merge remote-tracking branch 'origin' into rebornix/notebook
rebornix 572dd51
support picking renderers for the same mimetype
rebornix ee27322
Reduce scrolling position updates
rebornix e0b9c76
Fix markdown editing height update.
rebornix 209c2e7
List Output resize flashing.
rebornix a11145e
update mimetype renderer
rebornix fe67100
Update code cell mimetype picker rendering
rebornix 110c35e
update picked item in quick pick
rebornix 66bdaba
TODO for display order
rebornix a03546f
scroll webview container instead of scrolling the content inside.
rebornix cddc990
Merge branch 'master' into rebornix/notebook
jrieken 93ad98c
adopt API changes in notebook land
jrieken 6cd184e
no composite code editor knowledge in EditorService (but still in Cod…
jrieken 764b2ae
Add/Delete cell with right model update.
rebornix c5f0808
Fix folding issue.
rebornix 9887d9b
NotebookViewModel and tests
rebornix 882ae1e
no unverified commit
rebornix a3f5a85
fix cell editing
rebornix 1a63d4d
more typings error after adopting const enum
rebornix eec843a
Merge branch 'master' into rebornix/notebook
jrieken fc10037
Merge branch 'master' into rebornix/notebook
jrieken 1d4da40
adopt list API changes
jrieken 76eb17e
layer check
rebornix 93a642b
two more layer break
rebornix c4c3fdc
tests for mimetype ordering
rebornix e8c3f85
unit tests for diffing
rebornix 8cc3a0d
fix glob pattern matching on windows
rebornix 568a529
no glob pattern pre parsing
rebornix 0192291
disposables
rebornix effab81
Fix finding active notebook editor
rebornix 285d89e
Merge remote-tracking branch 'origin/master' into rebornix/notebook
rebornix 2451b72
Update webview
rebornix a4a17ae
store view states for cells
rebornix ad22aa2
save & restore all editor view states
rebornix fda49c0
Merge remote-tracking branch 'origin/master' into rebornix/notebook
rebornix 2af6cc1
npe
rebornix 3c2ef65
Fix #91498.
rebornix c0b268b
EditorService#activeTextEditorControl knows about composite code editor
jrieken e5ebccc
better folder organization
rebornix 8f081ef
lazy decoration for cells.
rebornix 4de61f3
use uuid instead of handle as decoration prefix
rebornix b5473f5
reveal cell into view.
rebornix 43ee60f
Test editor
rebornix 7e7e5d9
fix css import
jrieken 4adf8bf
fix double model issue
jrieken 82ecf89
use `overrideOpenEditor` to open a cell-uri as notebook editor, use N…
jrieken e933381
fix cycling dependency, more rich cell-options
jrieken c402304
dispose view model properly
rebornix 7457754
active find match
rebornix 69d2a45
reveal function call
rebornix d8df542
track newly created view cell in notebookviewmodel
rebornix 40b4199
reveal line into view center
rebornix 2be2c20
:lipstick:
rebornix 8aac73b
abstract notebook editor layout info
rebornix 18b4045
remove unused typings
rebornix 1387211
document INotebookEditor.
rebornix 58140d3
scroll line into view
rebornix 52ea4ff
Merge branch 'master' into rebornix/notebook
jrieken a8f61a8
remove ICompositeCodeEditor#activate
jrieken 98c180c
add Iterable.filter and use it
jrieken 6ed0201
first reveal a cell, then check for its editor
jrieken c1ec410
create cell uri on the extension host, expose as `NotebookCell#uri`
jrieken 990cc85
add CellUri-util but don't use it yet
jrieken ae4a4f8
event once for cell editor opening
rebornix d1354bb
adopt CellUri
jrieken 540fbab
fix resources#isEqual
jrieken 0c1eb03
Revert "fix resources#isEqual"
jrieken 922ff01
don't use isEqual for now
jrieken d4385d7
consolidate reveal range/line
rebornix c6c0385
reveal search result in center in markdown cell
rebornix e17815a
set selection for cell
rebornix 4c58ef4
test notebook editor
rebornix a036618
Add ctrl+enter to execute cell and go to next
roblourens e814a19
Add option+enter and shift+enter commands
roblourens 3f3dcf1
Add commands to convert cell to markdown/code
roblourens a9b5966
Merge pull request #92404 from microsoft/roblou/focusNextCellAfterExe…
roblourens e8dae6d
do not allow attach editor without model
rebornix 4aa13a5
find in markdown content
rebornix 35b5132
:lipstick:
rebornix a6af569
Add a toolbar with some basic actions
roblourens ce863ce
Add alt actions to some toolbar items
roblourens ecb6fc3
Fix merge mistake
roblourens f48e190
cursor movement first cut
rebornix 24c6547
cell state instead of isEditing boolean
rebornix bc94d62
update state and editing mode when focus to the container
rebornix 0faba33
ICellViewModel
rebornix befaab5
ICellViewModel only visible to contrib
rebornix 6cc118c
Find & Replace Widget
rebornix 508dead
list view resize: anchored element will keep its visual position
rebornix 589b523
Scroll cell into view major update:
rebornix 9675f47
NPE check find
rebornix e6fd2cf
move output height calculation internal
rebornix 5798020
Merge branch 'rebornix/notebook' into roblou/addToolbar
rebornix 635c2f2
toolbar height
rebornix 4996dcd
Merge pull request #92593 from microsoft/roblou/addToolbar
rebornix c41d631
move cell padding height to constants
rebornix d256d56
replace first cut
rebornix cdfd6ec
Merge remote-tracking branch 'origin/master' into rebornix/notebook
rebornix e66dc83
Undo/Redo for cell manipulation
rebornix 5f4f23f
Fix test errors
rebornix c7eff5b
progress bar for find and replace
rebornix 9b235ba
scrollTop should never be negative
rebornix 2c92a70
Use container height for markdown cell, consider separate MarkdownVie…
rebornix b8840e4
revert viewlines change..
rebornix d9b9cfe
Reveal range after editor is initialized.
rebornix 8e72cca
Persist cell data when delete cell content.
rebornix 9bb2c9d
TextModel and CellTextMode. Separate dirty state from text model.
rebornix 8c81d9a
move ICompositeCodeEditor to editor-layer
jrieken 542818b
Fix cursor movement not revealing cell into view
rebornix 915d3da
create new cell should be stateless
rebornix bef8944
:lipstick:
rebornix 58afb2b
:lipstick:
rebornix 5194415
Merge remote-tracking branch 'origin/master' into rebornix/notebook
rebornix 164c169
remove markdown sanity check change
rebornix fcf9d81
remove find controller reveal change
rebornix d5ef90d
revert simple find widget change
rebornix 0054a89
Merge branch 'master' into rebornix/notebook
rebornix File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is risky, right? Don't we use this function to also render extension contributed markdown and thereby also allowing them to render html?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the remind. This one is about allowing the core markdown renderer to display
checkbox
which is sanitized byinsane
. I'll revert the change for now and see ppl's feedback.