canary regress combobox

[Zeroes1]

Windows Terminal version

1.21.1011.0

Windows build number

10.0.19045.3448

Other Software

No response

Steps to reproduce

look the video https://github.com/microsoft/terminal/assets/11919060/f5c52bf2-d51c-482c-9dfc-557d919ec554

ps. for future is good idea post issue like BUG for CANARY builds WT?

Expected Behavior

good work wit combobox latest good canary buid: terminal-1.21.931.0

Actual Behavior

terminal-1.21.991.0 terminal-1.21.1011.0 is bad for work with combobox

[zadjii-msft]

for future is good idea post issue like BUG for CANARY builds WT

Yes absolutely! If you're using canary and find bugs in it, absolutely file them here 😄

---

21.931 included the font face combo box, so that wasn't the source of this regression.

[DHowett]

Here are the commits for that version range:

1.21.991.0 (bad ) -> * b90eb93d2 Localization Updates - main - 04/06/2024 03:04:16 (#17025)
                     * 156b9aeea Make marks a stable feature (#16938)
                     * c3f44f773 Rewrite how marks are stored & add reflow (#16937)
                     * dc4026d18 Add support for restoring non-terminal panes, and opening them with `splitPane`, `newTab` (#16914)
                     * c063d2bad Change default pane keybindings to duplicate, rather than use default profile (#16951)
                     * 6a69b94fc Increase usage of fmt and FMT_COMPILE (#17012)
                     * 8d6e7a8a7 Buffer Restore: Fix serializing >32KiB (#17022)
                     * 1a6ba43dd Get rid of UpdateTerminalSettings (#17009)
                     * 67ae9f6c3 Refactor the SettingsTab to be a pane (#16172)
1.21.931.0 (good) -> * 2bcbe6b49 ci/rel: publish symbols using the internal symbol request API instead (#16991)

[DHowett]

@zadjii-msft just a cold read, but due to 67ae9f6c3 you may be the killer

[zadjii-msft]

You know, this is pretty dang painful. Promoting to blocking to investigate

Notes:

Wait a sec this doesn't repro on the canary build I have open on this machine anymore. What the hay barrels? Did I lose the repro?

okay definitely repros on main. Click on the right side of the combobox, and slowly move to the left. Usually there's a point where it starts working?

• [x] ❌ 32fbb16d4 main
• [x] ❌ f49cf44b799b4a4017ab7dce5c29694cda0fe725 Add more TraceLogging to ApiDispatchers (#17085)
• [x] ❌ 8bd9578b3ca9d18ba362ce1fff9b396d426b39e9 Fix: UI style errors: Menu items capitalization and … (ellipses) mark misuse (#16886)
• [x] ❌ c063d2bad68d46525a1fc693f4ff1d1c136b6f60 Change default pane keybindings to duplicate, rather than use default profile (#16951)
• [x] ❌1a6ba43dd220d4a318627006bf2a6c8b9a70c7bc Get rid of UpdateTerminalSettings (#17009)
• [ ] 67ae9f6c3e42e1ad3431ca02123fcfce57eef188 Refactor the SettingsTab to be a pane (#16172)
• [ ] ✅ 2bcbe6b49 ci/rel: publish symbols using the internal symbol request API instead (#16991)

67ae9f6c3e42e1ad3431ca02123fcfce57eef188 is the first bad commit

[zadjii-msft]

• On the non-retemplated ones, it's just the arrow region that doesn't open it.
• On the retemplated ones, it's anything except the label that doesn't open it