Skip to content

Conversation

coliff
Copy link
Member

@coliff coliff commented Sep 29, 2025

Added keywords for accessibility and SEO to several rule docs, clarified rule incompatibility with Prettier, and improved explanations for tagname rules. Updated configuration example for clarity and bumped embedded HTMLHint version in VS Code extension docs. Minor spelling update in cspell config and commit message instructions.

Added keywords for accessibility and SEO to several rule docs, clarified rule incompatibility with Prettier, and improved explanations for tagname rules. Updated configuration example for clarity and bumped embedded HTMLHint version in VS Code extension docs. Minor spelling update in cspell config and commit message instructions.
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request primarily focuses on improving the documentation by adding keywords for better searchability, clarifying rule configurations and incompatibilities, and enhancing explanations. My review identified a potential rendering issue in one of the documentation files due to a likely typo in the markdown syntax, and also suggests a wording improvement in another file to enhance clarity and professionalism. Overall, these are positive changes that improve the project's documentation.

Comment on lines +33 to +34
Lowercase tagnames typically have higher compression rates compared to uppercase tagnames allowing for slightly faster page loads.
Lowercase tagnames are also more readable and easier to understand.
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

The explanation for this rule's importance could be phrased more clearly and professionally. The current version has a minor grammatical awkwardness and could better articulate the benefits.

Lowercase tagnames typically have higher compression rates compared to uppercase tagnames, leading to slightly faster page loads.
Lowercase tagnames are also more readable and align with common coding conventions.

Added guidelines for documentation language and referencing authoritative sources in general.mdc. Removed a trailing newline from attr-value-single-quotes.mdx.
Copy link

Deploying htmlhint with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6f6ff23
Status: ✅  Deploy successful!
Preview URL: https://515e6281.htmlhint.pages.dev
Branch Preview URL: https://dev-coliff-minor--docs--upda.htmlhint.pages.dev

View logs

@coliff coliff merged commit 1ae6468 into main Sep 29, 2025
19 of 24 checks passed
@coliff coliff deleted the dev/coliff/minor--docs--update branch September 29, 2025 06:45
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