rename RenderItem, RenderItemLogOptions

rendering is too generic; this is the layout tree, which is referred to
as the "tree" in the upcoming refactor

Author: chearon

src/layout-box.ts

@@ -20,7 +20,7 @@ export interface LogicalArea {
   inlineSize: number | undefined;
 }
 
-export interface RenderItemLogOptions {
+export interface TreeLogOptions {
   containingBlocks?: boolean;
   css?: keyof Style
   paragraphText?: string;
@@ -32,7 +32,7 @@ export interface PrelayoutContext {
   lastPositionedArea: BoxArea
 }
 
-export abstract class RenderItem {
+export abstract class TreeNode {
   public style: Style;
 
   constructor(style: Style) {
@@ -75,11 +75,11 @@ export abstract class RenderItem {
     return false;
   }
 
-  abstract logName(log: Logger, options?: RenderItemLogOptions): void;
+  abstract logName(log: Logger, options?: TreeLogOptions): void;
 
   abstract getLogSymbol(): string;
 
-  log(options?: RenderItemLogOptions, log?: Logger) {
+  log(options?: TreeLogOptions, log?: Logger) {
     const flush = !log;
 
     log ||= new Logger();
@@ -154,7 +154,7 @@ export abstract class RenderItem {
   }
 }
 
-export abstract class Box extends RenderItem {
+export abstract class Box extends TreeNode {
   public id: string;
   /**
    * General boolean bitfield shared by all box subclasses. The bits labeled
@@ -511,7 +511,7 @@ export abstract class Box extends RenderItem {
     }
   }
 
-  logName(log: Logger, options?: RenderItemLogOptions) {
+  logName(log: Logger, options?: TreeLogOptions) {
     log.text('Box');
   }
 

src/layout-flow.ts

@@ -15,7 +15,7 @@ import {
   sliceIfcRenderText
 } from './layout-text.ts';
 import {getImage} from './layout-image.ts';
-import {Box, FormattingBox, RenderItem} from './layout-box.ts';
+import {Box, FormattingBox, TreeNode} from './layout-box.ts';
 
 import type {InlineMetrics, ShapedItem, InlineFragment} from './layout-text.ts';
 import type {BoxArea, PrelayoutContext} from './layout-box.ts';
@@ -1199,7 +1199,7 @@ export function layoutFloatBox(box: BlockLevel, ctx: LayoutContext) {
   }
 }
 
-export class Break extends RenderItem {
+export class Break extends TreeNode {
   public className = 'break';
 
   isBreak(): this is Break {

src/layout-text.ts

@@ -1,5 +1,5 @@
 import {basename, loggableText, Logger} from './util.ts';
-import {Box, RenderItem} from './layout-box.ts';
+import {Box, TreeNode} from './layout-box.ts';
 import {Style} from './style.ts';
 import {
   ReplacedBox,
@@ -19,7 +19,7 @@ import {createItemizeState, itemizeNext} from './text-itemize.ts';
 
 import type {LoadedFontFace} from './text-font.ts';
 import type {HbFace, HbFont} from './text-harfbuzz.ts';
-import type {RenderItemLogOptions} from './layout-box.ts';
+import type {TreeLogOptions} from './layout-box.ts';
 import type {TextAlign, WhiteSpace} from './style.ts';
 import type {BlockLevel, InlineLevel, BlockContainer, LayoutContext} from './layout-flow.ts';
 
@@ -70,7 +70,7 @@ export function prevGrapheme(text: string, index: number) {
   return graphemeStart < index ? graphemeStart : index;
 }
 
-export class Run extends RenderItem {
+export class Run extends TreeNode {
   public textStart: number;
   public textEnd: number;
 
@@ -111,7 +111,7 @@ export class Run extends RenderItem {
     return true;
   }
 
-  logName(log: Logger, options?: RenderItemLogOptions) {
+  logName(log: Logger, options?: TreeLogOptions) {
     log.text(`${this.textStart},${this.textEnd}`);
     if (options?.paragraphText) {
       log.text(` "${loggableText(options.paragraphText.slice(this.textStart, this.textEnd))}"`);