Skip to content
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

v0.9 beta work #46

Merged
merged 63 commits into from
Oct 27, 2021
Merged

v0.9 beta work #46

merged 63 commits into from
Oct 27, 2021

Conversation

hawkerm
Copy link
Owner

@hawkerm hawkerm commented Mar 30, 2020

Just staging a comparison for myself on changes I want to finish for the v0.9 beta in the 0.9 milestone.

To do:

Work done:

  • Fixes Add ColorProvider #36 Adds ColorProvider support
  • Clean-up Language usage by adding shortcut on editor
  • Fixes Clean-up CssStyleBroker to automatically track requires styles #42 Cleans-up CssStyleBroker inner workings (auto-tracks editors, optimizes and only updates WebView styles when needed, removes previously breaking change, probably should be tested more with more than one CodeEditor instance...)
  • Fixes Library NuGet package Layout
  • Clean-up ts/js file relation in solution
  • Update readme/changelog

@hawkerm hawkerm added this to the v0.9-beta milestone Mar 30, 2020
@hawkerm
Copy link
Owner Author

hawkerm commented Mar 30, 2020

Actually was just thinking if we should make this the v0.20 release? As 1) it's 2020, and 2) that's the same version of Monaco it's based off of v0.20.0? Hmm...

@hez2010
Copy link
Contributor

hez2010 commented May 20, 2020

Ping. Any updates?

@hawkerm
Copy link
Owner Author

hawkerm commented May 26, 2020

@hez2010 sorry time's disappeared on my weekends lately, work's been pretty busy. I'll try and dig into this more in a week or two and get something together.

@hez2010
Copy link
Contributor

hez2010 commented Oct 21, 2021

I'm considering how about compiling monaco-editor manually to a lower target which is compatible with the old UWP WebView (maybe some polyfills are needed) so that we can use the latest monaco editor?

@hawkerm
Copy link
Owner Author

hawkerm commented Oct 23, 2021

@hez2010 bit confused, this is still using the EdgeHTML WebView, what is your scenario you're needing?

I've already added one polyfill for shadow-dom needed for the version that was supposed to still work on EdgeHTML (but they added an incompatible feature). Though I don't know about future versions and adding more.

My plan is to release this update that works on WebView still after investigating these 3 remaining bugs. Then I want to investigate generating the interfaces to a core .NET Standard lib and targeting WebView2 across different frameworks for a 'runtime' Make it easier to write code that works on top of the Monaco APIs but can then be run by multiple types of applications. I'm starting to investigate and experiment at least in parallel on this next week.

It'd be great if we could coordinate efforts to move things forward.

@hez2010
Copy link
Contributor

hez2010 commented Oct 23, 2021

Never mind, that was just a thought about what kind of work is needed post v0.9 (before WebView2 becomes available for UWP).

My plan is to release this update that works on WebView still after investigating these 3 remaining bugs. Then I want to investigate generating the interfaces to a core .NET Standard lib and targeting WebView2 across different frameworks for a 'runtime' Make it easier to write code that works on top of the Monaco APIs but can then be run by multiple types of applications. I'm starting to investigate and experiment at least in parallel on this next week.

Sounds great.

@hawkerm
Copy link
Owner Author

hawkerm commented Oct 23, 2021

@hez2010 cool, I also misunderstood that you were talking about monaco itself. 🙂 Agree that hopefully moving to WebView2 should be the next step and unlock us following the latest Monaco development too. 🙂

Now each editor has its own options object created.
…onverter when running in .NET Native

No idea why the existing declaration on the interface isn't being used/picked-up properly and ignored by Json.NET... config issue or quirk?
@hawkerm hawkerm marked this pull request as ready for review October 26, 2021 08:06
@hawkerm
Copy link
Owner Author

hawkerm commented Oct 26, 2021

Alright, think we're finally all set! 🤞

Too late now, but next time I sit down at this again in the next day or two, I'll kick-off a full build to get a package and do a final test with those bits. Then we can merge this to main and upload to NuGet... 🚀

@hawkerm hawkerm merged commit 64c35b5 into master Oct 27, 2021
@hawkerm hawkerm deleted the 0.9-beta-prep branch October 27, 2021 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clean-up CssStyleBroker to automatically track requires styles Add ColorProvider
5 participants