Skip to content

fix: markdown editor integ tests failing and tools panel string changes#2843

Merged
abose merged 8 commits intomainfrom
ai
Apr 25, 2026
Merged

fix: markdown editor integ tests failing and tools panel string changes#2843
abose merged 8 commits intomainfrom
ai

Conversation

@abose
Copy link
Copy Markdown
Member

@abose abose commented Apr 24, 2026

No description provided.

abose added 8 commits April 24, 2026 21:46
F8 is bound to the Live Preview edit toggle (_handlePreviewBtnClick),
which calls LiveDevelopment.setMode(LIVE_EDIT_MODE) and shows the
UPSELL_TYPE_LIVE_EDIT dialog for free users. The F-key forwarding
tests dispatched F8 into the mdviewer iframe, which was then forwarded
to Phoenix's document-level F8 handler — stacking 2 upsell dialogs
in the test runner.

Swap to F9 (unbound) — the test only needs to verify that F-keys
are forwarded from the iframe, it doesn't need F8 specifically.

Also rename the outer describe to "livepreview:Markdown Editor 1" to
match sibling suites ("Edit Mode", "Edit More", "Table Editing").
New string used by the device-size cycle button in phoenix-pro when
no other cycle target fits the available width in normal preview mode.
The fit-to-panel button only appeared in design mode when a responsive
viewport was active. Its role is now folded into the device size cycle
button in phoenix-pro, which treats "fit" as a 4th cycle state when in
design mode. Drops the dedicated button, its CSS, and the unused
DEVICE_SIZE_FIT_TO_PANEL string.
Used by the "Fit to screen" item in the live preview device size
dropdown in phoenix-pro (visible only in design mode).
At non-integer DPR (e.g. 110% browser zoom), subpixel rounding could
leave a ~1px strip of body bg visible between #centralControlBar and
its neighbors — sidebar on the left, editor on the right. Add a solid
zero-blur box-shadow on both sides so CCB-colored pixels paint into
those gaps without changing any geometry.
@sonarqubecloud
Copy link
Copy Markdown

@abose abose merged commit e46c9a4 into main Apr 25, 2026
14 of 21 checks passed
@abose abose deleted the ai branch April 25, 2026 15:53
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.

1 participant