From a20915664dc35cbc915e06bd881ac0384329fb79 Mon Sep 17 00:00:00 2001
From: Ibrahim Muhammad Nada <ibrrahim.nada@gmail.com>
Date: Fri, 29 Oct 2021 19:06:00 +0300
Subject: [PATCH] GitHub issue#11299 fix v8 (#11493)

* adding validation

* adding localization keys

* applying cleaning up notes

* fix spaces

* revert ex.xml

* spaces again

* keep it DRY

* chenge hasTabsAndFirstRender to hasTabsOrFirstRender

Co-authored-by: inada <inada@2p.com.sa>
---
 .../nestedcontent.doctypepicker.html          |  9 +++++++--
 src/Umbraco.Web.UI/Umbraco/config/lang/en.xml | 19 +++++++++----------
 .../Umbraco/config/lang/en_us.xml             |  1 +
 3 files changed, 17 insertions(+), 12 deletions(-)

diff --git a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/nestedcontent/nestedcontent.doctypepicker.html b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/nestedcontent/nestedcontent.doctypepicker.html
index fa146f12f04b..f429c04f1d31 100644
--- a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/nestedcontent/nestedcontent.doctypepicker.html
+++ b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/nestedcontent/nestedcontent.doctypepicker.html
@@ -22,7 +22,7 @@
                         <umb-icon icon="icon-navigation" class="handle"></umb-icon>
                     </td>
                     <td>
-                        {{ph = placeholder(config);""}}
+                        {{ph = placeholder(config);hasTabsOrFirstRender = (elemTypeTabs[config.ncAlias].length || config.ncAlias=='');""}}
                         <button type="button" class="btn-reset umb-nested-content__placeholder" ng-class="{'umb-nested-content__placeholder--selected':ph}" ng-click="openElemTypeModal($event, config)">
                             <umb-node-preview ng-if="ph" icon="ph.icon" name="ph.name"></umb-node-preview>
                             <localize key="content_nestedContentAddElementType" ng-if="!ph">Add element type</localize>
@@ -30,9 +30,14 @@
 
                     </td>
                     <td>
-                        <select id="{{model.alias}}_tab_select"
+                        <select ng-show="hasTabsOrFirstRender" id="{{model.alias}}_tab_select"
                                 ng-options="t for t in elemTypeTabs[config.ncAlias]"
                                 ng-model="config.ncTabAlias" required></select>
+                        <span ng-show="!hasTabsOrFirstRender" class="red">
+                            <localize key="content_nestedContentNoGroups">
+                                The selected element type does not contain any supported groups (tabs are not supported by this editor, either change them to groups or use the Block List editor).
+                            </localize>
+                        </span>
                     </td>
                     <td>
                         <input type="text" ng-model="config.nameTemplate" />
diff --git a/src/Umbraco.Web.UI/Umbraco/config/lang/en.xml b/src/Umbraco.Web.UI/Umbraco/config/lang/en.xml
index 2d1e7d9109f4..35d7b8862c08 100644
--- a/src/Umbraco.Web.UI/Umbraco/config/lang/en.xml
+++ b/src/Umbraco.Web.UI/Umbraco/config/lang/en.xml
@@ -273,6 +273,7 @@
     <key alias="nestedContentGroupHelpText">Select the group whose properties should be displayed. If left blank, the first group on the Element Type will be used.</key>
     <key alias="nestedContentTemplateHelpTextPart1">Enter an angular expression to evaluate against each item for its name. Use</key>
     <key alias="nestedContentTemplateHelpTextPart2">to display the item index</key>
+    <key alias="nestedContentNoGroups">The selected element type does not contain any supported groups (tabs are not supported by this editor, either change them to groups or use the Block List editor).</key>
     <key alias="addTextBox">Add another text box</key>
     <key alias="removeTextBox">Remove this text box</key>
     <key alias="contentRoot">Content root</key>
@@ -1091,17 +1092,16 @@ To manage your website, simply open the Umbraco backoffice and start adding cont
     <key alias="notificationsSavedFor">Notification settings saved for</key>
     <key alias="mailBody"><![CDATA[
       Hi %0%
-
+      
       This is an automated mail to inform you that the task '%1%'
       has been performed on the page '%2%'
       by the user '%3%'
-
+      
       Go to http://%4%/#/content/content/edit/%5% to edit.
-
+      
       %6%
-
+      
       Have a nice day!
-
       Cheers from the Umbraco robot
     ]]></key>
     <key alias="mailBodyVariantSummary">The following languages have been modified %0%</key>
@@ -1799,17 +1799,16 @@ To manage your website, simply open the Umbraco backoffice and start adding cont
     <key alias="includeSubpages">Include subpages</key>
     <key alias="mailBody"><![CDATA[
       Hi %0%
-
+      
       This is an automated mail to inform you that the document '%1%'
       has been requested for translation into '%5%' by %2%.
-
+      
       Go to http://%3%/translation/details.aspx?id=%4% to edit.
-
+      
       Or log into Umbraco to get an overview of your translation tasks
       http://%3%
-
+      
       Have a nice day!
-
       Cheers from the Umbraco robot
     ]]></key>
     <key alias="noTranslators">No translator users found. Please create a translator user before you start sending content to translation</key>
diff --git a/src/Umbraco.Web.UI/Umbraco/config/lang/en_us.xml b/src/Umbraco.Web.UI/Umbraco/config/lang/en_us.xml
index 62c4b4f739f0..2003536def7a 100644
--- a/src/Umbraco.Web.UI/Umbraco/config/lang/en_us.xml
+++ b/src/Umbraco.Web.UI/Umbraco/config/lang/en_us.xml
@@ -278,6 +278,7 @@
     <key alias="nestedContentGroupHelpText">Select the group whose properties should be displayed. If left blank, the first group on the Element Type will be used.</key>
     <key alias="nestedContentTemplateHelpTextPart1">Enter an angular expression to evaluate against each item for its name. Use</key>
     <key alias="nestedContentTemplateHelpTextPart2">to display the item index</key>
+    <key alias="nestedContentNoGroups">The selected element type does not contain any supported groups (tabs are not supported by this editor, either change them to groups or use the Block List editor).</key>
     <key alias="addTextBox">Add another text box</key>
     <key alias="removeTextBox">Remove this text box</key>
     <key alias="contentRoot">Content root</key>