René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"`androidOverflowInset` applies padding even when insets are marked as consumed","articleBody":"### Issue Description\n\n**Bug:** `androidOverflowInset` applies padding even when insets are marked as consumed\n\nWhen using `androidOverflowEdge=\"dont-apply\"` and listening to the `androidOverflowInset` event, NativeScript automatically applies padding even when insets are marked as consumed. This is a runtime issue.\n\n**Expected behavior:**\n\nWhen `inset.topConsumed = true` (and similar for other edges) is set in the `androidOverflowInset` event handler, NativeScript should not apply padding, regardless of the original inset values.\n\n**Actual behavior:**\n\nNativeScript applies padding based on whether the original inset values were 0 (`leftPreviouslyConsumed`), not based on whether the insets are marked as consumed (`leftConsumed`). This happens in `LayoutBase.java` line 280:\n\n```java\nbase.setPadding(leftPreviouslyConsumed ? 0 : leftInset, topPreviouslyConsumed ? 0 : topInset, rightPreviouslyConsumed ? 0 : rightInset, bottomPreviouslyConsumed ? 0 : bottomInset);\n```\n\nThe code checks `leftPreviouslyConsumed` (whether the original value was 0) instead of `leftConsumed` (whether it was marked as consumed by the event handler).\n\n**Workaround:**\n\nThe current workaround is to set inset values to 0 AND mark them as consumed:\n\n```typescript\ninset.top = 0;\ninset.bottom = 0;\ninset.left = 0;\ninset.right = 0;\ninset.topConsumed = true;\ninset.bottomConsumed = true;\ninset.leftConsumed = true;\ninset.rightConsumed = true;\n```\n\nThis is counterintuitive because marking as consumed should be sufficient.\n\n**Proposed fix:**\n\nIn `LayoutBase.java` line 280, the code should check `leftConsumed` instead of (or in addition to) `leftPreviouslyConsumed`:\n\n```java\n// Current (buggy):\nbase.setPadding(leftPreviouslyConsumed ? 0 : leftInset, ...);\n\n// Should be:\nbase.setPadding((leftPreviouslyConsumed || leftConsumed) ? 0 : leftInset, ...);\n```\n\n**Related file:**\n- `packages/ui-mobile-base/android/widgets/src/main/java/org/nativescript/widgets/LayoutBase.java` (line 280)\n\n\n### Reproduction\n\n1. Create a Frame with `androidOverflowEdge=\"dont-apply\"`:\n\n```xml\n\u003cFrame id=\"rootFrame\" androidOverflowEdge=\"dont-apply\" loaded=\"onLoaded\" /\u003e\n```\n\n2. Listen to the `androidOverflowInset` event and mark insets as consumed without setting their values to 0:\n\n```typescript\nexport function onLoaded(args: EventData) {\n  const rootFrame = args.object as Frame;\n  \n  rootFrame.on(\"androidOverflowInset\", (event: any) =\u003e {\n    const { inset } = event;\n    \n    // Store values for CSS variables\n    const values = { \n      top: inset.top, \n      bottom: inset.bottom, \n      left: inset.left, \n      right: inset.right \n    };\n    \n    // Mark as consumed (this should prevent padding, but doesn't)\n    inset.topConsumed = true;\n    inset.bottomConsumed = true;\n    inset.leftConsumed = true;\n    inset.rightConsumed = true;\n  });\n}\n```\n\n3. Run the app on an Android device/emulator\n4. Observe that padding is still applied automatically, even though insets are marked as consumed\n\n**Expected:** No padding should be applied when insets are marked as consumed.\n\n**Actual:** Padding is applied based on the original inset values.\n\n### Relevant log output (if applicable)\n\n```shell\nNo error messages, but the behavior can be observed visually as unwanted padding is applied to the view.\n```\n\n### Environment\n\n```yaml\nOS: Linux 6.17 Fedora Linux 43 (Workstation Edition)\nCPU: (20) x64 Intel(R) Core(TM) i9-10900K CPU @ 3.70GHz\nShell: /usr/bin/zsh\nnode: 22.14.0\nnpm: 10.9.2\nnativescript: 9.0.1\n\n# android\njava: 17.0.16\nndk: Not Found\napis: Not Found\nbuild_tools: Not Found\nsystem_images: Not Found\n\n# ios\nxcode: Not Found\ncocoapods: Not Found\npython: 3.14.2\npython3: 3.14.2\nruby: 3.4.7\nplatforms: Not Found\n```\n\n### Dependencies\n\n```json\n\"dependencies\": {\n  \"@learn6502/6502\": \"workspace:^\",\n  \"@learn6502/common-ui\": \"workspace:^\",\n  \"@nativescript/core\": \"~9.0.10\",\n  \"@nativescript/localize\": \"^5.2.0\",\n  \"@nativescript/tailwind\": \"^4.0.7\",\n  \"tailwindcss\": \"^4.1.18\"\n},\n\"devDependencies\": {\n  \"@nativescript/android\": \"9.0.1\",\n  \"@nativescript/types\": \"~9.0.0\",\n  \"@nativescript/webpack\": \"^5.0.29\",\n  \"@tailwindcss/postcss\": \"^4.1.18\",\n  \"autoprefixer\": \"^10.4.23\",\n  \"nativescript\": \"^9.0.1\",\n  \"postcss\": \"^8.5.6\",\n  \"postcss-cli\": \"^11.0.1\",\n  \"postcss-discard-empty\": \"^7.0.1\",\n  \"postcss-import\": \"^16.1.1\",\n  \"postcss-nested\": \"^7.0.2\",\n  \"string-replace-loader\": \"^3.3.0\",\n  \"typescript\": \"^5.9.3\"\n}\n```\n\n### Please accept these terms\n\n- [x] I have searched the [existing issues](https://github.com/NativeScript/NativeScript/issues) as well as [StackOverflow](https://stackoverflow.com/questions/tagged/nativescript) and this has not been posted before\n- [x] This is a bug report\n- [x] I agree to follow this project's [Code of Conduct](https://github.com/NativeScript/NativeScript/blob/master/tools/notes/CONTRIBUTING.md#coc)","author":{"url":"https://github.com/JumpLink","@type":"Person","name":"JumpLink"},"datePublished":"2025-12-31T12:29:15.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":1},"url":"https://github.com/11019/NativeScript/issues/11019"}