Ensure Sheets render above native peers by enabling lightweight mode and adding blocker

[shai-almog]

Motivation

• Fix an iOS rendering bug where a Sheet could appear behind native peers (e.g. BrowserComponent) by making the form render in lightweight mode while the sheet is visible.
• Prevent native peers from receiving pointer events and visually overlapping the sheet by inserting a full-screen blocker into the sheet layer when peers are present.
• Keep existing sheet animation/layout behavior while isolating sheet content into a dedicated sheet layer so replacement/animation works reliably.

Description

• Enable lightweight rendering on the current Form when showing a sheet if Form.activePeerCount > 0 by calling f.setLightweightMode(true) before showing and restoring it after the sheet is hidden.
• Create and use a dedicated sheet layer retrieved via the new getSheetLayer(Container) helper that stores the layer in a client property cn1$sheetLayer and centralizes layout/animation work.
• Add ensureSheetBlocker(Container, Container) which manages a SheetBlocker component stored under client property cn1$sheetBlocker to block pointer events and cover the display when native peers exist, and remove it when no peers are active.
• Add inner class SheetBlocker that grabs pointer events and reports full-screen preferred size via calcPreferredSize(); switch container layout to LayeredLayout and update add/replace/animate calls to operate on the sheet layer.
• Update imports to include Dimension and modify BrowserComponentScreenshotTest to show a Sheet during the test to exercise the native-peer/sheet interaction scenario.

Testing

• No automated tests were executed for this change.
• The BrowserComponentScreenshotTest was adjusted to present a Sheet to reproduce the issue, but the test was not run in this PR.

---

Codex Task

[github-actions]

✅ Continuous Quality Report

Test & Coverage

• ✅ Tests: 2300 total, 0 failed, 0 skipped
• 📊 Line coverage: 53.00% [HTML preview] [Download]
  • Lowest covered classes
    • com.codename1.ui.Display$EdtException – 0.00%
    • com.codename1.ui.plaf.CSSBorder$LinearGradient – 0.00%
    • com.codename1.util.EasyThread$InQueueRunnable – 0.00%
    • com.codename1.components.FloatingActionButton$ReleaseActionListener – 0.00%
    • com.codename1.io.Oauth2$ShowAuthenticationActionListener – 0.00%
    • com.codename1.io.Oauth2$RefreshTokenActionListener – 0.00%
    • com.codename1.util.EasyThread$RunAndWaitRunnable – 0.00%
    • com.codename1.components.ToastBar$FlushAnimationCallback – 0.00%
    • com.codename1.ui.plaf.CSSBorder$ColorStop – 0.00%
    • com.codename1.ui.ElevationComparator – 0.00%

Static Analysis

• SpotBugs [HTML preview] [Download]
  • ❌ ByteCodeTranslator: 97 findings (High: 2, Normal: 68, Low: 27)
    • Top findings
      • High: com/codename1/tools/translator/BytecodeMethod.java:559 – Method com.codename1.tools.translator.BytecodeMethod.findWritableFields(Set) seems to be useless (rule: UC_USELESS_VOID_METHOD)
      • High: com/codename1/tools/translator/bytecodes/TryCatch.java:75 – Write to static field com.codename1.tools.translator.bytecodes.TryCatch.hasTryCatch from instance method com.codename1.tools.translator.bytecodes.TryCatch.appendInstruction(StringBuilder, List) (rule: ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD)
      • Normal: com/codename1/tools/translator/ByteCodeClass.java:1791 – com.codename1.tools.translator.ByteCodeClass.getBaseClassObject() may expose internal representation by returning ByteCodeClass.baseClassObject (rule: EI_EXPOSE_REP)
      • …and 94 more
  • ❌ android: 412 findings (High: 24, Normal: 196, Low: 192)
    • Top findings
      • High: CodenameOne/src/com/codename1/impl/CodenameOneThread.java:48 – com.codename1.impl.CodenameOneThread.STACK_FRAME_SIZE isn't final but should be (rule: MS_SHOULD_BE_FINAL)
      • High: com/codename1/impl/android/AndroidImplementation.java:1066 – Write to static field com.codename1.impl.android.AndroidImplementation.context from instance method com.codename1.impl.android.AndroidImplementation.setContext(Context) (rule: ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD)
      • High: com/codename1/impl/android/AndroidImplementation.java:1267 – com.codename1.impl.android.AndroidImplementation.isDarkMode() has Boolean return type and returns explicit null (rule: NP_BOOLEAN_RETURN_NULL)
      • …and 409 more
  • ❌ codenameone-maven-plugin: 480 findings (High: 1, Normal: 316, Low: 163)
    • Top findings
      • High: com/codename1/maven/AbstractCN1Mojo.java:807 – com.codename1.maven.AbstractCN1Mojo.OS isn't final but should be (rule: MS_SHOULD_BE_FINAL)
      • Normal: com/codename1/builders/AndroidGradleBuilder.java:1018 – Dead store to googleServicesJson in com.codename1.builders.AndroidGradleBuilder.build(File, BuildRequest) (rule: DLS_DEAD_LOCAL_STORE)
      • Normal: com/codename1/builders/AndroidGradleBuilder.java:1043 – Possible null pointer dereference in com.codename1.builders.AndroidGradleBuilder.build(File, BuildRequest) due to return value of called method (rule: NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE)
      • …and 477 more
  • ✅ core-unittests: 0 findings (no issues)
  • ❌ ios: 125 findings (High: 4, Normal: 75, Low: 46)
    • Top findings
      • High: com/codename1/impl/ios/IOSImplementation.java:1218 – com.codename1.impl.ios.IOSImplementation.isDarkMode() has Boolean return type and returns explicit null (rule: NP_BOOLEAN_RETURN_NULL)
      • High: com/codename1/impl/ios/IOSImplementation.java:127 – com.codename1.impl.ios.IOSImplementation.nativeInstance isn't final but should be (rule: MS_SHOULD_BE_FINAL)
      • High: com/codename1/impl/ios/IOSImplementation.java:212 – Write to static field com.codename1.impl.ios.IOSImplementation.instance from instance method com.codename1.impl.ios.IOSImplementation.init(Object) (rule: ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD)
      • …and 122 more
• ✅ PMD: 0 findings (no issues) [Report archive]
• ✅ Checkstyle: 0 findings (no issues) [Report archive]

Generated automatically by the PR CI workflow.

[shai-almog]

Android screenshot updates

Compared 30 screenshots: 29 matched, 1 updated.

• BrowserComponent — updated screenshot. Screenshot differs (320x640 px, bit depth 8).

  
  Preview info: JPEG preview quality 70; JPEG preview quality 70.
  Full-resolution PNG saved as BrowserComponent.png in workflow artifacts.

Native Android coverage

• 📊 Line coverage: 7.16% (3632/50711 lines covered) [HTML preview] (artifact android-coverage-report, jacocoAndroidReport/html/index.html)
  • Other counters: instruction 5.58% (17826/319385), branch 2.72% (845/31063), complexity 3.32% (986/29687), method 5.84% (807/13824), class 9.53% (174/1825)
  • Lowest covered classes
    • kotlin.collections.kotlin.collections.ArraysKt___ArraysKt – 0.00% (0/6327 lines covered)
    • kotlin.collections.unsigned.kotlin.collections.unsigned.UArraysKt___UArraysKt – 0.00% (0/2384 lines covered)
    • org.jacoco.agent.rt.internal_b6258fc.asm.org.jacoco.agent.rt.internal_b6258fc.asm.ClassReader – 0.00% (0/1519 lines covered)
    • kotlin.collections.kotlin.collections.CollectionsKt___CollectionsKt – 0.00% (0/1148 lines covered)
    • org.jacoco.agent.rt.internal_b6258fc.asm.org.jacoco.agent.rt.internal_b6258fc.asm.MethodWriter – 0.00% (0/923 lines covered)
    • kotlin.sequences.kotlin.sequences.SequencesKt___SequencesKt – 0.00% (0/712 lines covered)
    • kotlin.text.kotlin.text.StringsKt___StringsKt – 0.00% (0/623 lines covered)
    • org.jacoco.agent.rt.internal_b6258fc.asm.org.jacoco.agent.rt.internal_b6258fc.asm.Frame – 0.00% (0/564 lines covered)
    • kotlin.collections.kotlin.collections.ArraysKt___ArraysJvmKt – 0.00% (0/495 lines covered)
    • kotlinx.coroutines.kotlinx.coroutines.JobSupport – 0.00% (0/423 lines covered)

[shai-almog]

iOS screenshot updates

Compared 30 screenshots: 29 matched, 1 updated.

• BrowserComponent — updated screenshot. Screenshot differs (1206x2622 px, bit depth 8).

  
  Preview info: Preview provided by instrumentation.
  Full-resolution PNG saved as BrowserComponent.png in workflow artifacts.

Benchmark Results

• VM Translation Time: 330 seconds
• Compilation Time: 362 seconds

Detailed Performance Metrics

Metric	Duration
Build Time Statistics
Setup & Unzip	31853 ms
Extract Extensions	35 ms
Google Services Setup	2 ms
Scan Classes	665 ms
Extract Libs	817 ms
Inject Build Hints	56 ms
Generate Unit Tests	14 ms
Generate Stubs	1216 ms
Compile Stubs	2578 ms
Generate Icons	1117 ms
Prepare ParparVM	186 ms
ParparVM Execution	181940 ms
Post-VM Setup	145 ms
CocoaPods	4982 ms
Finalize	38 ms
Total Time	225645 msMaven Overhead : 105000 ms
CocoaPods Install (Script)	1000 ms
Simulator Boot	169000 ms
Simulator Boot (Run)	2000 ms
App Install	13000 ms
App Launch	8000 ms
Test Execution	137000 ms