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.

Sign up for GitHub

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

Jump to bottom

fixed images #121

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

fixed images #121

wants to merge 4 commits into from

Conversation

musavveer
Copy link

Issue: #105
image

@ananya0504
Copy link
Contributor

ananya0504 commented Oct 20, 2021 •
edited
Loading

Could you please change the logo for Flutter? You can use the official Flutter framework logo, thanks.

ananya0504
ananya0504 requested changes Oct 20, 2021
View reviewed changes
Copy link
Contributor

@ananya0504 ananya0504 left a comment

Choose a reason for hiding this comment

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

Also, rather than using images directly from the web, please download them, make a folder called Images and then use them.

@musavveer
Copy link
Author

yeah sure I'll fix this

@musavveer
Copy link
Author

@ananya0504 Please do look into the changes made

@musavveer
Copy link
Author

@ananya0504 @priithaa Please do look into the changes made and accept my PR

@ananya0504
Copy link
Contributor

Hi @musavveer
Has the image for Flutter been changed?

@ananya0504
Copy link
Contributor

The code looks fine, I request you to pls upload a screenshot of the page with the changed image.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ananya0504 ananya0504 ananya0504 requested changes

Assignees
No one assigned
Labels
requested-change
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@musavveer @ananya0504