Skip to content

fix: App drawer closes upon account selection#10580

Open
mohitsatr wants to merge 1 commit intothunderbird:mainfrom
mohitsatr:app-drawer-regression
Open

fix: App drawer closes upon account selection#10580
mohitsatr wants to merge 1 commit intothunderbird:mainfrom
mohitsatr:app-drawer-regression

Conversation

@mohitsatr
Copy link
Copy Markdown
Contributor

@mohitsatr mohitsatr commented Feb 24, 2026

Fixes #10120

Behaviour:

Bin is set as AutoExpandFolder for both accounts

Screen_Recording_20260402_101406_Thunderbird.Debug.mp4

@mohitsatr mohitsatr force-pushed the app-drawer-regression branch from 42f12d8 to 12bb10f Compare March 9, 2026 08:25
@mohitsatr mohitsatr marked this pull request as ready for review March 9, 2026 08:25
@mohitsatr mohitsatr requested a review from a team as a code owner March 9, 2026 08:25
@mohitsatr mohitsatr requested a review from jbott-tbird March 9, 2026 08:25
@mohitsatr mohitsatr force-pushed the app-drawer-regression branch from 12bb10f to c739138 Compare March 9, 2026 08:27
@wmontwe wmontwe requested review from wmontwe and removed request for jbott-tbird March 10, 2026 15:45
@wmontwe wmontwe assigned wmontwe and unassigned jbott-tbird Mar 10, 2026
@mohitsatr mohitsatr force-pushed the app-drawer-regression branch from c739138 to e71ba41 Compare March 11, 2026 02:21
@mohitsatr
Copy link
Copy Markdown
Contributor Author

I feel creating a use case for getting the autoExpandAccount Id is an unnecessary abstraction. Is there a better way to do this?
In fact, we don't even need ID, just the boolean value to let us know whether or not AutoExpandAccount is set.

@wmontwe
Copy link
Copy Markdown
Member

wmontwe commented Mar 11, 2026

I feel creating a use case for getting the autoExpandAccount Id is an unnecessary abstraction. Is there a better way to do this? In fact, we don't even need ID, just the boolean value to let us know whether or not AutoExpandAccount is set.

A separate use case is not needed, you could update the GetDisplayAccounts and the DisplayAccount to hold that property.

@mohitsatr mohitsatr force-pushed the app-drawer-regression branch from e71ba41 to eef951b Compare April 2, 2026 04:30
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

Missing report label. Set exactly one of: report: include, report: exclude OR report: highlight.

@mohitsatr
Copy link
Copy Markdown
Contributor Author

@wmontwe Please review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

App drawer no longer closes upon account selection, regression??

3 participants