two-brothers / overcooked-cms

0 stars 0 forks source link

The sign-in link should be an icon #18

Closed nikeshnazareth closed 5 years ago

nikeshnazareth commented 5 years ago

Or at least a different colour. The default purple is hard to see against the blue app bar.

nikeshnazareth commented 5 years ago

I have styled the SignIn link to match the SignOut button. I did this by hardcoding the styles that get assigned to the button on my desktop. Oddly, the sizes don't seem to match up correctly.

This may cause inconsistencies with other devices/browsers. It also means we will need to change the link styles whenever the button styles are changed.

We also don't get any of the hover styles (but I could add them manually too).

I suspect there is a better way. @bigfriedicecream: any ideas?

Note: we can't just use a button by the link has to bypass the React-Router (we're navigating to the /auth route on the server). If there is a way to do that with a button onClick event, that would be ideal. Otherwise, is there a way to embed an tag inside a button such that the button click triggers the link.

nikeshnazareth commented 5 years ago

Perhaps we should just use the person icon to indicate "sign out" and the person_outline icon to indicate "sign in"

nikeshnazareth commented 5 years ago

Aha, obviously we can just put the button inside the link to ensure we get the same styles on the text.

That was embarrassing