microsoft / microsoft/WPF-Samples

Whitespaces after opening tags ARE significant

Open
#44 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C#
Stars
5.7k
Forks
3.3k
Avg merge
12d 8h
Merged PRs (30d)
2

Description

Contrary to the original implementation of HtmlLexicalAnalyzer.cs, I believe whitespaces after opening tags ARE significant. Furthermore IE, Edge and Chrome seem to agree. I changed line 100 to:

_ignoreNextWhitespace = false; // Whitespaces after opening tags ARE significant

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in HtmlLexicalAnalyzer.cs around line 100 and inspect how whitespace immediately after opening tags is handled. Reproduce the affected parsing behavior and compare it with the stated IE, Edge, and Chrome behavior. Done means whitespace after opening tags is preserved consistently with the intended HTML behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
desktop
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.