Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed #112 - added names to social icons #136

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AsifQamar
Copy link

added names to social icon for better user experience
Screenshot_175

Copy link

netlify bot commented Oct 13, 2024

Deploy Preview for codeittool ready!

Name Link
🔨 Latest commit 72eddc0
🔍 Latest deploy log https://app.netlify.com/sites/codeittool/deploys/670b5534e5cdbf0008727d93
😎 Deploy Preview https://deploy-preview-136--codeittool.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@AsifQamar
Copy link
Author

@multiverseweb please review and merge it

@AsifQamar
Copy link
Author

@multiverseweb merge it

@AsifQamar
Copy link
Author

@multiverseweb merge it please

Copy link
Owner

@multiverseweb multiverseweb left a comment

Choose a reason for hiding this comment

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

@AsifQamar in the issue you raised, you said you'll arrange the footer section properly. Can you help me know what changes have you made in the arrangement? Also, I don't think adding names to social media icons is necessary. It would look good if names appear on icon hover but adding them permanently like this doesn't look good.

@AsifQamar
Copy link
Author

@AsifQamar in the issue you raised, you said you'll arrange the footer section properly. Can you help me know what changes have you made in the arrangement? Also, I don't think adding names to social media icons is necessary. It would look good if names appear on icon hover but adding them permanently like this doesn't look good.

OK I'll add more features

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

Successfully merging this pull request may close these issues.

2 participants