Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scope willRender to PageRenderer only #741

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

seanpdoyle
Copy link
Contributor

The willRender property introduced to the generic Renderer<E, S> only applies to the PageRenderer specialized sub-class. This commit removes it from the root class and declares a
PageRenderer.constructor() method to accept and assign it.

@seanpdoyle seanpdoyle force-pushed the page-renderer-will-render branch from 978ccd8 to 279a9ca Compare November 19, 2022 18:09
@seanpdoyle seanpdoyle force-pushed the page-renderer-will-render branch from 279a9ca to 82982b8 Compare November 27, 2022 19:26
@seanpdoyle seanpdoyle force-pushed the page-renderer-will-render branch 2 times, most recently from 2aaba5c to e23abe0 Compare December 31, 2022 23:05
@seanpdoyle seanpdoyle force-pushed the page-renderer-will-render branch from e23abe0 to f5cc8fc Compare July 21, 2023 15:30
@seanpdoyle seanpdoyle force-pushed the page-renderer-will-render branch from f5cc8fc to af0c77c Compare September 14, 2023 13:41
@seanpdoyle seanpdoyle force-pushed the page-renderer-will-render branch 2 times, most recently from e4824da to 12b91f5 Compare January 30, 2024 14:34
The `willRender` property introduced to the generic `Renderer<E, S>`
only applies to the `PageRenderer` specialized sub-class. This commit
removes it from the root class and declares a
`PageRenderer.constructor()` method to accept and assign it.
@seanpdoyle seanpdoyle force-pushed the page-renderer-will-render branch from 12b91f5 to a34961e Compare March 29, 2024 23:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant