Skip to content

Commit 3973d24

Browse files
authored
Merge pull request #3142 from rohanjadhav05/rohanjadhav/fixes_#3083
Rohanjadhav - fixes #3083
2 parents 82834fd + aa9a760 commit 3973d24

File tree

3 files changed

+38
-19
lines changed

3 files changed

+38
-19
lines changed

client/components/PreviewNav.jsx

+8-6
Original file line numberDiff line numberDiff line change
@@ -12,12 +12,14 @@ const PreviewNav = ({ owner, project }) => {
1212
<nav className="nav preview-nav">
1313
<div className="nav__items-left">
1414
<div className="nav__item-logo">
15-
<LogoIcon
16-
role="img"
17-
aria-label={t('Common.p5logoARIA')}
18-
focusable="false"
19-
className="svg__logo"
20-
/>
15+
<Link to={`/${owner.username}/sketches`}>
16+
<LogoIcon
17+
role="img"
18+
aria-label={t('Common.p5logoARIA')}
19+
focusable="false"
20+
className="svg__logo"
21+
/>
22+
</Link>
2123
</div>
2224
<Link
2325
className="nav__item"

client/modules/IDE/components/Header/Nav.jsx

+20-7
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ const ProjectMenu = () => {
114114
const isUserOwner = useSelector(getIsUserOwner);
115115
const project = useSelector((state) => state.project);
116116
const user = useSelector((state) => state.user);
117-
117+
const userSketches = `${user.username}/sketches`;
118118
const isUnsaved = !project?.id;
119119

120120
const rootFile = useSelector(selectRootFile);
@@ -137,12 +137,25 @@ const ProjectMenu = () => {
137137
return (
138138
<ul className="nav__items-left">
139139
<li className="nav__item-logo">
140-
<LogoIcon
141-
role="img"
142-
aria-label={t('Common.p5logoARIA')}
143-
focusable="false"
144-
className="svg__logo"
145-
/>
140+
{user && user.username !== undefined ? (
141+
<Link to={userSketches}>
142+
<LogoIcon
143+
role="img"
144+
aria-label={t('Common.p5logoARIA')}
145+
focusable="false"
146+
className="svg__logo"
147+
/>
148+
</Link>
149+
) : (
150+
<a href="https://p5js.org">
151+
<LogoIcon
152+
role="img"
153+
aria-label={t('Common.p5logoARIA')}
154+
focusable="false"
155+
className="svg__logo"
156+
/>
157+
</a>
158+
)}
146159
</li>
147160
<NavDropdownMenu id="file" title={t('Nav.File.Title')}>
148161
<NavMenuItem onClick={newSketch}>{t('Nav.File.New')}</NavMenuItem>

client/modules/IDE/components/Header/__snapshots__/Nav.unit.test.jsx.snap

+10-6
Original file line numberDiff line numberDiff line change
@@ -473,12 +473,16 @@ exports[`Nav renders editor version for desktop 1`] = `
473473
<li
474474
class="nav__item-logo"
475475
>
476-
<test-file-stub
477-
aria-label="p5.js Logo"
478-
classname="svg__logo"
479-
focusable="false"
480-
role="img"
481-
/>
476+
<a
477+
href="https://p5js.org"
478+
>
479+
<test-file-stub
480+
aria-label="p5.js Logo"
481+
classname="svg__logo"
482+
focusable="false"
483+
role="img"
484+
/>
485+
</a>
482486
</li>
483487
<li
484488
class="nav__item"

0 commit comments

Comments
 (0)