Comments (7)
Hi, which components do you want to transform into functional components with hooks?
from home.
Hi @yonis9
Thank you for taking interest. I'm thinking of migrating all the components to hooks. This will make the codebase look more clean. I'll appreciate your thoughts on it.
Best,
Hashir
from home.
I don't think it's mandatory but we can do it.
In addition, maybe I didn't fully understand, but I think you misusing the state in some components, for example in the AboutMe.jsx:
this.state = {
heading: "About me",
aboutDev: Configs.aboutDev,
instaProfilePic: '',
showInsta: Configs.showInstaProfilePic,
resumeURL: Pdf
};
}`
I think you can get rid of the "heading" in the state since it never gets changed and affects the component, unless you want to reuse this component for other purposes with the same logic.
If you need any help I will be happy to take part.
from home.
Yes @yonis9. I did this on purpose. So that I did not have to go into the code the change the text. Simply i can change this from state.
from home.
OK no problem. So I will convert it to use hooks tomorrow, is it fine?
from home.
Yeah Sure.
from home.
The purpose is to increase the readability of code. By:
- removing unnecessary things,
- changing components to hooks
- reengineering complex logic to simple.
And feel free to share your ideas.
from home.
Related Issues (20)
- Uneven boxes HOT 1
- Deployment issue HOT 1
- sh: react-scripts: command not found HOT 4
- Make recent projects card height consistent HOT 1
- Add Preview thumbnails to recent projects cards HOT 2
- Add all of your technical skills that you currently know under the skills section HOT 1
- Unable to update the website after a few months of deployment HOT 7
- hi HOT 2
- Failed to compile 'jumbotron', this is depreciated as of bootstrap 5? HOT 4
- [Migration] React 16 -> 18
- error due to build HOT 2
- Primeiro issue
- Migrate to node v18 HOT 1
- Could you please help me with this error? HOT 4
- On the main screen, the particle effect is not visible on the left side. Zoom in to see the error HOT 1
- Fix Auto-Closing of Navbar Menu on Navigation Item Selection
- Migrate to node v20
- Only main page with social icons and resume/blog buttons HOT 1
- Blog section HOT 10
- Education Section missing HOT 3
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from home.