Skip to content

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

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

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

Conversation

@abxijth
Copy link

@abxijth abxijth commented Jan 8, 2026

No description provided.

@vercel
Copy link

vercel bot commented Jan 8, 2026

@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
Copy link
Member

@hrideshmg hrideshmg left a comment

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.

@@ -1,4 +1,4 @@
<!DOCTYPE html>
<DOCTYPE html>
Copy link
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
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 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>
Copy link
Member

@hrideshmg hrideshmg Jan 13, 2026

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