Skip to content

Replaced i and b tags with em and strong as per HTML 5 Standard#9

Open
noopur-tiwari wants to merge 1 commit into
hrishi7:masterfrom
noopur-tiwari:new-branh
Open

Replaced i and b tags with em and strong as per HTML 5 Standard#9
noopur-tiwari wants to merge 1 commit into
hrishi7:masterfrom
noopur-tiwari:new-branh

Conversation

@noopur-tiwari
Copy link
Copy Markdown

@noopur-tiwari noopur-tiwari commented Oct 19, 2019

Apart from replacing i and b tags with em and strong tags respectively, I have also indented the code in Visual Studio Code, thus it is showing almost the entire code as modified which is not the case.

@hrishi7
Copy link
Copy Markdown
Owner

hrishi7 commented Jul 28, 2020

Hey noopur-tiwari, the strong tag has no issue but the things where you changed the i tag with em that is not correct
in all those places i tage is used for icons(fontawesome) you could update that with span tag

@Vamsi045
Copy link
Copy Markdown

Hi I am new to open source community.How do i contribute to your work. i have forked and made changes replacinng i tags with span tags and pushed it. I want to know more. could you please help me

@hrishi7
Copy link
Copy Markdown
Owner

hrishi7 commented Jul 30, 2023

Hi I am new to open source community.How do i contribute to your work. i have forked and made changes replacinng i tags with span tags and pushed it. I want to know more. could you please help me

sure you can first resolve all teh conflicts for this PR and push it. i will review & merge it

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.

3 participants