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

Website logo #62

Open
vedikaag99 opened this issue Jan 20, 2021 · 7 comments · May be fixed by #67
Open

Website logo #62

vedikaag99 opened this issue Jan 20, 2021 · 7 comments · May be fixed by #67
Assignees

Comments

@vedikaag99
Copy link

Description

I have made a logo for the website. The preview of the same is provided below. Kindly assign me this issue so that I can make a PR and work on it under DWoC. @BALaka-18

For reference

Natural   Organic Logo Template with Hand And Leaves
Please review.

@github-actions
Copy link

This is your first issue for this repository ! Enjoy contributing : )

@kenkirito
Copy link

@BALaka-18 please review this

@BALaka-18
Copy link
Owner

Looks good, just change the color scheme to blue + yellow (coz they're the official colors of Python Software Foundation, so the logo will look better).
And remove the Py from the page like figure in the icon, keep it blank.

@vedikaag99
Copy link
Author

Natural   Organic Logo Template with Hand And Leaves (1)
@BALaka-18 I have made the required changes. Kindly review now.

@BALaka-18
Copy link
Owner

@vedikaag99 looks good

@vedikaag99
Copy link
Author

@BALaka-18 can I open a PR for the same then?

@BALaka-18
Copy link
Owner

@vedikaag99 do one thing, add the logo in between the two grey lines just above the ' ABOUT THIS PROJECT ' heading in the README file. Make the PR directly to the master branch, I'll merge it directly.

@vedikaag99 vedikaag99 linked a pull request Mar 8, 2021 that will close this issue
12 tasks
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 a pull request may close this issue.

3 participants