dotnet / Scaffolding

Code generators to speed up development.
MIT License
635 stars 227 forks source link

Make the ASP.NET Core MVC scaffolders use Bootstrap 4 #712

Closed TheMegaMatt closed 6 years ago

TheMegaMatt commented 6 years ago

or at least switch some way between the two, especially for the CRUD scaffolding, makes the feature useless, its more work to fix the one what got generated that writing from scratch

danroth27 commented 6 years ago

We will need to support both Bootstrap 3 and 4 for folks that are migrating apps that use Bootstrap 3.

danroth27 commented 6 years ago

Updating the title since the template work is tracked by https://github.com/aspnet/templating/issues/278

TheMegaMatt commented 6 years ago

Would it be compete madness having a switch to have 3, 4 or none? Like having a starter template but with just some really basic html so people can chose what framework to use

Il ven 22 giu 2018, 18:37 Daniel Roth notifications@github.com ha scritto:

We will need to support both Bootstrap 3 and 4 for folks that are migrating apps that use Bootstrap 3.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/aspnet/Scaffolding/issues/712#issuecomment-399504804, or mute the thread https://github.com/notifications/unsubscribe-auth/ACef8bJQiFeZU63ji_SAvsJ-dqNQOuL_ks5t_R1RgaJpZM4SxwfI .

danroth27 commented 6 years ago

@TheMegaMatt Sounds like a reasonable ask, but please file a separate issue so that we can track the ask separately from the work to update to Bootstrap 4.

TheMegaMatt commented 6 years ago

Was just brainstorming, will do as I get on a PC

Il 22 giu 2018 18:55, "Daniel Roth" notifications@github.com ha scritto:

@TheMegaMatt https://github.com/TheMegaMatt Sounds like a reasonable ask, but please file a separate issue so that we can track the ask separately from the work to update to Bootstrap 4.

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/aspnet/Scaffolding/issues/712#issuecomment-399510004, or mute the thread https://github.com/notifications/unsubscribe-auth/ACef8WuDJtbxMPS9MOV-xl5yPpH3iH4eks5t_SF9gaJpZM4SxwfI .

danroth27 commented 6 years ago

Splitting off the work to update the identity scaffolder: https://github.com/aspnet/Scaffolding/issues/849

ryanbrandenburg commented 6 years ago

Can I get a bit more guidance here? How do we intend to separate the bootstrap4 stuff from the bootstrap3 stuff? Do we have a list of the items which we need to BS4-ify? How can I most easily test these bad boys out to make sure they look good while I'm working on them? This might be a meeting I should have with the Scaffolding folks, but if we also answer some of these questions here we can preserve them for others.

ryanbrandenburg commented 6 years ago

@danroth27 do you know the answers to any of my above questions? If not where might I start looking for them?

danroth27 commented 6 years ago

How do we intend to separate the bootstrap4 stuff from the bootstrap3 stuff?

I believe the agreement is that you just need to worry about providing the BS4 content. The scaffolding folks will work out how to integrate it with the existing scaffolders.

Do we have a list of the items which we need to BS4-ify?

This issue tracks BS4-ify the Razor Pages with EF Core (CRUD) content and the MVC with EF Core (CRUD) content.

How can I most easily test these bad boys out to make sure they look good while I'm working on them?

@mkArtakMSFT Since we're just providing content would it make sense to implement the content as a sample app?

ryanbrandenburg commented 6 years ago

I have the content changes @danroth27 described in a branch here. That's not actually all this issue tracks now that I'm thinking about it though, we also, at minimum, need to update the _Layout.cshtml to match our new layout (though that should just be a copy-paste). @danroth27 are there any other hidden items I'm missing?

ryanbrandenburg commented 6 years ago

I updated that branch with out new layout, adjusted to meet the existing standards where I thought appropriate. It does NOT contain an updated site.css (required for the layout to look good) because the only one I saw was in the Identity folder and I didn't want to cause conflicts for @javiercn when he merged his Identity Scaffolding stuff.

I don't see any further work to do here, so if there's something I missed let me know.

ryanbrandenburg commented 6 years ago

@danroth27 who on the Scaffolding team to I pass this to to complete the layup?

javiercn commented 6 years ago

@seancpeters

vijayrkn commented 6 years ago

Bootstrap 4 support is now available for both ASP.NET Core MVC and Identity Scaffolders.