Skip to content

Commit 66e851e

Browse files
committed
fix: address review feedback (logo update, chrome button bg, social icon spacing)
1 parent 9b4a5a1 commit 66e851e

3 files changed

Lines changed: 37 additions & 15 deletions

File tree

app/[slug]/page.tsx

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ import { useTheme } from "@/app/contexts/themeContext";
2626
const Page = ({ params: { slug } }: { params: { slug: string } }) => {
2727
const [browser, setBrowser] = useState("chrome");
2828
const { isLightTheme } = useTheme();
29+
const logoSrc = isLightTheme ? "/images/bb-logo-light.svg" : "/images/bb-logo.svg";
2930

3031
useEffect(() => {
3132
setBrowser(detectBrowser());
@@ -67,7 +68,7 @@ const Page = ({ params: { slug } }: { params: { slug: string } }) => {
6768
{/* Left Section */}
6869
<div className="text-left flex-1">
6970
<Image
70-
src="/images/bb-logo.svg"
71+
src={logoSrc}
7172
alt="BetterBugs Logo"
7273
className="mb-8"
7374
width={200}
@@ -88,7 +89,9 @@ const Page = ({ params: { slug } }: { params: { slug: string } }) => {
8889
</p>
8990
<div className="flex items-center gap-3 mt-6 mb-4">
9091
<Link target="_blank" href={Extension_URL}>
91-
<Button className="btn-primary flex items-center z-20 justify-center gap-3 rounded-full px-4">
92+
<Button
93+
className={`btn-primary flex items-center z-20 justify-center gap-3 rounded-full px-4 ${DevelopmentToolsStyles.addToChromeButton}`}
94+
>
9295
{browser === "edge" ? (
9396
<EdgeIcon className="w-7 h-7" />
9497
) : (
@@ -891,7 +894,7 @@ const Page = ({ params: { slug } }: { params: { slug: string } }) => {
891894
{/* Left Section */}
892895
<div className="text-left flex-1">
893896
<Image
894-
src="/images/bb-logo.svg"
897+
src={logoSrc}
895898
alt="BetterBugs Logo"
896899
className="mb-8"
897900
width={200}
@@ -912,7 +915,9 @@ const Page = ({ params: { slug } }: { params: { slug: string } }) => {
912915
</p>
913916
<div className="flex items-center gap-3 mt-6 mb-4">
914917
<Link target="_blank" href={Extension_URL}>
915-
<Button className="btn-primary flex items-center z-20 justify-center gap-3 rounded-full px-4">
918+
<Button
919+
className={`btn-primary flex items-center z-20 justify-center gap-3 rounded-full px-4 ${DevelopmentToolsStyles.addToChromeButton}`}
920+
>
916921
{browser === "edge" ? (
917922
<EdgeIcon className="w-7 h-7" />
918923
) : (

app/components/layout/footer/footerComponent.tsx

Lines changed: 23 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -118,6 +118,18 @@ const FooterComponent = () => {
118118
},
119119
];
120120

121+
const socialIconWrapperClass = isLightTheme
122+
? 'w-9 h-9 p-2 rounded-xl flex items-center justify-center cursor-pointer bg-transparent border-none'
123+
: 'w-9 h-9 p-2 rounded-xl flex items-center justify-center cursor-pointer bbFooterSocialIcon';
124+
125+
const socialIconWrapperWideClass = isLightTheme
126+
? 'w-10 h-10 p-2 rounded-xl flex items-center justify-center cursor-pointer bg-transparent border-none'
127+
: 'w-10 h-10 p-2 rounded-xl flex items-center justify-center cursor-pointer bbFooterSocialIcon';
128+
129+
const socialIconClass = isLightTheme
130+
? 'text-[#6b7280] hover:text-[#10b981]'
131+
: 'text-white/60 hover:text-white';
132+
121133
// const PartnershipMenu = [
122134
// { label: "For agencies", url: "/" },
123135
// { label: "For enterprise", url: "/" },
@@ -269,50 +281,50 @@ const FooterComponent = () => {
269281
priority
270282
/>
271283
</Link>
272-
<div className="flex mt-1 lg:mt-5">
284+
<div className="flex mt-1 lg:mt-5 gap-3">
273285
<Link
274286
href="https://www.linkedin.com/company/betterbugs/"
275287
target="_blank"
276288
aria-label="LinkedIn"
277289
>
278-
<div className="w-9 h-9 p-2 rounded-xl flex items-center justify-center cursor-pointer bbFooterSocialIcon">
279-
<LinkedinIcon className="text-white/60 hover:text-white" />
290+
<div className={socialIconWrapperClass}>
291+
<LinkedinIcon className={socialIconClass} />
280292
</div>
281293
</Link>
282294
<Link
283295
href="https://twitter.com/BetterBugs"
284296
target="_blank"
285297
aria-label="Twitter"
286298
>
287-
<div className="w-9 h-9 p-2 rounded-xl flex items-center justify-center cursor-pointer bbFooterSocialIcon">
288-
<TwitterIcon className="text-white/60 hover:text-white" />
299+
<div className={socialIconWrapperClass}>
300+
<TwitterIcon className={socialIconClass} />
289301
</div>
290302
</Link>
291303
<Link
292304
href="https://www.instagram.com/betterbugshq/"
293305
target="_blank"
294306
aria-label="Instagram"
295307
>
296-
<div className="w-9 h-9 p-2 rounded-xl flex items-center justify-center cursor-pointer bbFooterSocialIcon">
297-
<InstagramIcon className="text-white/60 hover:text-white" />
308+
<div className={socialIconWrapperClass}>
309+
<InstagramIcon className={socialIconClass} />
298310
</div>
299311
</Link>
300312
<Link
301313
href="https://www.youtube.com/@betterbugshq"
302314
target="_blank"
303315
aria-label="LinkedIn"
304316
>
305-
<div className="w-10 h-10 p-2 rounded-xl flex items-center justify-center cursor-pointer bbFooterSocialIcon">
306-
<YouTubeIcon className="text-white/60 hover:text-white !w-12 !h-12" />
317+
<div className={socialIconWrapperWideClass}>
318+
<YouTubeIcon className={`${socialIconClass} !w-12 !h-12`} />
307319
</div>
308320
</Link>
309321
<Link
310322
href="https://discord.com/invite/HF8XjwVtPh"
311323
target="_blank"
312324
aria-label="Discord"
313325
>
314-
<div className="w-10 h-10 p-2 rounded-xl flex items-center justify-center cursor-pointer bbFooterSocialIcon">
315-
<DiscordIcon className="text-white/60 hover:text-white !w-12 !h-12" />
326+
<div className={socialIconWrapperWideClass}>
327+
<DiscordIcon className={`${socialIconClass} !w-12 !h-12`} />
316328
</div>
317329
</Link>
318330
</div>

app/developmentToolsStyles.module.scss

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -244,6 +244,11 @@
244244
background: #00d0ff9a;
245245
}
246246

247+
.addToChromeButton {
248+
background: #000000;
249+
color: #ffffff;
250+
}
251+
247252
/* Webkit custom scrollbar styles */
248253
.scrollbar {
249254
overflow: auto;

0 commit comments

Comments
 (0)