Skip to content

PONG Software #29

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

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

PONG Software #29

wants to merge 10 commits into from

Conversation

suraj-adewale
Copy link

@suraj-adewale suraj-adewale commented Jul 3, 2025

Submitter checklist:

  • My changes follow the guidelines in the README.md.
  • My changes appear correctly in the preview link (if applicable).

Reviewer checklist:

  • The changes follow the guidelines in the README.md.
  • The changes appear correctly in the preview link (if applicable).

Copy link

netlify bot commented Jul 3, 2025

PR Preview ready! Add ?print to the link below to preview print images

Name Link
🔨 Latest commit baa7718
🔍 Latest deploy log https://app.netlify.com/projects/wall-of-software/deploys/6866ca012b267e0008562957
😎 Deploy Preview https://deploy-preview-29--wall-of-software.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 project configuration.

@suraj-adewale
Copy link
Author

The preview appears very well

Copy link
Author

@suraj-adewale suraj-adewale left a comment

Choose a reason for hiding this comment

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

Updating images

Copy link
Author

@suraj-adewale suraj-adewale left a comment

Choose a reason for hiding this comment

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

Updating images

Copy link
Author

@suraj-adewale suraj-adewale left a comment

Choose a reason for hiding this comment

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

add lab logo

Copy link
Author

@suraj-adewale suraj-adewale left a comment

Choose a reason for hiding this comment

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

add lab logo

Copy link
Author

@suraj-adewale suraj-adewale left a comment

Choose a reason for hiding this comment

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

add lab logo

@vincerubinetti
Copy link
Contributor

vincerubinetti commented Jul 14, 2025

Sorry for the delay in reviewing. This looks good, but aesthetically I think you should make the background of your lab logo either all white or all transparent. Probably easier to just fill it in with white. Or you use a tool like Photopea to magic erase the chunks of white background.

If you're okay with this, let me know and I can just do it for you.

Screenshot 2025-07-14 at 11 56 09 AM

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