Added player to navbar - Githubissues

CSI-280 / MusicGames

0 stars 0 forks source link

Added player to navbar #44

Closed JakeCapra closed 4 years ago

JakeCapra commented 4 years ago

Add player to navbar and re-styled navbar. This resolves #36 .

JakeCapra commented 4 years ago

this has been sitting here for a few days, maybe if I change the reviewers someone will approve it.

jbuzzell commented 4 years ago

i'll give it a look tonight, sorry i haven't gotten around to it. i'll be more prompt with review in the future

JakeCapra commented 4 years ago

don't worry about it, i think we all got a lot going on rn

jbuzzell commented 4 years ago

looks good to me. nice song choice. did you add the hover states for the links? the transitions look good, but i'd suggest we go with the orange (or even just white) for the hover color.

JakeCapra commented 4 years ago

I changed it to white

JakeCapra commented 4 years ago

@thomasszp what do you think?