[Rt-commit] rt branch, 4.2-trunk, updated. rt-4.2.16-10-g6b3f75f80b

? sunnavy sunnavy at bestpractical.com
Fri Jan 29 08:56:29 EST 2021


The branch, 4.2-trunk has been updated
       via  6b3f75f80b5f7b2b8d59f0259de696e8ebed25d3 (commit)
      from  ece459fa43170ddfdd228a3a44ed526e42236827 (commit)

Summary of changes:
 share/html/Elements/Login     | 3 ++-
 share/html/NoAuth/Logout.html | 4 +++-
 2 files changed, 5 insertions(+), 2 deletions(-)

- Log -----------------------------------------------------------------
commit 6b3f75f80b5f7b2b8d59f0259de696e8ebed25d3
Author: Jim Brandt <jbrandt at bestpractical.com>
Date:   Wed Jan 27 09:18:59 2021 -0500

    Remove extra closing div on Login/Logout pages
    
    The Footer template adds a closing div, normally for the PageLayout
    template, so Login/Logout can omit the closing div for the "body" to
    follow the same pattern.
    
    Fixes: I#37049

diff --git a/share/html/Elements/Login b/share/html/Elements/Login
index 6181394288..b53f13a77f 100644
--- a/share/html/Elements/Login
+++ b/share/html/Elements/Login
@@ -100,7 +100,8 @@ jQuery(function(){
 <& /Elements/LoginHelp &>
 </div><!-- #login-box -->
 % $m->callback( %ARGS, CallbackName => 'AfterForm' );
-</div><!-- #login-body -->
+% # Omit closing div for login-body because Footer adds a closing div,
+% # normally for the PageLayout template
 <& /Elements/Footer, Menu => 0 &>
 <%ARGS>
 $next => ''
diff --git a/share/html/NoAuth/Logout.html b/share/html/NoAuth/Logout.html
index beee12955a..c6a6fa2fe9 100644
--- a/share/html/NoAuth/Logout.html
+++ b/share/html/NoAuth/Logout.html
@@ -61,7 +61,9 @@
 
 % $m->callback( %ARGS );
 
-</div></div>
+</div>
+% # Omit closing div for login-body because Footer adds a closing div,
+% # normally for the PageLayout template
 <& /Elements/Footer, Menu => 0 &>
 
 % $m->abort();

-----------------------------------------------------------------------


More information about the rt-commit mailing list