Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixing a bunch of smaller issues and links #105

Merged
merged 8 commits into from
Apr 6, 2020

Conversation

Twissi
Copy link
Contributor

@Twissi Twissi commented Apr 5, 2020

  • fix links Add hover to links  #79
  • fix undefined error for missing venue location (events)
  • fix missing links on homepage
  • update global links styles and adjust all links
  • fix icon for events on city page
  • fix team section name alignment
  • remove copy to suggest an event from the events section title (probably makes more sense to have it below the events as a link)

@vercel
Copy link

vercel bot commented Apr 5, 2020

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/ots-next/websitenext/9lmtjhcu6
✅ Preview: https://websitenext-git-fork-twissi-links-79.ots-next.now.sh

@Twissi Twissi requested a review from gsambrotta April 5, 2020 22:02
Copy link
Member

@gsambrotta gsambrotta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏 👏 👏 👏 Amazing!

I notice there is no active on main navigation links.
We should do it but if you prefer we merge this PR and do that in a new PR. Fine with me

@Twissi
Copy link
Contributor Author

Twissi commented Apr 6, 2020

@gsambrotta created an issue for that #106

@Twissi Twissi merged commit b5359e7 into OpenTechSchool:master Apr 6, 2020
@Twissi Twissi deleted the links-79 branch April 6, 2020 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants