Refactored modal, form and cards.

src/Volo.Abp.AspNetCore.Mvc.UI.Bootstrap/TagHelpers/Card/AbpCardBodyTagHelper.cs

@@ -1,12 +1,11 @@
 using Microsoft.AspNetCore.Razor.TagHelpers;
 
-namespace Volo.Abp.AspNetCore.Mvc.UI.Bootstrap.TagHelpers
+namespace Volo.Abp.AspNetCore.Mvc.UI.Bootstrap.TagHelpers.Card
 {
     public class AbpCardBodyTagHelper : TagHelper
     {
         public override void Process(TagHelperContext context, TagHelperOutput output)
         {
-            //TODO: Remove abp-modal HTML element which is unnecessary!
             output.TagName = "div";
             output.Attributes.AddClass("card-body");
         }

src/Volo.Abp.AspNetCore.Mvc.UI.Bootstrap/TagHelpers/Card/AbpCardHeaderTagHelper.cs

@@ -1,12 +1,11 @@
 using Microsoft.AspNetCore.Razor.TagHelpers;
 
-namespace Volo.Abp.AspNetCore.Mvc.UI.Bootstrap.TagHelpers
+namespace Volo.Abp.AspNetCore.Mvc.UI.Bootstrap.TagHelpers.Card
 {
     public class AbpCardHeaderTagHelper : TagHelper
     {
         public override void Process(TagHelperContext context, TagHelperOutput output)
         {
-            //TODO: Remove abp-modal HTML element which is unnecessary!
             output.TagName = "div";
             output.Attributes.AddClass("card-header");
         }

src/Volo.Abp.AspNetCore.Mvc.UI.Bootstrap/TagHelpers/Card/AbpCardTagHelper.cs

@@ -1,12 +1,11 @@
 using Microsoft.AspNetCore.Razor.TagHelpers;
 
-namespace Volo.Abp.AspNetCore.Mvc.UI.Bootstrap.TagHelpers
+namespace Volo.Abp.AspNetCore.Mvc.UI.Bootstrap.TagHelpers.Card
 {
     public class AbpCardTagHelper : TagHelper
     {
         public override void Process(TagHelperContext context, TagHelperOutput output)
         {
-            //TODO: Remove abp-modal HTML element which is unnecessary!
             output.TagName = "div";
             output.Attributes.AddClass("card");
         }

src/Volo.Abp.AspNetCore.Mvc.UI.Bootstrap/TagHelpers/Modal/AbpModalTagHelper.cs

@@ -7,7 +7,7 @@ namespace Volo.Abp.AspNetCore.Mvc.UI.Bootstrap.TagHelpers.Modal
     {
         public override void Process(TagHelperContext context, TagHelperOutput output)
         {
-            //TODO: Remove abp-modal HTML element which is unnecessary!
+            output.TagName = null;
             output.PreContent.SetHtmlContent(CreatePreContent());
             output.PostContent.SetHtmlContent(CreatePostContent());
         }

src/Volo.Abp.Identity.Web/Pages/Identity/Users/CreateModal.cshtml

@@ -7,10 +7,10 @@
     Layout = null;
 }
 
-<abp-modal>
-
 <form method="post" asp-page="/Identity/Users/CreateModal">
 
+    <abp-modal>
+
         <abp-modal-header title="@L["NewUser"]"></abp-modal-header>
 
         <abp-modal-body>
@@ -71,6 +71,6 @@
 
         <abp-modal-footer></abp-modal-footer>
 
-    </form>
-
     </abp-modal>
+
+</form>

src/Volo.Abp.Identity.Web/Pages/Identity/Users/EditModal.cshtml

@@ -7,10 +7,10 @@
     Layout = null;
 }
 
-<abp-modal>
-
 <form method="post" asp-page="/Identity/Users/EditModal">
 
+    <abp-modal>
+
         <abp-modal-header title="@L["NewUser"]"></abp-modal-header>
 
         <abp-modal-body>
@@ -68,6 +68,6 @@
 
         <abp-modal-footer></abp-modal-footer>
 
-    </form>
-
     </abp-modal>
+
+</form>

src/Volo.Abp.Identity.Web/wwwroot/modules/identity/helpers/ModalManager.js

@@ -58,10 +58,10 @@ $.validator.defaults.ignore = ''; //TODO: Would be better if we can apply only f
 
             function _initAndShowModal() {
                 _$modal = _$modalContainer.find('.modal');
+                _$form = _$modalContainer.find('form');
 
-                _$form = _$modal.find('form');
                 if (_$form.length) {
-                    $.validator.unobtrusive.parse(_$form);
+                    $.validator.unobtrusive.parse(_$form); //TODO: We should do a more common system to initialize component on ajax-loaded DOM elements. validator.unobtrusive.parse is only one thing to do.
                     if (_$form.attr('data-ajaxForm') !== 'false') {
                         //TODO: Create abpAjaxForm to not repeat that code!
                         _$form.ajaxForm({