IndieCoderMM / IndieCoderMM/js-capstone
Peer-to-peer code review
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 4
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
### In this Issue :
Hi @IndieCoderMM ✋
Your project is looking very good and you did a great job but there are some improvements 👍 .
### Highlights!
- Followed Gitflow✔️
- Good commit✔️
- No Linters errors✔️
- Professional readme✔️
- Webpack✔️
### Suggestions
- Nice work Hein, I would suggest you reduce the number of items on your home page also please center your popup window on the screen and resize your popup images. Well done again. 💯
- I think it's better to keep the comments add before.
- I think is better to display the articles in the same order.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No files or tests are named. Start by locating the home-page item list, popup markup and styles, and the comments and articles rendering; done means fewer home-page items, a centered and resized popup, and preserved requested ordering.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, webpack
- Domain
- frontend, web-dev
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100