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

Fixed a few errors in the code markup. #342

Open
wants to merge 3 commits into
base: gh-pages
Choose a base branch
from

Conversation

mdqst
Copy link

@mdqst mdqst commented Nov 6, 2024

Description

faq.html

Closing </script> tag in the <head> section
The </script> closing tag here is unnecessary, as this is a stylesheet link, not a script.

Corrected.

guide.html

The tag for Bootstrap CSS is incorrectly closed with </script>. It should not have a closing tag at all, or it should be closed as a self-closing

Corrected.

support.html

Incorrect Closing Tag for h1:

In the header section, <h1 class="font-weight-light text-secondary mb-2"><span>Support Dogecoin</span><h1> should be closed with </h1>, not <h1>.

Corrected.

Motivation and Context

Nothing critical, but I love Doge, so I want the code to be clean and correct.

How Has This Been Tested?

I identified and corrected the obvious structural errors in the code, as I am a web developer myself.

Checklist:

  • [] GitHub issue linked
  • The necessary translations have been added for all languages
  • Any documentation has been updated accordingly
  • Responsive sizes (Mobile) tested
  • Cross Browser tested if necessary
  • Conflicts resolved

mdqst added 3 commits November 6, 2024 13:52
Closing </script> tag in the <head> section

The </script> closing tag here is unnecessary, as this is a stylesheet link, not a script.

Corrected.
The <link> tag for Bootstrap CSS is incorrectly closed with </script>. It should not have a closing tag at all, or it should be closed as a self-closing

Corrected.
Incorrect Closing Tag for <h1>:

In the header section, <h1 class="font-weight-light text-secondary mb-2"><span>Support Dogecoin</span><h1> should be closed with </h1>, not <h1>.

Corrected.
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.

1 participant