PR#1173: Replace Login icon with text - pagure

pagure

#1173 Replace Login icon with text

Merged 7 years ago by pingou. Opened 7 years ago by ryanlerch.

ryanlerch/pagure login_icon into master

Replace Login icon with text

Ryan Lerch • 7 years ago

6359fe4

pagure/templates/master.html

file modified

+1 -2

		`@@ -73,8 +73,7 @@`
		`{% else %}`
		`<li class="nav-item p-l-1">`
		`<a class="nav-link" href="{{ url_for('auth_login') }}?next={{request.url}}">`
		`- <span class="oi" data-glyph="account-login" title="Log In" aria-hidden="true">`
		`- </span>`
		`+ Login`
		`</a>`
		`</li>`
		`{% endif %}`

ryanlerch commented 7 years ago

quick fix for the login icon looking like a logout icon

fixes: #1171

farhaan commented 7 years ago

:laughing: @abompard will be really happy now

pingou commented 7 years ago

I'm going to merge this as is (after a rebase) but I wonder if we couldn't add a icon next to the login as Login by itself is a little "dry" so to say.

rebased

7 years ago