Skip to content

Conversation

@thePunderWoman
Copy link
Contributor

This fixes a case in which the document exists, but the body does not.

@angular-robot angular-robot bot added the area: core Issues related to the framework runtime label Jul 23, 2025
@ngbot ngbot bot added this to the Backlog milestone Jul 23, 2025
This fixes a case in which the document exists, but the body does not.
@thePunderWoman thePunderWoman force-pushed the animate-instructions-fix branch from 3baa40d to aec0b6b Compare July 23, 2025 16:24
(typeof ngServerMode === 'undefined' || !ngServerMode) &&
typeof document !== 'undefined' &&
// tslint:disable-next-line:no-toplevel-property-access
typeof document.body !== 'undefined' &&
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As a follow-up: we should be able to check document.documentElement instead.

@thePunderWoman thePunderWoman added action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release labels Jul 23, 2025
@kirjs
Copy link
Contributor

kirjs commented Jul 23, 2025

This PR was merged into the repository by commit 593cc8a.

The changes were merged into the following branches: main

@kirjs kirjs closed this in 593cc8a Jul 23, 2025
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Aug 23, 2025
@thePunderWoman thePunderWoman deleted the animate-instructions-fix branch September 12, 2025 16:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: core Issues related to the framework runtime target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants