Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Always show back button in the right panel #29128

Merged
merged 7 commits into from
Feb 2, 2025

Conversation

MidhunSureshR
Copy link
Member

@MidhunSureshR MidhunSureshR commented Jan 29, 2025

For https://github.com/element-hq/wat-internal/issues/240

We always want to show a back button in the right panel. This PR modifies the setCard method which is used in the room header (through showOrHidePhase) and elsewhere to generate a history for a given phase.
eg: setCard(RoomMemberInfo) would have previously set the history as just [RoomMemberInfo] but now it becomes [RoomSummary, MemberList, RoomMemberInfo].

So that back buttons are always shown in the right panel
@MidhunSureshR MidhunSureshR added this pull request to the merge queue Jan 30, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 30, 2025
@MidhunSureshR MidhunSureshR added this pull request to the merge queue Jan 30, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 30, 2025
@MidhunSureshR MidhunSureshR added this pull request to the merge queue Jan 30, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 30, 2025
@MidhunSureshR MidhunSureshR added this pull request to the merge queue Feb 2, 2025
Merged via the queue into develop with commit aa01b17 Feb 2, 2025
31 checks passed
@MidhunSureshR MidhunSureshR deleted the midhun/show-back-button-always branch February 2, 2025 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants