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

Perform a pass cleaning up the web templates #30162

Closed
pranavkm opened this issue Feb 12, 2021 · 9 comments
Closed

Perform a pass cleaning up the web templates #30162

pranavkm opened this issue Feb 12, 2021 · 9 comments
Labels
affected-all This issue impacts all the customers area-blazor Includes: Blazor, Razor Components enhancement This issue represents an ask for new feature or an enhancement to an existing one feature-templates Priority:1 Work that is critical for the release, but we could probably ship without severity-minor This label is used by an internal tool

Comments

@pranavkm
Copy link
Contributor

pranavkm commented Feb 12, 2021

  • Could we move the DevServer in to the SDK / tool that's magically wired up the SDK?
  • Should we consider creating a framework reference for WASM? We could remove all package references
  • Why does the Client in the hosted app reference the dev server?
  • Simplify file structure
  • Theme redesign (for all the templates)

Also, use this issue to synchronize templates (web with MAUI): dotnet/maui#7868

@pranavkm pranavkm added the area-blazor Includes: Blazor, Razor Components label Feb 12, 2021
@pranavkm pranavkm added this to the Next sprint planning milestone Feb 12, 2021
@ghost
Copy link

ghost commented Feb 12, 2021

Thanks for contacting us.
We're moving this issue to the Next sprint planning milestone for future evaluation / consideration. We will evaluate the request when we are planning the work for the next milestone. To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

@SteveSandersonMS SteveSandersonMS added affected-all This issue impacts all the customers enhancement This issue represents an ask for new feature or an enhancement to an existing one severity-minor This label is used by an internal tool labels Feb 19, 2021 — with ASP.NET Core Issue Ranking
@ghost
Copy link

ghost commented Jul 20, 2021

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.

@mkArtakMSFT mkArtakMSFT added triaged Priority:1 Work that is critical for the release, but we could probably ship without labels Nov 2, 2021
@mkArtakMSFT mkArtakMSFT changed the title Perform a pass cleaning up the Blazor Wasm templates Perform a pass cleaning up the web templates Nov 16, 2021
@TanayParikh
Copy link
Contributor

TanayParikh commented Nov 16, 2021

I believe #37201 could be taken care of during this cleanup pass of templates.

@guardrex
Copy link
Contributor

Reported by Kirk: There's a delta on CSS file indentation with most templates now adopting two spaces. The Blazor templates have four today.

@javiercn
Copy link
Member

#40935

@guardrex
Copy link
Contributor

guardrex commented Apr 25, 2022

Consider dropping the underscore prefix throughout project templates to make user code consistent among project templates, Blazor docs samples/examples, and dev coding guidelines. A search turned up the following. This might be all of them for Blazor project templates:

Cross References:

@ghost
Copy link

ghost commented May 24, 2022

Thanks for contacting us.

We're moving this issue to the .NET 7 Planning milestone for future evaluation / consideration. We would like to keep this around to collect more feedback, which can help us with prioritizing this work. We will re-evaluate this issue, during our next planning meeting(s).
If we later determine, that the issue has no community involvement, or it's very rare and low-impact issue, we will close it - so that the team can focus on more important and high impact issues.
To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

@ghost
Copy link

ghost commented Oct 27, 2022

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.

@mkArtakMSFT mkArtakMSFT removed this from the Backlog milestone Oct 27, 2022
@mkArtakMSFT mkArtakMSFT added this to the .NET 8 Planning milestone Oct 27, 2022
@dotnet dotnet deleted a comment Oct 27, 2022
@danroth27
Copy link
Member

We think the important parts of this issue are already covered by other issues.

@ghost ghost locked as resolved and limited conversation to collaborators Dec 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affected-all This issue impacts all the customers area-blazor Includes: Blazor, Razor Components enhancement This issue represents an ask for new feature or an enhancement to an existing one feature-templates Priority:1 Work that is critical for the release, but we could probably ship without severity-minor This label is used by an internal tool
Projects
None yet
Development

No branches or pull requests

7 participants