Improve the docs
authorAlexander Ebert <ebert@woltlab.com>
Mon, 22 Jan 2024 17:14:35 +0000 (18:14 +0100)
committerAlexander Ebert <ebert@woltlab.com>
Mon, 22 Jan 2024 17:14:35 +0000 (18:14 +0100)
wcfsetup/install/files/lib/system/html/node/HtmlNodePlainLink.class.php

index 3d949e9d52183e3bd01e96b87aaf3b34bf4a60f6..726c0dc6753d66f10cba3cc8ea9b1b05aba1fb57 100644 (file)
@@ -186,8 +186,8 @@ class HtmlNodePlainLink
                 $next = $this->findBr($this->link, 'nextSibling');
                 $previous = $this->findBr($this->link, 'previousSibling');
 
-                // If multiple links are in the same paragraph, the `topLevelParent` link to old parent in the DOM
-                // and have to be found again.
+                // When multiple links are in the same paragraph, `topLevelParent`
+                // may no longer be a valid reference.
                 if ($this->topLevelParent->parentElement === null) {
                     $this->topLevelParent = $this->link;
                     while ($this->topLevelParent->parentNode->nodeName !== 'body') {
@@ -202,8 +202,8 @@ class HtmlNodePlainLink
                     $previous = $this->findBr($parent, 'previousSibling');
                 }
 
-                // Link is the only content in the top level parent.
-                // This will happen, when in one paragraph are multiple links.
+                // The link is the only content in the top level parent. This
+                // can happen when there are multiple links within one paragraph.
                 if ($next === null && $previous === null) {
                     $this->topLevelParent->parentNode->insertBefore($this->link, $this->topLevelParent);
                     DOMUtil::removeNode($this->topLevelParent);