OrchardCore icon indicating copy to clipboard operation
OrchardCore copied to clipboard

POC Blazor Component

Open Skrypt opened this issue 1 year ago • 9 comments

I'm getting a lot of requests around Blazor components recently since .NET 8 is out.

We need to create a POC and for that matter we could replace a Vue component that we use in the Admin UI. Refactoring the OptionEditor to a Blazor component would be a good starting point to see how it would integrate as part of OC.

We need to evaluate the maintainability of these components. We also need to see if we could integrate a more complex component like the Media Library, Media Library picker or even the CORS UI eventually.

@ns8482e

Skrypt avatar Jan 11 '24 20:01 Skrypt

It was one of those ideas that I was thinking of years ago, @agriffard did some Blazor integration with OC. If someone is interested he can show us a demo in the Orchard Core meeting, or I might do it after finishing plenty of my pending PRs :)

hishamco avatar Jan 11 '24 20:01 hishamco

See https://github.com/OrchardCMS/OrchardCore/discussions/14722 for a discussion about using OC in a Blazor .NET 8 web app.

agriffard avatar Jan 11 '24 20:01 agriffard

Cool thanks, we need a POC pull request that will be using source code now to see how we will integrate it. I'm guessing we can take ideas from the Nuget package solution but we also need to think on the OC structure if we start adding these.

Skrypt avatar Jan 11 '24 20:01 Skrypt

Adding how-to-guide will be helpful.

ns8482e avatar Jan 22 '24 19:01 ns8482e

@ns8482e Would you have time to do something? I can't really move away from the Vue3 Pull Request much.

Skrypt avatar Jan 22 '24 21:01 Skrypt

@Skrypt created POC at ns/blazor-poc branch

  • Created Option Editor and Modal Dialog and replaced Vue option editor with optioneditor component for predefinedlist

Prerequisite PR #15247

ns8482e avatar Feb 15 '24 14:02 ns8482e

Hey @ns8482e. Commented on your POC branch: please make your intention more clear.

soma-kurisu avatar Feb 17 '24 17:02 soma-kurisu

Thank for pointing out typo mistake. It should be collocated.

ns8482e avatar Feb 17 '24 17:02 ns8482e

@ns8482e have a look on PR #15354. I think we should take some safety measurements.

soma-kurisu avatar Feb 17 '24 19:02 soma-kurisu