--- loncom/interface/lonextresedit.pm	2023/05/22 21:10:55	1.31
+++ loncom/interface/lonextresedit.pm	2023/07/12 00:05:02	1.34
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Documents
 #
-# $Id: lonextresedit.pm,v 1.31 2023/05/22 21:10:55 raeburn Exp $
+# $Id: lonextresedit.pm,v 1.34 2023/07/12 00:05:02 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -109,7 +109,7 @@ sub handler {
         %ltitools = (
                       dom => \%domtools,
                       crs => \%crstools,
-                    );  
+                    );
     }
     my $js = &Apache::lonhtmlcommon::scripttag(&extedit_javascript());
     my $pathitem = '<input type="hidden" name="folderpath" value="'.
@@ -520,7 +520,7 @@ sub extedit_form {
                         if (ref($ltitools->{'dom'}) eq 'HASH') {
                             $ltihash = $ltitools->{'dom'}->{$tool};
                         }
-                    }     
+                    }
                 }
                 if (($tool ne '') && (ref($ltihash) eq 'HASH')) {
                     my $tooltitle = $ltihash->{'title'};
@@ -631,7 +631,7 @@ sub extedit_form {
                     $exttypeofftext = &mt('No external tools defined in the domain are available for selection.');
                 } elsif (!$numcrstools && $numdomtools) {
                     $defcheck{'dom'} = ' checked="checked"';
-                    $subdivstyle = 'display:none';
+                    $subdivstyle = 'display:block';
                     $exttypeonsty = 'display:block';
                     my $firstoption = '<option value="" selected="selected">'.&mt('Select').'</option>';
                     $seloptions = &ordered_tooloptions($ltitools->{'dom'});
@@ -716,13 +716,17 @@ $preview
 </div>
 ";
     }
+    my $idattr;
+    unless ($subdivid eq '') {
+        $idattr = 'id="'.$subdivid.'"';
+    }
     $form = <<ENDFORM;
 <form action="$action" method="post" name="$formname" id="$formid">
 <fieldset id="$fieldsetid" style="$fieldsetstyle">
 $legend
 $active
 $chooser
-<div id="$subdivid" style="$subdivstyle">
+<div $idattr style="$subdivstyle">
 <span$srcclass>
 $title
 <input type="text" size="$size" name="exttitle" value="$orig_title" $disabled />
@@ -1053,7 +1057,7 @@ function setExternal(extform,residx,type
         if (residx > 0) {
             labelinput += '_'+residx;
             titleinput += '_'+residx;
-            appendinput += '_'+residx; 
+            appendinput += '_'+residx;
         }
         if (document.getElementById(labelinput)) {
             var crslabel = document.getElementById(labelinput).value;
@@ -1360,7 +1364,7 @@ function updateExttool(caller,form,suppl
                                 if ((ltitoolsDisplay[i][j]) && (ltitoolsTarget[i][j] == 'window')) {
                                     dimen = 'block';
                                     dimenwidth = ltitoolsWidth[i][j];
-                                    dimenheight = ltitoolsHeight[i][j];                    
+                                    dimenheight = ltitoolsHeight[i][j];
                                 }
                                 if (document.getElementById(dimendiv)) {
                                     document.getElementById(dimendiv).style.display = dimen;
@@ -1387,7 +1391,7 @@ function updateExttool(caller,form,suppl
                                 }
                                 if (document.getElementById(linktextinput)) {
                                     if (ltitoolsLink[i][j]) {
-                                        document.getElementById(linktextinput).value = ltitoolsLinkDef[i][j]; 
+                                        document.getElementById(linktextinput).value = ltitoolsLinkDef[i][j];
                                     } else {
                                         document.getElementById(linktextinput).value = '';
                                     }