Skip to content

Commit 892abe5

Browse files
committed
Code cleanup
1 parent 7506b64 commit 892abe5

File tree

5 files changed

+16
-38
lines changed

5 files changed

+16
-38
lines changed

src/components/HireMeModal.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ const HireMeModal = ({ onClose, onRequest }) => {
1717
exit={{ opacity: 0 }}
1818
className="font-general-medium fixed inset-0 z-30 transition-all duration-500"
1919
>
20-
{/* Modal Background */}
20+
{/* Modal Backdrop */}
2121
<div className="bg-filter bg-black bg-opacity-50 fixed inset-0 w-full h-full z-20"></div>
2222

2323
{/* Modal Content */}

src/components/about/AboutMeBio.js

-4
Original file line numberDiff line numberDiff line change
@@ -7,13 +7,10 @@ const AboutMeBio = () => {
77

88
return (
99
<div className="block sm:flex sm:gap-10 mt-10 sm:mt-20">
10-
{/* About me portfolio image start */}
1110
<div className="w-full sm:w-1/4 mb-7 sm:mb-0">
1211
<img src={profileImage} className="rounded-lg w-96" alt="" />
1312
</div>
14-
{/* About me portfolio image end */}
1513

16-
{/* About me details start */}
1714
<div className="font-general-regular w-full sm:w-3/4 text-left">
1815
{aboutMe.map((bio) => (
1916
<p
@@ -24,7 +21,6 @@ const AboutMeBio = () => {
2421
</p>
2522
))}
2623
</div>
27-
{/* About me details end */}
2824
</div>
2925
);
3026
};

src/components/projects/ProjectInfo.js

+5-11
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ const ProjectInfo = () => {
77
return (
88
<div className="block sm:flex gap-0 sm:gap-10 mt-14">
99
<div className="w-full sm:w-1/3 text-left">
10-
{/* Single project client details start */}
10+
{/* Single project client details */}
1111
<div className="mb-7">
1212
<p className="font-general-regular text-2xl font-semibold text-secondary-dark dark:text-secondary-light mb-2">
1313
{singleProjectData.ProjectInfo.ClientHeading}
@@ -39,9 +39,8 @@ const ProjectInfo = () => {
3939
)}
4040
</ul>
4141
</div>
42-
{/* Single project client details end */}
4342

44-
{/* Single project objectives start */}
43+
{/* Single project objectives */}
4544
<div className="mb-7">
4645
<p className="font-general-regular text-2xl font-semibold text-ternary-dark dark:text-ternary-light mb-2">
4746
{singleProjectData.ProjectInfo.ObjectivesHeading}
@@ -50,9 +49,8 @@ const ProjectInfo = () => {
5049
{singleProjectData.ProjectInfo.ObjectivesDetails}
5150
</p>
5251
</div>
53-
{/* Single project objectives end */}
5452

55-
{/* Single project technologies start */}
53+
{/* Single project technologies */}
5654
<div className="mb-7">
5755
<p className="font-general-regular text-2xl font-semibold text-ternary-dark dark:text-ternary-light mb-2">
5856
{singleProjectData.ProjectInfo.Technologies[0].title}
@@ -63,9 +61,8 @@ const ProjectInfo = () => {
6361
)}
6462
</p>
6563
</div>
66-
{/* Single project technologies end */}
6764

68-
{/* Single project social sharing start */}
65+
{/* Single project social sharing */}
6966
<div>
7067
<p className="font-general-regular text-2xl font-semibold text-ternary-dark dark:text-ternary-light mb-2">
7168
{singleProjectData.ProjectInfo.SocialSharingHeading}
@@ -90,11 +87,9 @@ const ProjectInfo = () => {
9087
)}
9188
</div>
9289
</div>
93-
{/* Single project social sharing end */}
9490
</div>
95-
{/* Single project left section details end */}
9691

97-
{/* Single project right section details start */}
92+
{/* Single project right section */}
9893
<div className="w-full sm:w-2/3 text-left mt-10 sm:mt-0">
9994
<p className="font-general-regular text-primary-dark dark:text-primary-light text-2xl font-bold mb-7">
10095
{singleProjectData.ProjectInfo.ProjectDetailsHeading}
@@ -110,7 +105,6 @@ const ProjectInfo = () => {
110105
);
111106
})}
112107
</div>
113-
{/* Single project right section details end */}
114108
</div>
115109
);
116110
};

src/components/shared/AppFooter.js

+1-2
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ const AppFooter = () => {
2828
return (
2929
<div className="container mx-auto">
3030
<div className="pt-20 sm:pt-30 pb-8 mt-20 border-t-2 border-primary-light dark:border-secondary-dark">
31-
{/* Footer social links start */}
31+
{/* Footer social links */}
3232
<div className="font-general-regular flex flex-col justify-center items-center mb-12 sm:mb-28">
3333
<p className="text-3xl sm:text-4xl text-primary-dark dark:text-primary-light mb-5">
3434
Follow me
@@ -49,7 +49,6 @@ const AppFooter = () => {
4949
</ul>
5050
</div>
5151

52-
{/* Footer copyright */}
5352
<AppFooterCopyright />
5453
</div>
5554
</div>

src/components/shared/AppHeader.js

+9-20
Original file line numberDiff line numberDiff line change
@@ -41,11 +41,9 @@ const AppHeader = () => {
4141
id="nav"
4242
className="sm:container sm:mx-auto"
4343
>
44-
{/* Header start */}
4544
<div className="z-10 max-w-screen-lg xl:max-w-screen-xl block sm:flex sm:justify-between sm:items-center py-6">
46-
{/* Header menu links and small screen hamburger menu start */}
45+
{/* Header menu links and small screen hamburger menu */}
4746
<div className="flex justify-between items-center px-4 sm:px-0">
48-
{/* Logo start */}
4947
<div>
5048
<Link to="/">
5149
{activeTheme === 'dark' ? (
@@ -63,9 +61,8 @@ const AppHeader = () => {
6361
)}
6462
</Link>
6563
</div>
66-
{/* Logo end */}
6764

68-
{/* Theme switcher small screen start */}
65+
{/* Theme switcher small screen */}
6966
<div
7067
onClick={() => setTheme(activeTheme)}
7168
aria-label="Theme Switcher"
@@ -77,9 +74,8 @@ const AppHeader = () => {
7774
<FiSun className="text-gray-200 hover:text-gray-50 text-xl" />
7875
)}
7976
</div>
80-
{/* Theme switcher small screen end */}
8177

82-
{/* Small screen hamburger menu start */}
78+
{/* Small screen hamburger menu */}
8379
<div className="sm:hidden">
8480
<button
8581
onClick={toggleMenu}
@@ -100,11 +96,9 @@ const AppHeader = () => {
10096
</svg>
10197
</button>
10298
</div>
103-
{/* Small screen hamburger menu end */}
10499
</div>
105-
{/* Header menu links and small screen hamburger menu end */}
106100

107-
{/* Header links small screen start*/}
101+
{/* Header links small screen */}
108102
<div
109103
className={
110104
showMenu
@@ -142,9 +136,8 @@ const AppHeader = () => {
142136
</button>
143137
</div>
144138
</div>
145-
{/* Header links small screen end */}
146139

147-
{/* Header links large screen start*/}
140+
{/* Header links large screen */}
148141
<div className="font-general-medium hidden m-0 sm:ml-4 mt-5 sm:mt-3 sm:flex p-5 sm:p-0 justify-center items-center shadow-lg sm:shadow-none">
149142
<Link
150143
to="/projects"
@@ -168,11 +161,10 @@ const AppHeader = () => {
168161
Contact
169162
</Link>
170163
</div>
171-
{/* Header links small screen end */}
172164

173-
{/* Header right section buttons start */}
165+
{/* Header right section buttons */}
174166
<div className="hidden sm:flex justify-between items-center flex-col md:flex-row">
175-
{/* Hire me button start */}
167+
{/* Hire me button */}
176168
<div className="hidden md:flex">
177169
<button
178170
onClick={showHireMeModal}
@@ -182,9 +174,8 @@ const AppHeader = () => {
182174
Hire Me
183175
</button>
184176
</div>
185-
{/* Hire me button end */}
186177

187-
{/* Theme switcher large screen start */}
178+
{/* Theme switcher large screen */}
188179
<div
189180
onClick={() => setTheme(activeTheme)}
190181
aria-label="Theme Switcher"
@@ -196,10 +187,9 @@ const AppHeader = () => {
196187
<FiSun className="text-gray-200 hover:text-gray-50 text-xl" />
197188
)}
198189
</div>
199-
{/* Theme switcher large screen start */}
200190
</div>
201191
</div>
202-
{/* Hire me modal start */}
192+
{/* Hire me modal */}
203193
<div>
204194
{showModal ? (
205195
<HireMeModal
@@ -209,7 +199,6 @@ const AppHeader = () => {
209199
) : null}
210200
{showModal ? showHireMeModal : null}
211201
</div>
212-
{/* Hire me modal end */}
213202
</motion.nav>
214203
);
215204
};

0 commit comments

Comments
 (0)