[Rt-commit] rt branch, 4.4-trunk, updated. rt-4.4.4-236-g0116ce10e8

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


The branch, 4.4-trunk has been updated
       via  0116ce10e86c697fb3df3f1737364cc9f5baa58b (commit)
      from  7f379a59d0be8207750779e83b6f10c3df14bb6d (commit)

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

- Log -----------------------------------------------------------------
commit 0116ce10e86c697fb3df3f1737364cc9f5baa58b
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 25304b47a8..b0af97fd91 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