mightyjoew/opensourcedevs

Open Source Devs - An open-source contribution project for newcomers.

opensourcedevs's Issues

`reach/router` dependency conflict

@MightyJoeW
I got a dependency conflict, when first tried to install this project. I am using npm v7 along. When I tried npm i, it gave me a dependency conflict with reach/router. I digged in a little bit, and found that reach/router is not being maintained as of now.
storybookjs/storybook#14619

Also, since this project is pretty new, switching to react-router would make a lot more sense.
Thanks!

Add unit tests for components

Add unit tests for components (preferably with Jest and React Testing Library)

Add icons for social buttons

This is an enhancement. I want to replace the text with their social icons, it will give a good look.
Please assign this issue to me.

Create Open Source Devs logo for navbar

The Open Source Devs text in the navbar gets stacked in mobile view. Would look better with a logo (or maybe OSD with carets around it)

Prevent card photo from being clickable on individual user page

Clicking on photo on user page routes to error page

(JS-0463) Consider adding `propTypes` to the React component definition

Description

It is recommended to always add propTypes for a component as it avoids runtime errors. Using propTypes ensures valid properties being passed as props or being used as props.

Occurrences

There are 12 occurrences of this issue in the repository.

See all occurrences on DeepSource → deepsource.io/gh/MightyJoeW/OpenSourceDevs/issue/JS-0463/occurrences/

Resolve warning <button> cannot appear as a descendant of <button>.

Search Bar UI

The search bar ui after search has blue old html vibes, we can add line dividers and links to make it look better.

(JS-0128) Should not have unused variables

Description

Found variables that are declared but not used anywhere.

Occurrences

There is 1 occurrence of this issue in the repository.

See all occurrences on DeepSource → deepsource.io/gh/MightyJoeW/OpenSourceDevs/issue/JS-0128/occurrences/

Add a scroll to top feature on the home page

Error with <p> elements inside components/about.js

Reason to fix this: https://stackoverflow.com/questions/41928567/div-cannot-appear-as-a-descendant-of-p

Fix navbar search on mobile view

• On mobile view, the search bar extends beyond the navbar.
• On mobile view, the search bar covers the About link

(JS-0454) Found invalid characters in markup

Description

The issue is raised because of missing or unescaped JSX escape characters. These characters would be injected as a text node in JSX statements, which might not be intentional. These won't throw any syntax or runtime errors, but the rendered output will not be the same as expected.

Occurrences

There are 2 occurrences of this issue in the repository.

See all occurrences on DeepSource → deepsource.io/gh/MightyJoeW/OpenSourceDevs/issue/JS-0454/occurrences/

Create toggle for dark mode

Style the error page

Currently, error.jsx only has a div that says Error. Add styling and a message indicating that the user tried to hit a page that doesn't exist. Also, include a link to route back to home.

Swap footer hr with MUI Divider

The MUI Divider is lighter and comes with some handy props that'll make customizing it easier if desired in the future.

You'll essentially do the following:

import Divider from '@mui/material/Divider';

<Divider sx={{ margin: '10px 0' }} />

If you want to see its full list of props, check here https://mui.com/api/divider/.

Change appbar to black