Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Admin: Log out page styled #2397

Merged
merged 1 commit into from
Sep 25, 2024
Merged

Admin: Log out page styled #2397

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
35 changes: 35 additions & 0 deletions benefits/templates/registration/logged_out.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
{% extends "admin/agency-base.html" %}
{% load static %}

{% block title %}
Logged out | Cal-ITP Benefits Administrator
{% endblock title %}

{% block extrastyle %}
{{ block.super }}
<link rel="stylesheet" href="{% static "admin/css/login.css" %}">
{% endblock extrastyle %}

{% block bodyclass %}
{{ block.super }} login
{% endblock bodyclass %}

{% block branding %}
<div class="d-flex justify-content-center w-100 bg-primary">
<img class="my-3" src="{% static "img/logo-lg.svg" %}" width="220" height="50" alt="California Integrated Travel Project: Benefits logo (large)" />
</div>

<div id="site-name">
<h1 class="text-center text-white fs-3 py-3 m-0">Administrator</h1>
</div>
Comment on lines +18 to +24
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the same exact code as https://github.com/cal-itp/benefits/blob/main/benefits/templates/admin/login.html#L21-L29 so I could turn it into an includes... but I didn't think we'd re-use this a third time, so I opted not to.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, I can't see where we would re-use it since it's specific to the log in and log out pages.

{% endblock branding %}

{% block content %}

<div id="content-main">
<p class="text-center mt-4">You have logged out successfully.</p>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just noticed that we have You've logged out successfully. in Miro but since we don't have a strict copy I think it's ok.


<a class="btn btn-lg btn-outline-primary d-block my-5" href="{% url 'admin:index' %}">Log in again</a>
</div>

{% endblock content %}
Loading