ENG-13348: Fix Contact Sales dialog scrolling on mobile

[carlosabadia]

[linear-code]

ENG-13348

[cubic-dev-ai]

No issues found across 3 files

_{View guided diff | Turn on auto-fix | Re-trigger cubic}

[greptile-apps]

Confidence Score: 5/5

[Medium impact] The PR appears safe to merge; no blocking issues were found.

Summary

Keeps the Contact Sales dialog within the mobile viewport.

• Contact Sales fields scroll while the dialog header and Submit stay visible.

The earlier, unnumbered finding about missing browser coverage is addressed by the scrolling test. No new actionable issues were found in the changes since the previous review.

_{Reviews (3) · Last reviewed commit: "Re-register dialog icons for each browse..." · Reviewed by Greptile}

[codspeed]

Merging this PR will improve performance by 4.45%

⚠️ 20 benchmarks spent significant time in system calls

System calls cannot be consistently instrumented, so they are not included in the measure, which understates the real cost. Please switch to the Walltime instrument to accurately measure system calls.

Measurement and system calls

• test_get_state[memory]
• test_emit_update
• test_cli_startup[deploy_help]
• test_state_manager_memory_modify
• test_state_manager_memory_read_only_modify
• test_get_state_uncached[disk]
• test_import_cli
• test_state_manager_memory_cold_get
• test_event_shutdown_cancel
• test_modify_state[disk]
• …and 10 more

⚡ 1 improved benchmark
✅ 147 untouched benchmarks

Performance Changes

	Benchmark	BASE	HEAD	Efficiency
⚡	test_route_argument_extraction	119.7 µs	114.6 µs	+4.45%

Tip

Curious why performance improved? Comment @codspeedbot explain why performance improved on this PR, or directly use the CodSpeed MCP with your agent.

---

_{Comparing carlos/eng-13348-contact-sales-mobile (b195be5) with main (7ccaa4e)}