make general fixes

[christopher-alba]

Description

• add appropriate links to pages in post preview component
• update readme to include netlify deploy link and Qunzhi Zhou in contributions
• other general fixes that are deemed necessary
• include useEffect hooks to set scroll position to (0,0) on page load on multiple pages

  Related Issue

resolves #53

Type of change

• [x] New feature (enhancement)
• [x] Bug fix
• [x] Refactor
• [ ] Breaking change
• [x] Documentation

How Has This Been Tested?

• [ ] Automated testing
• [x] Manual testing

Checklist:

• [x] Does a similar (open or closed) pull request not already exist?
• [x] Is the pull request head repository a fork repository?
• [x] Is the pull request compare branch a development branch?
• [x] Does the code build without new warnings?
• [x] Has testing been performed that proves changes are effective and work?
• [ ] Has a self- and/or peer-review of the code been performed?
• [x] Is the person responsible for the repository assigned to the pull request?
• [x] Is the pull request linked to a project?
• [x] Is the pull request linked to a milestone?

For more information, refer to the Contributing Guidelines and Code of Conduct links at the bottom of this page.

[netlify[bot]]

✅ Deploy Preview for uniforum ready!

🔨 Explore the source changes: fdb853028addb79a0c43eddb545fccdaa8bf5a2b

🔍 Inspect the deploy log: https://app.netlify.com/sites/uniforum/deploys/6236c92d9678b80008a8b743

😎 Browse the preview: https://deploy-preview-54--uniforum.netlify.app/

[hpar461]

Hi, love the changes!

Perhaps one potential UI fix would be on the post preview components, where I've noticed that a link icon? has been appearing on the right side of the title. Perhaps this should be removed following the ui prototype design? Final decision is up to you.

[christopher-alba]

Hi, love the changes!

Perhaps one potential UI fix would be on the post preview components, where I've noticed that a link icon? has been appearing on the right side of the title. Perhaps this should be removed following the ui prototype design? Final decision is up to you.

Yup that's because the title is now also a link to the particular post (based on its ID) so i felt like it needed a call to action for people to realise they can click on it

[Xiaoqi510]

Looks all good now, but just one potential ui issue at the bottom of the communities page, i think we can add . padding-bottom:20px for the class com-bodyContainer in the communities.css.

[hpar461]

Hi, love the changes! Perhaps one potential UI fix would be on the post preview components, where I've noticed that a link icon? has been appearing on the right side of the title. Perhaps this should be removed following the ui prototype design? Final decision is up to you.

Yup that's because the title is now also a link to the particular post (based on its ID) so i felt like it needed a call to action for people to realise they can click on it

Right! That's algood then. Thanks!

[christopher-alba]

Looking good, thanks for all the hard work! Just following on from Xiaoqi's comment, since we added padding to the communities page, did we also want to pad the bottom of the home and posts page?

Lets make this an issue for the next team :P