From 9495538f8633729de26fdce6fd56373afd422729 Mon Sep 17 00:00:00 2001
From: Retro_Guy <retro.guy@rocksolidbbs.com>
Date: Tue, 30 Apr 2024 04:31:50 -0700
Subject: [PATCH] Add green button to 'Latest' link in groups list for logged
 in users.

---
 .../common/images/new-articles.png            | Bin 0 -> 1607 bytes
 Rocksolid_Light/rocksolid/newsportal.php      |  57 ++++++++++--------
 2 files changed, 33 insertions(+), 24 deletions(-)
 create mode 100644 Rocksolid_Light/common/images/new-articles.png

diff --git a/Rocksolid_Light/common/images/new-articles.png b/Rocksolid_Light/common/images/new-articles.png
new file mode 100644
index 0000000000000000000000000000000000000000..fca67151ed6dbc17be9f54b19d1cffd7234a1df7
GIT binary patch
literal 1607
zcmV-N2Dtf&P)<h;3K|Lk000e1NJLTq001BW001Be1^@s6b9#F800006VoOIv0RI60
z0RN!9r;`8x00(qQO+^RV3KIb@4H^Pl-T(jv=}AOER9J<zmuqZXM;V5nIcKlFG+SRb
zPUN_^L_uvuAV3iW!^T0RQRzjfMpcPKK|(?xRRt-63aF?^;Rm;Wr56MO(SIl*pb~+K
z0|68gV!(FO25JgO!H#2lce8i<oSBy&-gVa-+p%c!jaJ$--<)~h_xrw?Gh^7GWipu#
zYwd@DAxXO=-6AO^2}zfMe>vyIfe~x%i=(5Xmp3x6x!Qoi!NDz_=Y2)eBa)JJKqTRu
zLlOYZ{gSGZeqybCa%^nuzyBA30|ySotJUh4B|Rpo!#PK}T&Cl;B=5NQZEP9XLQhu@
z#u(<ZbIeZ7G5)J_O#gKT-}f7&s=#-X$>iZvr%qKjOdylV^hr7*={?Rls;<I4U;Gfm
zLl3dD{dTNELP8vtu->dfES8zdMSlMKqx|ylODvXx29Z(E^X^|Q8dnh5yLay$&bi}~
zw$*Aic7O6t4t?Sa^hW!Z;Tm8eVN;)Nq7zcA6?pcC&oc4M-!aA@Y06rA-}&?BN7p2<
zZ{NPnVHlp3)L&jG^T^W=^U=3Ij8l_RmcScPSdN5N`a?oO1swas5q|sVpYffKq>Dii
zylZlD^1>AahKGk^Gcz+Mfp-_m1wQ}N&+@*34=hclCZ&z(wGzy##oyHT!e4&G>z|s%
zgdUPci^bymr>3SV0Dgl&cX#&_z{hH}8h3v?!w2u&Pn9ZFDpXaWLWNrWxl-?2_EVvv
z7C#lLT6*5xx0$nDf5bbP#5uP$7K`1G&1R1;M<A2Q^jT{sopbGWu*|o<_8o*aoO4@$
zUg2*_lf`l(JkEcfoWuOA9ZA+0vukp4@>~SKTKj~g_Ix$R!Owk!k_zh<*lIbyq9*ET
zYW&}1Zlw0<S84loJFX<(IrlhlFtTUQo=!<0cg}Hn|1>wJZXvJS8cNp5d23CyM8(SD
zdP3a6BKlq%l7{>H`yY*Xo_CLPuB}?F()VB&70S!%+*)8Q>YCHE8u5^jB1=2-!u~vo
z=Q;_3Am;o20}*TO5I~_L$BjGOlKR%1w}M?I=glc-MCFpZ2!Oq50e^>BJSTvmNd3b?
zZFhmJYUJGfiUMrT`C96lBh^aWVp(K?yt2#tjqX~&O#rYfBI#C1xO6~O75?E~A#M8B
z%=v2S){2UgB_N=nJP!8;fpm!EBk8t?q#K-b&}{+un?1uj{rA?RuyRvbP2GCM8Ym!O
z5up{^z3T}9A4$oGBqIsq83YtnVATGJo4i{wn03{yw(Kn73bm`5F>idnBa#Y0x0#PD
z^DHROD|U=dvx$UBE)`GfQfx{=tHP_9Hy3T3C2A2#uS@FoE=5=Hw%Db@5tw>Yy}eoU
z=A$@Q1DNzAjXURvPjs&Nf)EaOO~3)nnRw$wHQ`xnPe>wusSWd5?3w_qXOXD{wU@&N
zf!}!Xc>EWVs=n{zzu0lz33QC~QYw{@6a+zV%sYMhbV1S!l4w8Li@6lJZs2|{O8EVF
zqX3T-i^Vw)z!>vYNfkeg5PQ1&x)DhIcpJG3c_cYWUx)gEWM*cjm`<lVBn|qnB~WjO
zeM{|{f_ILjDLw@Y3kx{sp2_F)KLGF=1LE=cVdvagNhH3RCUU0znt>-@-Gu#82jy}Z
zNfU`g;!wl4d?cHmo(@u})Ny0XLkiUyeW?@EE9}lU7m3c1G<qyW_F@i6v%t{&{QT?{
z1OT$xY$26O{Z7)o3f1YKY$HC`hP}Ot+ScD*Tg}f$>3aTViie?)Eg)&u^St|Vx!lXm
z<5s<xY~Q}U-#K?o(r!saH$^Z9D^%~!5^h^RBGIWN?I(J$&nL)U$YHHT(l~H`KA-=0
z>-aUVW?Q#zZHvWX-;nefNfAlJVlkpS4Z3Rq(iWEvFvj4`MKQAxI31%hzDTK5YAiJ;
z>A6@ec4&5XwzL|~RWGLl0|R#ek2~icloW5igaRursZGBCI0`(K%jL$`oByf=8rI+6
z-{qWpK+=$;w@TU}DNzTkN%|)+CTS!Hf@8&Eac(2?{tvMhWoOC<Q|15w002ovPDHLk
FV1kkJ5iI}!

literal 0
HcmV?d00001

diff --git a/Rocksolid_Light/rocksolid/newsportal.php b/Rocksolid_Light/rocksolid/newsportal.php
index 06bac47..8d7f41b 100644
--- a/Rocksolid_Light/rocksolid/newsportal.php
+++ b/Rocksolid_Light/rocksolid/newsportal.php
@@ -633,27 +633,6 @@ function groups_show($gruppen)
             if ($acttype != "group") {
                 $acttype = "group";
             }
-            /* Display group name and description */
-            if (isset($userdata[$g->name])) {
-                $lineclass = "np_thread_line2";
-            } else {
-                $lineclass = "np_thread_line1";
-            }
-            $groupdisplay = '<tr class="' . $lineclass . '"><td style="text-align: center;" class="' . $lineclass . '">';
-            $groupdisplay .= '<a href="overboard.php?thisgroup=' . _rawurlencode($g->name) . '">';
-            if ((isset($_SESSION['theme'])) && file_exists('../common/themes/' . $_SESSION['theme'] . '/images/latest.png')) {
-                $latest_image = '../common/themes/' . $_SESSION['theme'] . '/images/latest.png';
-            } else {
-                $latest_image = '../common/images/latest.png';
-            }
-            $groupdisplay .= '<img src="' . $latest_image . '">';
-            $groupdisplay .= '</a>';
-            $groupdisplay .= '</td>';
-
-            $groupdisplay .= '<td class="' . $lineclass . '">';
-            $groupdisplay .= '<span class="np_group_line_text">';
-            $groupdisplay .= '<a ';
-            $groupdisplay .= 'target="' . $frame['content'] . '" ';
             // Get last article info from overview
             $database = $spooldir . '/articles-overview.db3';
             $table = 'overview';
@@ -667,6 +646,7 @@ function groups_show($gruppen)
                 $found = 1;
                 break;
             }
+            $overview_dbh = null;
             if ($found == 1) {
                 $lastarticleinfo['date'] = $row['date'];
             }
@@ -678,6 +658,37 @@ function groups_show($gruppen)
                 $new_style_off = '</i></b>';
                 $new = true;
             }
+            /* Display group name and description */
+            if (isset($userdata[$g->name])) {
+                $lineclass = "np_thread_line2";
+            } else {
+                $lineclass = "np_thread_line1";
+            }
+            if($new) {
+                $latest_link = '&time=' . $userdata[$g->name];
+            } else {
+                $latest_link = '';
+            }
+            $groupdisplay = '<tr class="' . $lineclass . '"><td style="text-align: center;" class="' . $lineclass . '">';
+            $groupdisplay .= '<a href="overboard.php?thisgroup=' . _rawurlencode($g->name) . $latest_link . '">';
+            if ((isset($_SESSION['theme'])) && file_exists('../common/themes/' . $_SESSION['theme'] . '/images/latest.png')) {
+                $latest_image = '../common/themes/' . $_SESSION['theme'] . '/images/latest.png';
+            } else {
+                $latest_image = '../common/images/latest.png';
+            }
+            if ($new) {
+                $latest_image = '../common/images/new-articles.png';
+                $groupdisplay .= '<img src="' . $latest_image . '" title="New articles">';
+            } else {
+                $groupdisplay .= '<img src="' . $latest_image . '" title="Recent articles">';
+            }
+            $groupdisplay .= '</a>';
+            $groupdisplay .= '</td>';
+
+            $groupdisplay .= '<td class="' . $lineclass . '">';
+            $groupdisplay .= '<span class="np_group_line_text">';
+            $groupdisplay .= '<a ';
+            $groupdisplay .= 'target="' . $frame['content'] . '" ';
             $groupdisplay .= 'href="' . $file_thread . '?group=' . urlencode($g->name) . '"><span class="np_group_line_text">' . $new_style_on . group_display_name($g->name) . $new_style_off . "</span></a>\n";
             if ($new) {
                 echo '</i></b>';
@@ -685,13 +696,11 @@ function groups_show($gruppen)
             if ($g->description != "-") {
                 $groupdisplay .= '</span><br><p class="np_group_desc">' . $g->description . '</p>';
             }
-            $overview_dbh = null;
             if (isset($userdata[$g->name])) {
                 $groupdisplay .= '</span><p class="np_group_user_tools">';
                 $groupdisplay .= '<a class="np_group_user_tools" href="index.php?unsub=' . urlencode($g->name) . '">(unsubscribe)</a>';
                 if ($new) {
-                    $groupdisplay .= '&nbsp;<a href="overboard.php?thisgroup=' . urlencode($g->name) . '&time=' . $userdata[$g->name] . '"><b>(new)</b></a> ';
-                    $groupdisplay .= '&nbsp;<a class="np_group_user_tools" href="index.php?mark_read=' . urlencode($g->name) . '">(mark read)</a>';
+                    $groupdisplay .= '<a class="np_group_user_tools" href="index.php?mark_read=' . urlencode($g->name) . '">(mark read)</a>';
                 }
                 $groupdisplay .= '</p';
             } else {