Changed ordering of Room details screen (#5290)
* Changed ordering of Room details screen Added media, polls and pinned section on top, because the user is more likely to access those. * Update snapshots.
This commit is contained in:
@@ -20,14 +20,14 @@ struct RoomDetailsScreen: View {
|
|||||||
|
|
||||||
topicSection
|
topicSection
|
||||||
|
|
||||||
|
aboutSection
|
||||||
|
|
||||||
configurationSection
|
configurationSection
|
||||||
|
|
||||||
if context.viewState.dmRecipientInfo == nil {
|
if context.viewState.dmRecipientInfo == nil {
|
||||||
peopleSection
|
peopleSection
|
||||||
}
|
}
|
||||||
|
|
||||||
aboutSection
|
|
||||||
|
|
||||||
securitySection
|
securitySection
|
||||||
|
|
||||||
if let recipient = context.viewState.dmRecipientInfo?.member {
|
if let recipient = context.viewState.dmRecipientInfo?.member {
|
||||||
@@ -136,6 +136,11 @@ struct RoomDetailsScreen: View {
|
|||||||
|
|
||||||
private var aboutSection: some View {
|
private var aboutSection: some View {
|
||||||
Section {
|
Section {
|
||||||
|
ListRow(label: .default(title: L10n.screenMediaBrowserTitle, icon: \.image),
|
||||||
|
kind: .navigationLink {
|
||||||
|
context.send(viewAction: .processTapMediaEvents)
|
||||||
|
})
|
||||||
|
|
||||||
ListRow(label: .default(title: L10n.screenRoomDetailsPinnedEventsRowTitle, icon: \.pin),
|
ListRow(label: .default(title: L10n.screenRoomDetailsPinnedEventsRowTitle, icon: \.pin),
|
||||||
details: context.viewState.pinnedEventsActionState.isLoading ? .isWaiting(true) : .title(context.viewState.pinnedEventsActionState.count),
|
details: context.viewState.pinnedEventsActionState.isLoading ? .isWaiting(true) : .title(context.viewState.pinnedEventsActionState.count),
|
||||||
kind: context.viewState.pinnedEventsActionState.isLoading ? .label : .navigationLink {
|
kind: context.viewState.pinnedEventsActionState.isLoading ? .label : .navigationLink {
|
||||||
@@ -148,11 +153,6 @@ struct RoomDetailsScreen: View {
|
|||||||
context.send(viewAction: .processTapPolls)
|
context.send(viewAction: .processTapPolls)
|
||||||
})
|
})
|
||||||
.accessibilityIdentifier(A11yIdentifiers.roomDetailsScreen.pollsHistory)
|
.accessibilityIdentifier(A11yIdentifiers.roomDetailsScreen.pollsHistory)
|
||||||
|
|
||||||
ListRow(label: .default(title: L10n.screenMediaBrowserTitle, icon: \.image),
|
|
||||||
kind: .navigationLink {
|
|
||||||
context.send(viewAction: .processTapMediaEvents)
|
|
||||||
})
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Reference in New Issue
Block a user