Commit 8c0bf96
committed
[DevTools] Render selected outline on top of every other rect (facebook#35012)
When rects are close together (or overlapping) the outline can end up
being covered up by sibling rects or deeper rects. This renders the
selected outline on top of everything so it's always visible.
<img width="275" height="730" alt="Screenshot 2025-10-29 at 8 43 28 PM"
src="https://github.com/user-attachments/assets/69224883-f548-45ec-ada1-1a04ec17eaf5"
/>
<img width="266" height="737" alt="Screenshot 2025-10-29 at 8 58 53 PM"
src="https://github.com/user-attachments/assets/946f7dde-450d-49fd-9fbd-57487f67f461"
/>
Additionally, this makes it so that it's not part of the translucent
tree when things are hidden by the timeline. That way it's easier to see
what is selected inside a hidden tree.
<img width="498" height="196" alt="Screenshot 2025-10-29 at 8 45 24 PM"
src="https://github.com/user-attachments/assets/723107ab-a92c-42c2-8a7d-a548ac3332d0"
/>
<img width="571" height="735" alt="Screenshot 2025-10-29 at 8 59 06 PM"
src="https://github.com/user-attachments/assets/d653f1a7-4096-45c3-b92a-ef155d4742e6"
/>
DiffTrain build for [6fb7754](facebook@6fb7754)1 parent a855864 commit 8c0bf96
File tree
21 files changed
+72
-72
lines changed- compiled-rn
- facebook-fbsource/xplat/js
- RKJSModules/vendor/react
- react-dom/cjs
- react-test-renderer/cjs
- react/cjs
- react-native-github/Libraries/Renderer
- implementations
- tools/eslint-plugin-react-hooks
21 files changed
+72
-72
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
414 | 414 | | |
415 | 415 | | |
416 | 416 | | |
417 | | - | |
| 417 | + | |
418 | 418 | | |
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
209 | 209 | | |
210 | 210 | | |
211 | 211 | | |
212 | | - | |
| 212 | + | |
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
209 | 209 | | |
210 | 210 | | |
211 | 211 | | |
212 | | - | |
| 212 | + | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
29744 | 29744 | | |
29745 | 29745 | | |
29746 | 29746 | | |
29747 | | - | |
| 29747 | + | |
29748 | 29748 | | |
29749 | 29749 | | |
29750 | 29750 | | |
29751 | | - | |
| 29751 | + | |
29752 | 29752 | | |
29753 | 29753 | | |
29754 | 29754 | | |
| |||
29785 | 29785 | | |
29786 | 29786 | | |
29787 | 29787 | | |
29788 | | - | |
| 29788 | + | |
29789 | 29789 | | |
29790 | 29790 | | |
29791 | | - | |
| 29791 | + | |
29792 | 29792 | | |
29793 | 29793 | | |
29794 | 29794 | | |
| |||
29938 | 29938 | | |
29939 | 29939 | | |
29940 | 29940 | | |
29941 | | - | |
| 29941 | + | |
29942 | 29942 | | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
17374 | 17374 | | |
17375 | 17375 | | |
17376 | 17376 | | |
17377 | | - | |
| 17377 | + | |
17378 | 17378 | | |
17379 | 17379 | | |
17380 | 17380 | | |
17381 | 17381 | | |
17382 | 17382 | | |
17383 | 17383 | | |
17384 | | - | |
| 17384 | + | |
17385 | 17385 | | |
17386 | 17386 | | |
17387 | 17387 | | |
| |||
17403 | 17403 | | |
17404 | 17404 | | |
17405 | 17405 | | |
17406 | | - | |
| 17406 | + | |
17407 | 17407 | | |
17408 | 17408 | | |
17409 | | - | |
| 17409 | + | |
17410 | 17410 | | |
17411 | 17411 | | |
17412 | 17412 | | |
| |||
17513 | 17513 | | |
17514 | 17514 | | |
17515 | 17515 | | |
17516 | | - | |
| 17516 | + | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
19478 | 19478 | | |
19479 | 19479 | | |
19480 | 19480 | | |
19481 | | - | |
| 19481 | + | |
19482 | 19482 | | |
19483 | 19483 | | |
19484 | 19484 | | |
19485 | 19485 | | |
19486 | 19486 | | |
19487 | 19487 | | |
19488 | | - | |
| 19488 | + | |
19489 | 19489 | | |
19490 | 19490 | | |
19491 | 19491 | | |
| |||
19507 | 19507 | | |
19508 | 19508 | | |
19509 | 19509 | | |
19510 | | - | |
| 19510 | + | |
19511 | 19511 | | |
19512 | 19512 | | |
19513 | | - | |
| 19513 | + | |
19514 | 19514 | | |
19515 | 19515 | | |
19516 | 19516 | | |
| |||
19633 | 19633 | | |
19634 | 19634 | | |
19635 | 19635 | | |
19636 | | - | |
| 19636 | + | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
29800 | 29800 | | |
29801 | 29801 | | |
29802 | 29802 | | |
29803 | | - | |
| 29803 | + | |
29804 | 29804 | | |
29805 | 29805 | | |
29806 | 29806 | | |
29807 | | - | |
| 29807 | + | |
29808 | 29808 | | |
29809 | 29809 | | |
29810 | 29810 | | |
| |||
29841 | 29841 | | |
29842 | 29842 | | |
29843 | 29843 | | |
29844 | | - | |
| 29844 | + | |
29845 | 29845 | | |
29846 | 29846 | | |
29847 | | - | |
| 29847 | + | |
29848 | 29848 | | |
29849 | 29849 | | |
29850 | 29850 | | |
| |||
30324 | 30324 | | |
30325 | 30325 | | |
30326 | 30326 | | |
30327 | | - | |
| 30327 | + | |
30328 | 30328 | | |
30329 | 30329 | | |
30330 | 30330 | | |
| |||
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
17385 | 17385 | | |
17386 | 17386 | | |
17387 | 17387 | | |
17388 | | - | |
| 17388 | + | |
17389 | 17389 | | |
17390 | 17390 | | |
17391 | 17391 | | |
17392 | 17392 | | |
17393 | 17393 | | |
17394 | 17394 | | |
17395 | | - | |
| 17395 | + | |
17396 | 17396 | | |
17397 | 17397 | | |
17398 | 17398 | | |
| |||
17414 | 17414 | | |
17415 | 17415 | | |
17416 | 17416 | | |
17417 | | - | |
| 17417 | + | |
17418 | 17418 | | |
17419 | 17419 | | |
17420 | | - | |
| 17420 | + | |
17421 | 17421 | | |
17422 | 17422 | | |
17423 | 17423 | | |
| |||
17684 | 17684 | | |
17685 | 17685 | | |
17686 | 17686 | | |
17687 | | - | |
| 17687 | + | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
19493 | 19493 | | |
19494 | 19494 | | |
19495 | 19495 | | |
19496 | | - | |
| 19496 | + | |
19497 | 19497 | | |
19498 | 19498 | | |
19499 | 19499 | | |
19500 | 19500 | | |
19501 | 19501 | | |
19502 | 19502 | | |
19503 | | - | |
| 19503 | + | |
19504 | 19504 | | |
19505 | 19505 | | |
19506 | 19506 | | |
| |||
19522 | 19522 | | |
19523 | 19523 | | |
19524 | 19524 | | |
19525 | | - | |
| 19525 | + | |
19526 | 19526 | | |
19527 | 19527 | | |
19528 | | - | |
| 19528 | + | |
19529 | 19529 | | |
19530 | 19530 | | |
19531 | 19531 | | |
| |||
19808 | 19808 | | |
19809 | 19809 | | |
19810 | 19810 | | |
19811 | | - | |
| 19811 | + | |
19812 | 19812 | | |
19813 | 19813 | | |
19814 | 19814 | | |
| |||
0 commit comments