Skip to content

HAR-9460 - handle alternate content node#486

Merged
harbournick merged 1 commit intomainfrom
artem-HAR-9460-v2
May 1, 2025
Merged

HAR-9460 - handle alternate content node#486
harbournick merged 1 commit intomainfrom
artem-HAR-9460-v2

Conversation

@artem-harbour
Copy link
Copy Markdown
Contributor

@artem-harbour artem-harbour commented Apr 29, 2025

Changes from this PR are used here as a basis.
#478

@artem-harbour artem-harbour added the don't merge Don't merge yet label Apr 29, 2025
@artem-harbour artem-harbour self-assigned this Apr 29, 2025
@artem-harbour artem-harbour changed the title HAR-9460 - handle alternate content node [WIP] HAR-9460 - handle alternate content node Apr 29, 2025
@artem-harbour artem-harbour changed the title [WIP] HAR-9460 - handle alternate content node HAR-9460 - handle alternate content node Apr 30, 2025
element.style.paddingLeft = pageMargins.left + 'in';
element.style.paddingRight = pageMargins.right + 'in';
element.style.boxSizing = 'border-box';
element.style.isolation = 'isolate'; // to create new stacking context.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_positioned_layout/Stacking_context

To make z-index: -1 work for the element. This is probably a safer way than assigning z-index to the editor.

@artem-harbour artem-harbour added ready for review and removed don't merge Don't merge yet labels Apr 30, 2025
@artem-harbour artem-harbour marked this pull request as ready for review April 30, 2025 14:15
@artem-harbour
Copy link
Copy Markdown
Contributor Author

FYI @harbournick - this is ready for review.

Copy link
Copy Markdown
Collaborator

@harbournick harbournick left a comment

Choose a reason for hiding this comment

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

LGTM

@harbournick harbournick merged commit 27d14a9 into main May 1, 2025
5 of 6 checks passed
@harbournick harbournick deleted the artem-HAR-9460-v2 branch May 1, 2025 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants