-
Notifications
You must be signed in to change notification settings - Fork 256
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
📝[Documentation Update]: Modification of Readme.md file #411
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job, @Anjaliavv51! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project.We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines
@varshith257 please review the Changes. |
@Anjaliavv51 This is good but README (Markdown files) doesn't;t embed custom styles. See this:
This part of the code mayn't compile and appear as code in README |
@varshith257 should i remove it? |
Better embed the font styles in separate section i.e.NOTE section header and put the code of <style></style> naming as used fonts |
Then how the font can be used here as described in issue |
Can i write the Readme.md file code in HTML form |
? |
I think Readme files could be better in the Markdown files :) |
I will solve this |
Ok
…On Sun, 2 Jun, 2024, 7:01 pm Lakshmi Pavananjali, ***@***.***> wrote:
I think Readme files could be better in the Markdown files :)
I will solve this
—
Reply to this email directly, view it on GitHub
<#411 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A4BF3HDORWVGA2DSDPNEX53ZFMNDXAVCNFSM6AAAAABIU5IQ4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTHA2TKNBWGY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***
com>
|
It is edited @varshith257 |
@varshith257 please review the PR ,changes are made |
One screenshot isn't loaded check it once in view file |
@varshith257 now u can check i made changes in that. |
@varshith257 review the PR |
@varshith257 what happened? |
Make the last line into center |
@varshith257 Now is it okay? |
@varshith257 i changed it please check |
|
||
</div> | ||
<h3 style="text-align:center;font-size:2rem;"> | ||
If you find this project helpful, please consider giving it a star! <img src="https://raw.githubusercontent.com/Tarikul-Islam-Anik/tarikul-islam-anik/main/assets/images/Star.png" width="30" height="30"></p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you find this project helpful, please consider giving it a star! <img src="https://raw.githubusercontent.com/Tarikul-Islam-Anik/tarikul-islam-anik/main/assets/images/Star.png" width="30" height="30"></p> | |
If you find this project helpful, please consider giving it a star! <img src="https://raw.githubusercontent.com/Tarikul-Islam-Anik/tarikul-islam-anik/main/assets/images/Star.png" width="30" height="30"></p> |
Can you make this line to align in center?
@varshith257 is it okay now? |
🎉 Your pull request has been successfully merged! 🎉 Thank you for your contribution to our project. Your efforts are greatly appreciated. Keep up the fantastic work! 🚀 |
@Anjaliavv51 Thanks |
@varshith257 added the labels? |
Related Issues
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]
Type of PR
Screenshots / videos (if applicable)
Checklist
main
branch before making this PR