Skip to content

Commit a2cdc82

Browse files
jhaygood86claude
authored andcommitted
Remove upstream-project references from source comments
The vendored CSS engine and the Html-layer files derived from the same lineage carried doc comments naming the upstream project and its file paths. Provenance belongs in the commit history and the PR description, not in the source, so the comments now describe what the code does rather than where it came from. No functional change. The one remaining mention is the vendor-prefixed property name `-peachpdf-pdf-tag-type` itself, which is a CSS identifier rather than a comment; it is inert here (nothing consumes tagged-PDF output) and renaming a vendor prefix would diverge from the engine it is synced against. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
1 parent 2ae26df commit a2cdc82

14 files changed

Lines changed: 16 additions & 16 deletions

File tree

‎Source/HtmlRenderer/Core/CssData.cs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,8 @@ namespace TheArtOfDev.HtmlRenderer.Core
2626
/// specificity-ordered selector matching against the box tree.<br/>
2727
/// </summary>
2828
/// <remarks>
29-
/// Ported from PeachPDF's Html/Core/CssData.cs (itself a fork of Tyler Brinks' ExCSS), adapted to
30-
/// HTML-Renderer's CssBox/HtmlTag shape. One deliberate omission from the source: the
29+
/// Derived from the same ExCSS lineage as the vendored CSS-OM (originally Tyler Brinks' ExCSS),
30+
/// adapted to HTML-Renderer's CssBox/HtmlTag shape. One deliberate omission from the source: the
3131
/// <c>::marker</c> pseudo-element box-synthesis path is removed - <c>::before</c>/<c>::after</c>
3232
/// synthesis is kept, but this codebase has no <c>CssBox.IsMarkerPseudoElement</c> concept and no
3333
/// consumer for it (list markers are painted procedurally, not via a synthesized box).

‎Source/HtmlRenderer/Core/CssEngine/BackgroundPositionGrammar.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ namespace TheArtOfDev.HtmlRenderer.Core.CssEngine
66
/// Shared grammar for the CSS <c>background-position</c> value (the 1/2/3/4-token forms,
77
/// including the edge-relative offset syntax, e.g. "right 20px bottom 10px"). Used by both
88
/// <see cref="Converters.PointConverter"/> (CSS-OM grammar validation/serialization) and
9-
/// PeachPDF.Html.Core.Utils.BackgroundLayerResolver (render-time pixel-offset arithmetic) so
9+
/// the render-time background-layer resolution (pixel-offset arithmetic) so
1010
/// there is one parser for this grammar, not two independent re-implementations - mirroring how
1111
/// CalcParser is shared between CalcValueConverter and CssValueParser rather than re-parsed twice.
1212
/// </summary>

‎Source/HtmlRenderer/Core/CssEngine/BackgroundSizeGrammar.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ namespace TheArtOfDev.HtmlRenderer.Core.CssEngine
66
/// Shared grammar for the CSS <c>background-size</c> value (<c>auto</c>, <c>cover</c>,
77
/// <c>contain</c>, or a 1-2 token length/percentage/auto form). Used by both
88
/// <see cref="Converters.BackgroundSizeConverter"/> (CSS-OM grammar validation/serialization)
9-
/// and PeachPDF.Html.Core.Utils.BackgroundLayerResolver (render-time size resolution), so there
9+
/// and the render-time background-layer resolution (size resolution), so there
1010
/// is one parser for this grammar, not two.
1111
/// </summary>
1212
internal static class BackgroundSizeGrammar

‎Source/HtmlRenderer/Core/CssEngine/Calc/CalcEvaluator.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ public CalcContext(double hundredPercent, double emFactor, double remFactor, boo
3434

3535
/// <summary>
3636
/// Evaluates a validated calc-family AST to a pixel-space number. This is the one place calc()
37-
/// numbers actually get computed — called only from Layer B (<see cref="PeachPDF.Html.Core.Parse.CssValueParser.ParseLength(string, double, double, double, string, bool, bool)"/>),
37+
/// numbers actually get computed — called only from Layer B (<c>CssValueParser.ParseLength</c>),
3838
/// since only layout has the <see cref="CalcContext"/> a percentage/em/rem leaf needs to resolve.
3939
/// Reuses <see cref="Length.ToPixels"/> for every leaf, so no unit-conversion arithmetic is duplicated
4040
/// here. A null result signals a divide-by-zero; per the type-checker's rules every legal divisor is

‎Source/HtmlRenderer/Core/CssEngine/Enumerations/PdfTagType.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
namespace TheArtOfDev.HtmlRenderer.Core.CssEngine
22
{
33
/// <summary>
4-
/// Values for the PeachPDF-specific <c>-peachpdf-pdf-tag-type</c> property, which controls
4+
/// Values for the vendor-prefixed <c>-peachpdf-pdf-tag-type</c> property, which controls
55
/// the PDF structure (tag) type an element maps to when tagged PDF output is enabled.
66
/// </summary>
77
internal enum PdfTagType

‎Source/HtmlRenderer/Core/CssEngine/Model/Converters.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -500,7 +500,7 @@ public static readonly IValueConverter
500500

501501
public static readonly IValueConverter MultipleShadowConverter = ShadowConverter.FromList().OrNone();
502502

503-
// The image functions PeachPDF validates but does not render (CSS Images 4). Composed into
503+
// The image functions the engine validates but the renderer does not paint (CSS Images 4). Composed into
504504
// ImageSourceConverter so every <image> property (background-image, list-style-image, cursor,
505505
// content, and @property syntax:"<image>") accepts them syntactically; the render path
506506
// (CssValueParser.ParseImage → CssImagePainter) still handles only url()/gradients, so they paint

‎Source/HtmlRenderer/Core/CssEngine/ValueConverters/ExtendedImageConverters.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ namespace TheArtOfDev.HtmlRenderer.Core.CssEngine
77

88
/// <summary>
99
/// A validated-but-not-rendered image function (<c>image-set()</c>/<c>cross-fade()</c>/<c>element()</c>,
10-
/// CSS Images 4 §2/§3). PeachPDF accepts these as syntactically-valid <c>&lt;image&gt;</c> values (so a
10+
/// CSS Images 4 §2/§3). The engine accepts these as syntactically-valid <c>&lt;image&gt;</c> values (so a
1111
/// property or <c>@property</c> registration using one is valid, per spec) but paints nothing for them —
1212
/// the render path (<c>CssValueParser.ParseImage</c> → <c>CssImagePainter</c>) only handles url()/gradients.
1313
/// This value carries the original tokens for serialization only.

‎Source/HtmlRenderer/Core/Dom/CssContentEngine.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ namespace TheArtOfDev.HtmlRenderer.Core.Dom
2323
/// Resolves a synthesized <c>::before</c>/<c>::after</c> pseudo-element box's <see cref="CssBox.Content"/>
2424
/// value into literal text, so it renders through the box's normal <see cref="CssBox.Text"/>/
2525
/// <see cref="CssBox.ParseToWords"/> machinery like any other text box.<br/>
26-
/// Deliberately a small, self-contained subset of PeachPDF's CssContentEngine: literal strings,
26+
/// Deliberately a small, self-contained subset of the full content grammar: literal strings,
2727
/// <c>attr()</c>, and the four quote keywords are supported; <c>counter()</c>/<c>counters()</c>/
2828
/// <c>string()</c>/<c>content()</c> are not (CSS counters and named strings remain out of scope
2929
/// for this backport - an unrecognized component is simply skipped, the same graceful-ignore

‎Source/HtmlRenderer/Core/Entities/CssImage.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ namespace TheArtOfDev.HtmlRenderer.Core.Entities
22
{
33
/// <summary>
44
/// The parsed form of a single <c>background-image</c> value: either a <c>url()</c> reference or a
5-
/// gradient function. Unlike PeachPDF's <c>CssImage</c> (which this is a trimmed-down port of, same
5+
/// gradient function. Unlike the fuller model this is trimmed down from (same
66
/// name for an easy diff), <c>background-image</c> here is a single value, not a comma-separated
77
/// layered list - this project only backports enough to replace the old non-standard
88
/// "background-gradient" property.

‎Source/HtmlRenderer/Core/Entities/ParsedLinearGradient.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ namespace TheArtOfDev.HtmlRenderer.Core.Entities
55
{
66
/// <summary>
77
/// A fully parsed <c>linear-gradient()</c>/<c>repeating-linear-gradient()</c> value - the angle and
8-
/// raw (not yet position-resolved) color stop list. Ported from PeachPDF's
8+
/// raw (not yet position-resolved) color stop list. Derived from
99
/// Html/Core/Entities/ParsedLinearGradient.cs, minus CSS Color 4 interpolation-color-space support
1010
/// (ColorSpace/HueMethod) - gradients always interpolate in sRGB here.
1111
/// </summary>

0 commit comments

Comments
 (0)