[1.12] Split pane on parent loses opacity? #15381

Open
opened 2026-01-31 04:36:56 +00:00 by claunia · 3 comments
Owner

Originally created by @zadjii-msft on GitHub (Sep 28, 2021).

discovered in the 1.12 bug bash on 9/28, but probably not blocking

Split pane on parent loses opacity? (bad merge?)
Repro steps:

  1. Have multiple panes open
  2. MoveFocus to parent pane
  3. SplitPane
    Behavior: new pane has transparency, but old panes lost it
Originally created by @zadjii-msft on GitHub (Sep 28, 2021). _discovered in the 1.12 bug bash on 9/28, but probably not blocking_ Split pane on parent loses opacity? (bad merge?) Repro steps: 1. Have multiple panes open 1. MoveFocus to parent pane 1. SplitPane Behavior: new pane has transparency, but old panes lost it
Author
Owner

@Rosefield commented on GitHub (Sep 29, 2021):

Ah, this is probably that GetFocusedProfile tries to get the profile off the actually focused pane instead of leaf control. I can fix this quickly.

Edit: Well, it was not the problem I thought it was, and deleting the newly split pane restores the opacity to the original panes, so I don't know what might be causing this.

@Rosefield commented on GitHub (Sep 29, 2021): Ah, this is probably that `GetFocusedProfile` tries to get the profile off the actually focused pane instead of leaf control. I can fix this quickly. Edit: Well, it was not the problem I thought it was, and deleting the newly split pane restores the opacity to the original panes, so I don't know what might be causing this.
Author
Owner

@zadjii-msft commented on GitHub (Nov 28, 2022):

(note to self, still repros in 1.17 selfhost. Might want to knock out at the same time as #14260 even though the two are unrelated)

@zadjii-msft commented on GitHub (Nov 28, 2022): (note to self, still repros in 1.17 selfhost. Might want to knock out at the same time as #14260 even though the two are unrelated)
Author
Owner

@zadjii-msft commented on GitHub (Nov 29, 2022):

There's a Background that's not getting removed. That's the issue. If you manually decrease the opacity of the pane down to 0, you'll see a weird grey color fill the pane (possibly just the inactive border color).

Probably one of those Pane backgrounds that we insert for the sake of the animation and then remove when the animation is completed - I bet that's wacky when a parent pane is selected.

b6b1ff8b2c/src/cascadia/TerminalApp/Pane.cpp (L89-L91)

This line is sus - removing it fixes the bug, but re-introduces "transparent background as the pane animates in". Hmm.

@zadjii-msft commented on GitHub (Nov 29, 2022): There's a `Background` that's not getting removed. That's the issue. If you manually decrease the opacity of the pane down to 0, you'll see a weird grey color fill the pane (possibly just the inactive border color). Probably one of those Pane backgrounds that we insert for the sake of the animation and then remove when the animation is completed - I bet that's wacky when a parent pane is selected. https://github.com/microsoft/terminal/blob/b6b1ff8b2cfea71d45ba5345ec7a4af9f07dc66d/src/cascadia/TerminalApp/Pane.cpp#L89-L91 This line is sus - removing it fixes the bug, but re-introduces "transparent background as the pane animates in". Hmm.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#15381