Merge branch 'bugfix/ui-improvements' into develop

Various user interface related fit-and-finish items. Many of these are long-standing issues that aren't critical to the functionality, but help ensure that interactive elements respond as expected, without any rough edges or surprises.

This includes:
- In the `ContentTypeListViewComponent`, reset the content type selection after initializing the modal (b6868cf)
- In the `ContentTypeListViewComponent`, display parent content types when creating a new topic (9beeb8a)
- In the `Edit` template, hide the warning regarding `DisableDelete` when creating a new topic (5ddbd26)
- In the toolbar, ensure the version history dropbox is visible in normal width modal windows (780e7ed)
- In the toolbar, ensure the version history dropbox is wide enough to fit the widest date format (0072c55)

Author: JeremyCaney

OnTopic.Editor.AspNetCore/Areas/Editor/Views/Editor/Components/ContentTypeList/Default.cshtml

@@ -20,7 +20,9 @@ else if (Model.TopicList.Count > 2) {
           var option = $('option:selected', this);
           @if (Model.EnableModal?? false) {
             <text>
-              initEditorModal('@Model.AttributeKey', option.text(), option.val(), @Model.OnModalClose); return false;
+              initEditorModal('@Model.AttributeKey', option.text(), option.val(), @Model.OnModalClose);
+              $(this).prop('selectedIndex', 0);
+              return false;
             </text>
           }
           else {

OnTopic.Editor.AspNetCore/Areas/Editor/Views/Editor/Edit.cshtml

@@ -20,7 +20,7 @@
   <div id="FormArea" class="row form area">
 
     <div class="col-md-12">
-      @if (Model.ContentTypeDescriptor.DisableDelete) {
+      @if (Model.ContentTypeDescriptor.DisableDelete && !Model.IsNew) {
         <div class="alert alert-warning" role="alert">
           <button type="button" class="close" data-dismiss="alert"><span aria-hidden="true">×</span><span class="sr-only">Close</span></button>
           <strong>Warning:</strong> This topic is part of the OnTopic internal organizational structure and is not intended to be modified. Deletion of this Topic has been disabled.

OnTopic.Editor.AspNetCore/Areas/Editor/Views/Shared/_Toolbar.cshtml

@@ -11,13 +11,13 @@
 
     @if (Model.Topic.VersionHistory.Count > 1) {
       <!-- Versions -->
-      <div id="VersionsDropdown" class="dropdown d-none d-lg-inline-block" style="display:inline-block;">
+      <div id="VersionsDropdown" class="dropdown d-none d-md-inline-block" style="display:inline-block;">
         <button class="btn btn-sm btn-ancillary dropdown-toggle" type="button" id="VersionsButton" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
           <span class="fa fa-history"></span>
           Versions
           <span class="caret"></span>
         </button>
-        <ul class="dropdown-menu" role="menu" aria-labelledby="VersionsButton" style="left: 3px; top: 98%;">
+        <ul class="dropdown-menu" role="menu" aria-labelledby="VersionsButton">
           @foreach (DateTime version in Model.Topic.VersionHistory) {
             <li role="presentation" class="small">
               <a tabindex="-1" role="menuitem" href="@Url.Action("SetVersion", "Editor", new { path=Model.Topic.WebPath, version=version.ToString("MM/dd/yyyy hh:mm:ss.fff tt"), isModal=Model.IsModal })">@version.ToString()</a>

OnTopic.Editor.AspNetCore/Components/ContentTypeListViewComponent.cs

@@ -100,13 +100,14 @@ public IViewComponentResult Invoke(
       | Get content type
       >-------------------------------------------------------------------------------------------------------------------------
       | If the database is uninitialized, the content type won't be found. In that case, return an empty view, which will
-      | effectively hide the component.
+      | effectively hide the component. If the topic cannot be found, assume it is a new topic and attempt to load the parent
+      | for context.
       \-----------------------------------------------------------------------------------------------------------------------*/
       var contentTypes          = _topicRepository.GetContentTypeDescriptors();
-      var actualTopic           = _topicRepository.Load(currentTopic.Id);
+      var actualTopic           = _topicRepository.Load(currentTopic.Id)?? _topicRepository.Load(currentTopic.Parent.Id);
       var actualContentType     = contentTypes.GetTopic(currentTopic.ContentType);
 
-      if (actualContentType is null) {
+      if (actualContentType is null || actualTopic is null) {
         return View(viewModel);
       }
 

OnTopic.Editor.AspNetCore/Shared/Styles/Layout/_layout.scss

@@ -78,6 +78,7 @@ body {
 //----------------------------------------------------------------------------------------------
 .dropdown-menu {
   padding                       : 10px;
+  min-width                     : 180px;
   li a {
     line-height                 : 1.4rem;
   }