Skip to content

feat: add harness test for issue #3897#3989

Open
DaveyEke wants to merge 1 commit into
mrousavy:mainfrom
DaveyEke:repro-test
Open

feat: add harness test for issue #3897#3989
DaveyEke wants to merge 1 commit into
mrousavy:mainfrom
DaveyEke:repro-test

Conversation

@DaveyEke
Copy link
Copy Markdown

@DaveyEke DaveyEke commented Jun 2, 2026

This PR introduces a test meant to reproduce this issue to be certain it doesn't happen again once fixed.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 2, 2026

@DaveyEke is attempting to deploy a commit to the Margelo Team on Vercel.

A member of the Team first needs to authorize it.

it('preserves preview position when laid out below a sibling spacer (issue #3897)', async (context) => {
if (Platform.OS !== 'android') {
return context.skip(
'Preview spacer-positioned layout: Android-only regression',
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Test should run on all platforms. No need at making the test narrower - what benefit would that provide us?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

The initial Github issue stated the bug happened on Android.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants