--- loncom/interface/lonrss.pm	2021/12/14 03:22:21	1.53.2.5
+++ loncom/interface/lonrss.pm	2025/03/17 00:25:51	1.65
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # RSS Feeder
 #
-# $Id: lonrss.pm,v 1.53.2.5 2021/12/14 03:22:21 raeburn Exp $
+# $Id: lonrss.pm,v 1.65 2025/03/17 00:25:51 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -88,7 +88,7 @@ sub changefeeddisplay {
 }
 
 sub advertisefeeds {
-    my ($uname,$udom,$edit,$count,$hidden)=@_;
+    my ($uname,$udom,$edit,$count,$hidden,$skipheader)=@_;
     my $feeds='';
     my %feednames=&Apache::lonnet::dump('nohist_all_rss_feeds',$udom,$uname);
     my $mode='public';
@@ -127,10 +127,20 @@ sub advertisefeeds {
 	}
     }
     if ($feeds) {
-	return '<h4>'.&mt('Available RSS Feeds and Blogs').'</h4><ul>'.$feeds.'</ul>';
+	if ($skipheader) {
+	    return '<ul>'.$feeds.'</ul>';
+	} else {
+	    return '<h2 class="LC_heading_3">'.&mt('Available RSS Feeds and Blogs').'</h2><ul>'.$feeds.'</ul>';
+	}
     } elsif (!$edit) {
-        return '<h4>'.&mt('No available RSS Feeds and Blogs').'</h4>';
+	my $info = &mt('No available RSS Feeds and Blogs');
+	if ($skipheader) {
+	    return '<p>'.$info.'</p>';
+	} else {
+	    return '<h2 class="LC_heading_3">'.$info.'</h2>';
+	}
     }
+    return;
 }
 
 sub rss_link {
@@ -291,7 +301,7 @@ sub blocking_blogdisplay {
             if ($showdom eq '') {
                 $showdom = $blockdom;
             }
-            $output .= &mt('This restriction was set by an administrator in the [_1] LON-CAPA domain',$showdom);
+            $output .= &mt('This restriction was set by an administrator in the [_1] LON-CAPA domain.',$showdom);
         } else {
             $output .= &mt('This is because the blog owner is a student in one or more courses in which communication is being blocked.');
         }
@@ -337,7 +347,7 @@ sub handler {
     my ($displayfeedname,$displayoption)=&displayfeedname($filename,$uname,$udom);
     my ($blocked,$blocktext,$disabled,$disabletext);
     if (!&Apache::lonnet::is_course($udom,$uname)) {
-        my $clientip = &Apache::lonnet::get_requestor_ip($r);
+        my $clientip = &Apache::lonnet::get_requestor_ip($r); 
         ($blocked,$blocktext) = &blocking_blogdisplay($uname,$udom,$html,$filterfeedname,$clientip);
         if (&Apache::lonnet::usertools_access($uname,$udom,'blog')) {
             $disabled = 0;
@@ -365,7 +375,8 @@ sub handler {
 			 {'bread_crumbs'   => $brcrumb,
 			  'domain'         => $udom,
 			  'force_register' => $env{'form.register'}}).
-		  &changed_js());
+		  &changed_js().
+	          '<div class="LC_landmark" role="main">');
     } else { # render RSS
         my $server = &Apache::lonnet::absolute_url();
 	$r->print("<rss version='2.0' xmlns:dc='http://purl.org/dc/elements/1.1'>\n<channel>".
@@ -425,10 +436,10 @@ sub handler {
 	}
 # Add a new feed
         if (($html) && ($edit)) {
-	    $r->print('<h4>' . &mt('New RSS Feed or Blog'). '</h4>');
+	    $r->print('<h2 class="LC_heading_3">' . &mt('New RSS Feed or Blog'). '</h2>');
 	    $r->print('<form method="post" name="makenewfeed" action="">');
-            $r->print(&mt('Name').": <input type='text' size='40' name='namenewblog' />");
-	    $r->print('<input type="submit" value="'.&mt('New Feed').'" />');
+            $r->print('<label>'.&mt('Name').": <input type='text' size='40' name='namenewblog' />");
+	    $r->print('</label><input type="submit" value="'.&mt('New Feed').'" />');
 	    $r->print('</form>');
 	}
         if ($displayfeedname) { # this is an existing feed
@@ -525,13 +536,13 @@ sub handler {
 # Render private items?
             my $viewpubliconly=1;
 	    $r->print("\n".
-		      ($html?'<hr /><h3>':'<title>').
+		      ($html?'<hr /><h3 class="LC_heading_3">':'<title>').
 		      &mt('LON-CAPA Feed "[_1]" for [_2]',$displayfeedname,$name).
 		      ($displayoption eq 'hidden'?' ('.&mt('Hidden').')':'').
 		      ($html?'</h3>'.($edit?'<form method="post" name="lonhomework" enctype="multipart/form-data" action=""><br />'.
-				      &mt('Name of this Feed').
+				      '<label>'.&mt('Name of this Feed').
 				      ': <input type="text" size="50" name="newblogname" value="'.
-				      $displayfeedname.'" />':'').'<ul>':'</title>'));
+				      $displayfeedname.'" />':'').'</label><br /><ul>':'</title>'));
 	    if (($env{'user.name'} eq $uname) && ($env{'user.domain'} eq $udom)) {
 		$viewpubliconly=0;
             }
@@ -549,7 +560,8 @@ sub handler {
 							   'title' => 'Title',
 							   'link' => 'Link',
 							   'description' => 'Description',
-                                                           'enc' => 'Podcasted enclosure');
+                                                           'enc' => 'Podcasted enclosure',
+                                                           'status' => 'Status');
                         my $uploadlink;
                         if ($entry==$newid) {
 # Generate upload link only for last (new) entry
@@ -565,7 +577,11 @@ sub handler {
                             my $free_space = $disk_quota - ($current_disk_usage / 1024.);
                             # Format this number since it will be displayed onscreen
                             $free_space = sprintf("%.1f", $free_space);
-                            $uploadlink=&Apache::inputtags::file_selector(0,0,'*','both','',$free_space);
+                            my $fileclass = 'LC_left';
+                            my $notitle = 1;
+                            $uploadlink = &Apache::lonhtmlcommon::start_pick_box().
+                                          &Apache::inputtags::file_selector(0,0,'*','both','',$free_space,$fileclass,$notitle).
+                                          &Apache::lonhtmlcommon::end_pick_box();
 			} else {
 # Otherwise, display
                             $uploadlink='<tt>'.$newsfeed{$id.'_enclosureurl'}.'</tt>'.
@@ -579,6 +595,7 @@ sub handler {
 <li>
 <label><input name='$id\_modified' type='checkbox' value="modified" /> $lt{'store'}</label>
 &nbsp;&nbsp;
+<fieldset class="LC_landmark" style="display:inline;"><legend class="LC_visually_hidden">$lt{'status'}</legend>
 <label><input name='$id\_status' type="radio" value="public" $status{'public'} onclick="changed(this.form,'$id');" /> $lt{'public'}</label>
 &nbsp;&nbsp;
 <label><input name='$id\_status' type="radio" value="private" $status{'private'} onclick="changed(this.form,'$id');" /> $lt{'private'}</label>
@@ -586,13 +603,16 @@ sub handler {
 <label><input name='$id\_status' type="radio" value="hidden" $status{'hidden'} onclick="changed(this.form,'$id');" /> $lt{'hidden'}</label>
 &nbsp;&nbsp;
 <label><input name='$id\_status' type="radio" value="deleted" onclick="changed(this.form,'$id');" /> $lt{'delete'}</label>
+</fieldset>
+<br />
 <br />
+<label>
 $lt{'title'}:
-<input name='$id\_title' type='text' size='60' value='$newsfeed{$id.'_title'}' onchange="changed(this.form,'$id');" /><br />
-$lt{'description'}:<br />
-<textarea name='$id\_description' rows="6" cols="80" onchange="changed(this.form,'$id');">$newsfeed{$id.'_description'}</textarea><br />
-$lt{'link'}:
-<input name='$id\_link' type='text' size='60' value='$newsfeed{$id.'_link'}' onchange="changed(this.form,'$id');" /><br />
+<input name='$id\_title' type='text' size='60' value='$newsfeed{$id.'_title'}' onchange="changed(this.form,'$id');" /></label><br />
+<label>$lt{'description'}:<br />
+<textarea name='$id\_description' rows="6" cols="80" onchange="changed(this.form,'$id');">$newsfeed{$id.'_description'}</textarea></label><br /><br />
+<label>$lt{'link'}:
+<input name='$id\_link' type='text' size='60' value='$newsfeed{$id.'_link'}' onchange="changed(this.form,'$id');" /></label><br />
 $lt{'enc'} -
 $uploadlink
 <hr /></li>
@@ -644,7 +664,7 @@ ENDEDIT
                 $r->print('</form>');
 	    }
 	} # was a real display feedname
-	$r->print(($html?&Apache::loncommon::end_page():'</channel></rss>'."\n"));
+	$r->print(($html?'</div>'.&Apache::loncommon::end_page():'</channel></rss>'."\n"));
     } # a real user
     return OK;
 } # end handler