Skip to content

Fix footer text, update title & favicon, add Ctrl+V image paste support - #1

Open
abxijth wants to merge 4 commits into
amfoss:mainfrom
abxijth:update/IconAndTitle
Open

Fix footer text, update title & favicon, add Ctrl+V image paste support#1
abxijth wants to merge 4 commits into
amfoss:mainfrom
abxijth:update/IconAndTitle

Conversation

@abxijth

@abxijth abxijth commented Jan 8, 2026

Copy link
Copy Markdown

No description provided.

@vercel

vercel Bot commented Jan 8, 2026

Copy link
Copy Markdown

@abxijth is attempting to deploy a commit to the amFOSS Dev Team on Vercel.

A member of the Team first needs to authorize it.

@abxijth abxijth changed the title fix: footer text issue & update: title and favicon fix: footer text issue & update: title and favicon Jan 8, 2026
@abxijth abxijth changed the title fix: footer text issue & update: title and favicon fix: footer text issue, update: title and favicon, feat: add ctrl+v paste option for images Jan 8, 2026
@abxijth abxijth changed the title fix: footer text issue, update: title and favicon, feat: add ctrl+v paste option for images Fix footer text, update title & favicon, add Ctrl+V image paste support Jan 8, 2026

@hrideshmg hrideshmg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @abxijth, thanks for the PR.

The favicon and image paste logic look fine, but I've left some comments regarding the other few changes.

Comment thread public/index.html
@@ -1,4 +1,4 @@
<!DOCTYPE html>
<DOCTYPE html>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why was this changed?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was an accidental change during editing and not intentional. Will fix it in the next commit.

Comment thread src/App.jsx
Comment on lines 40 to +169
@@ -130,9 +141,7 @@ export default function App() {
amFOSS
</a>
</span>

<span className="text-gray-600">•</span>

<a
href="https://github.com/amfoss/"
target="_blank"
@@ -141,17 +150,23 @@ export default function App() {
>
GitHub
</a>
</footer>
<header>
<a href="https://amfoss.in/">
<img
src={logo}
alt="amFOSS logo"
className="fixed top-6 left-6 z-20"
width={170}
/>
</a>
</header>
</footer>

<style>{`
.grid-pattern::before {
content: "";
position: absolute;
inset: 0;
background-image:
linear-gradient(45deg, #1b1b1b 25%, transparent 25%),
linear-gradient(-45deg, #1b1b1b 25%, transparent 25%),
linear-gradient(45deg, transparent 75%, #1b1b1b 75%),
linear-gradient(-45deg, transparent 75%, #1b1b1b 75%);
background-size: 100px 100px;
background-position: 0 0, 0 50px, 50px -50px, -50px 0;
z-index: 0;
}
`}</style>

@hrideshmg hrideshmg Jan 13, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you explain what this change does as well as why we're using useRef now? I'm unable to see any differences with the footer -

Before:
Image

After:
Image

It has also changed the colors and the text alignment a bit, I'm not sure if this is intentional but I prefer the old look more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants