Fix: Form elements in thirdparty dashboard views get removed (#14152)

Author: iOvergaard

src/Umbraco.Web.UI.Client/src/views/common/dashboard.html

@@ -1,6 +1,6 @@
 <div ng-controller="Umbraco.DashboardController">
 
-    <form name="dashboardForm" val-form-manager>
+    <ng-form name="dashboardForm" val-form-manager>
 
         <umb-load-indicator ng-show="page.loading"></umb-load-indicator>
 
@@ -32,6 +32,6 @@
 
         </div>
 
-    </form>
+    </ng-form>
 
 </div>

tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/Tabs/tabs.spec.ts

@@ -46,6 +46,14 @@ test.describe('Tabs', () => {
     await openDocTypeFolder(umbracoUi, page);
   }
 
+  test.only('Click dashboard tabs', async ({umbracoUi, page}) => {
+    await umbracoUi.goToSection('content');
+    await page.locator('[data-element="tab-contentRedirectManager"] > button').click();
+    expect(page.locator('.redirecturlsearch')).not.toBeNull();
+    await page.locator('[data-element="tab-contentIntro"] > button').click();
+    await expect(page.locator('[data-element="tab-contentIntro"]')).toHaveClass('umb-tab ng-scope umb-tab--active');
+  });
+
   test('Create tab', async ({umbracoUi, umbracoApi, page}) => {
     await umbracoApi.documentTypes.ensureNameNotExists(tabsDocTypeName);
     await umbracoApi.content.deleteAllContent();
@@ -67,7 +75,7 @@ test.describe('Tabs', () => {
     await umbracoUi.waitForTreeLoad('settings');
 
     await umbracoUi.clickElement(umbracoUi.getTreeItem("settings", ["Document Types", tabsDocTypeName]))
-    // Create a tab 
+    // Create a tab
     await page.locator('.umb-group-builder__tabs__add-tab').click();
     await page.locator('ng-form.ng-invalid > .umb-group-builder__group-title-input').fill('Tab 1');
     // Create a 2nd tab manually
@@ -178,7 +186,7 @@ test.describe('Tabs', () => {
     await expect(await page.locator('[title=urlPicker]')).toHaveCount(0);
   });
 
-  test('Reorders tab', async ({umbracoUi, umbracoApi, page}) => { 
+  test('Reorders tab', async ({umbracoUi, umbracoApi, page}) => {
     await umbracoApi.documentTypes.ensureNameNotExists(tabsDocTypeName);
 
     const tabsDocType = new DocumentTypeBuilder()
@@ -485,24 +493,24 @@ test.describe('Tabs', () => {
     await openDocTypeFolder(umbracoUi, page);
     await page.locator('[alias="reorder"]').click();
     await page.locator('.umb-group-builder__tab').last().click();
-    
+
     // Drag and drop property from tab 2 into tab 1
     await page.locator('.umb-group-builder__property-meta > .flex > .icon >> nth=1').last().hover();
     await page.mouse.down();
     await page.locator('.umb-group-builder__tab >> nth=1').hover({force:true});
     await page.waitForTimeout(500);
     await page.locator('[data-element="group-Tab group"]').hover({force:true});
     await page.mouse.up();
-    
+
     // Stop reordering and save
     await page.locator('[alias="reorder"]').click();
     await umbracoUi.clickElement(umbracoUi.getButtonByLabelKey(ConstantHelper.buttons.save));
-    
+
     // Assert
     await umbracoUi.isSuccessNotificationVisible();
     await expect(await page.locator('[title="urlPickerTabTwo"]')).toBeVisible();
   });
-  
+
   test('Drags and drops a group and converts to tab', async ({umbracoUi, umbracoApi, page}) => {
     await umbracoApi.documentTypes.ensureNameNotExists(tabsDocTypeName);
     const tabsDocType = new DocumentTypeBuilder()
@@ -552,4 +560,4 @@ test.describe('Tabs', () => {
     await umbracoUi.isSuccessNotificationVisible();
     await expect(await page.locator('[title="tabGroup"]').first()).toBeVisible();
   });
-});
+});