Convert Authentication component style to match with rest of the links

chingu-voyages / v24-geckos-team-02

Books Plus | Voyage-24 | https://chingu.io/

GNU General Public License v3.0

1 stars 1 forks source link

Convert Authentication component style to match with rest of the links #78

Closed ArunJose closed 3 years ago

ArunJose commented 3 years ago

This PR addresses issue #77

Changes

Convert login/logoff buttons to text links
The login button text is changed to "Login with Google" to make clear it's apparent to the user that a Google account is required for login.

ArunJose commented 3 years ago

@snrelghgub Thanks for the feedback. I replaced "with Google" with a Google logo. There's still some issue with styling of auth links inside Hamburger menu.

snrelghgub commented 3 years ago

@ArunJose nice work dude! You found the best substitution for 'with Google' in my opinion. I found out recently that displaying the Google Logo is also common policy & practice for apps which makes use of Google APIs like ours, and by displaying the Google Logo you are doing exactly that 👍