Skip to content

FEAT: add link to OSSPH logo#54

Merged
jofftiquez merged 2 commits intoOSSPhilippines:mainfrom
AbbeyIT:AbbeyIT_Feature_Link-to-OSS
Jan 9, 2023
Merged

FEAT: add link to OSSPH logo#54
jofftiquez merged 2 commits intoOSSPhilippines:mainfrom
AbbeyIT:AbbeyIT_Feature_Link-to-OSS

Conversation

@AbbeyIT
Copy link
Contributor

@AbbeyIT AbbeyIT commented Jan 9, 2023

Fixes

Issue #53

Details

On the homepage of Freefolio, I added the OSSPH link in the logo.

index.html Outdated
<header class="bg-violet-500">
<div class="container mx-auto px-5 py-8 flex flex-row justify-between">
<img class="h-14" src="./images/osslogo-text-white.png" alt="">
<a href="https://ossph.org/" target="_blank"><img class="h-14" src="./images/osslogo-text-white.png" alt=""></a>
Copy link
Member

Choose a reason for hiding this comment

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

Hi, all good. I just have one last request. Please add "OSSPH Logo" as alt in the image. Thanks.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done. Kindly please review it, Thank you.

@jofftiquez
Copy link
Member

Thank you for this @AbbeyIT. Just one last request in the comments. Thank you <3

Copy link
Member

@jofftiquez jofftiquez left a comment

Choose a reason for hiding this comment

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

Perfect! Thanks!

@jofftiquez jofftiquez merged commit 66ed79f into OSSPhilippines:main Jan 9, 2023
@AbbeyIT
Copy link
Contributor Author

AbbeyIT commented Jan 9, 2023

Thank you @jofftiquez

@AbbeyIT AbbeyIT deleted the AbbeyIT_Feature_Link-to-OSS branch January 9, 2023 02:57
@mpotane mpotane mentioned this pull request Jan 30, 2023
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