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

Issue #25 : Created the Greetings Page #146

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

LEARNER-dakshesh
Copy link

I have added Greetings page in the home page where the user can see different types of expressions and listen to its pronunciation . I haven't got some of the specific images like for good morning and good night because i was unable to find image through which user can relate to.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Welcome to Our repository.🎊 Thank you so much for taking the time to point this out.

@sapatevaibhav
Copy link
Member

Bro, can you make it lighter by using SVG files? The app size is increasing drastically. You can simply convert image files to SVGs if a proper SVG image is not found.

@LEARNER-dakshesh
Copy link
Author

I have converted the images to svg . Could you please check it

@LEARNER-dakshesh
Copy link
Author

Bro @sapatevaibhav i have submitted the updated code . Could you please merge it .

@sapatevaibhav
Copy link
Member

please resolve the conflicts

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