Skip to content

Update Razor Components template to have only one Pages directory #8516

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

Closed
mkArtakMSFT opened this issue Mar 14, 2019 · 4 comments
Closed

Update Razor Components template to have only one Pages directory #8516

mkArtakMSFT opened this issue Mar 14, 2019 · 4 comments
Assignees
Labels
area-blazor Includes: Blazor, Razor Components Done This issue has been fixed enhancement This issue represents an ask for new feature or an enhancement to an existing one

Comments

@mkArtakMSFT
Copy link
Member

We should do this after #6374.

@mkArtakMSFT mkArtakMSFT added 1 - Ready enhancement This issue represents an ask for new feature or an enhancement to an existing one area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates area-blazor Includes: Blazor, Razor Components labels Mar 14, 2019
@mkArtakMSFT mkArtakMSFT added this to the 3.0.0-preview4 milestone Mar 14, 2019
@javiercn
Copy link
Member

@rynowak Can we do something here already?

@danroth27
Copy link
Member

danroth27 commented Mar 25, 2019

Proposed project structure:

  • Pages
    • _Imports.razor
    • _ViewImports.cshtml
    • Counter.razor
    • FetchData.razor
    • Index.razor
    • Host.cshtml
  • Services
    • WeatherForecast.cs
    • WeatherForecastService.cs
  • Shared
    • MainLayout.razor
    • NavMenu.razor
  • _Imports.razor
  • App.razor

Some downsides to this approach

@rynowak @SteveSandersonMS @DamianEdwards

@SteveSandersonMS
Copy link
Member

Maybe Index.cshtml should be something like Host.cshtml as a shorter-term way to sidestep the first issue.

@danroth27
Copy link
Member

Maybe Index.cshtml should be something like Host.cshtml as a shorter-term way to sidestep the first issue.

We would need an explicit route in Host.cshtml (@page "/"), but I think that works.

@javiercn javiercn added 2 - Working Done This issue has been fixed and removed 1 - Ready labels Mar 25, 2019
@mkArtakMSFT mkArtakMSFT removed area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates labels May 9, 2019
@ghost ghost locked as resolved and limited conversation to collaborators Dec 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-blazor Includes: Blazor, Razor Components Done This issue has been fixed enhancement This issue represents an ask for new feature or an enhancement to an existing one
Projects
None yet
Development

No branches or pull requests

4 participants