fixes #3621

Closes #3621

Merge request studip/studip!2512

lib/navigation/LoginNavigation.php

@@ -31,7 +31,7 @@ class LoginNavigation extends Navigation
                 $navigation->setLinkAttributes([
                     'id' => 'toggle-login'
                 ]);
-                $navigation->setURL('#login-form');
+                $navigation->setURL('#toggle-login');
                 $this->addSubNavigation('standard_login', $navigation);
                 $standard_login_active = true;
             }

templates/loginform.php

@@ -68,7 +68,7 @@ $show_hidden_login = false;
                                 <? $name_and_title = explode(' - ', $nav->getTitle()) ?>
                                 <li class="login_link">
                                     <? if (is_internal_url($url = $nav->getURL())) : ?>
-                                    <? SkipLinks::addLink($name_and_title[0], $url) ?>
+                                    <? SkipLinks::addLink($name_and_title[0], URLHelper::getLink($url, ['cancel_login' => 1])) ?>
                                     <a href="<?= URLHelper::getLink($url, ['cancel_login' => 1]) ?>" <?= arrayToHtmlAttributes($nav->getLinkAttributes()) ?>>
                                         <? else : ?>
                                         <a href="<?= htmlReady($url) ?>" target="_blank" rel="noopener noreferrer">
@@ -83,7 +83,6 @@ $show_hidden_login = false;
                         <? endforeach ?>
                     </ul>
                 </nav>
-
                 <? if ($show_hidden_login) : ?>
                     <?= $this->render_partial('_standard_loginform', [
                         'hidden' => empty($loginerror),