#3602 fix text colour on login button
Merged 2 years ago by ryanlerch. Opened 2 years ago by ryanlerch.
ryanlerch/pagure login-button-colours  into  master

file modified
+1 -1
@@ -89,7 +89,7 @@ 

                      </li>

                    {% else %}

                      <li class="nav-item">

-                       <a class="nav-link btn btn-primary" href="{{

+                       <a class="btn btn-primary" href="{{

                        url_for('auth_login') }}?next={{request.url

                        }}">Log In</a>

                      </li>

@@ -234,7 +234,7 @@ 

              flask_v = tuple(int(el) for el in flask.__version__.split('.'))

              if flask_v < (0, 12, 0):

                  self.assertIn(

-                     '<a class="nav-link btn btn-primary" '

+                     '<a class="btn btn-primary" '

                      'href="/login/?next=http://localhost/">', output_text)

                  self.assertIn(

                      'Could not set the session in the db, please report '
@@ -576,7 +576,7 @@ 

              flask_v = tuple(int(el) for el in flask.__version__.split('.'))

              if flask_v <= (0, 12, 0):

                  self.assertIn(

-                     '<a class="nav-link btn btn-primary" '

+                     '<a class="btn btn-primary" '

                      'href="/login/?next=http://localhost/">', output_text)

                  self.assertIn(

                      'Could not set the session in the db, please report '
@@ -842,7 +842,7 @@ 

          self.assertIn('<title>Home - Pagure</title>', output.get_data(as_text=True))

          self.assertNotIn('You have been logged out', output.get_data(as_text=True))

          self.assertIn(

-             '<a class="nav-link btn btn-primary" '

+             '<a class="btn btn-primary" '

              'href="/login/?next=http://localhost/">', output.get_data(as_text=True))

  

          user = tests.FakeUser(username='foo')

previously, the text on the login button was blueish, which
was not good since the button was also blue. this fixes this
so the text is now white.

Fixes #3581

rebased onto 647ecf64c30f2af152c6c74fb11392b79fb24f19

2 years ago

Pretty please pagure-ci rebuild

:thumbsup: , @ryanlerch go ahead and merge this PR after a rebase :)

rebased onto ca4b80b

2 years ago

Pull-Request has been merged by ryanlerch

2 years ago