Skip to content

Commit 521fd1f

Browse files
authored
Merge pull request #5847 from vector-im/bugfix/eric/missing-call-icons
Fixes Missing Call Icons when threads are enabled
2 parents d694ea1 + 9c95168 commit 521fd1f

File tree

2 files changed

+10
-5
lines changed

2 files changed

+10
-5
lines changed

changelog.d/5847.bugfix

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Fixes missing call icons when threads are enabled

vector/src/main/res/menu/menu_timeline.xml

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
<?xml version="1.0" encoding="utf-8"?>
22
<menu xmlns:android="http://schemas.android.com/apk/res/android"
33
xmlns:app="http://schemas.android.com/apk/res-auto"
4-
xmlns:tools="http://schemas.android.com/tools">
4+
xmlns:tools="http://schemas.android.com/tools"
5+
tools:ignore="AlwaysShowAction">
56

67
<item
78
android:id="@+id/timeline_setting"
@@ -19,24 +20,27 @@
1920
android:title="@string/action_invite"
2021
app:showAsAction="never" />
2122

23+
<!-- We always want to show this item as an icon -->
2224
<item
2325
android:id="@+id/video_call"
2426
android:icon="@drawable/ic_video"
2527
android:title="@string/action_video_call"
2628
android:visible="false"
2729
app:iconTint="?colorPrimary"
28-
app:showAsAction="ifRoom"
30+
app:showAsAction="always"
2931
tools:visible="true" />
3032

33+
<!-- We always want to show this item as an icon -->
3134
<item
3235
android:id="@+id/voice_call"
3336
android:icon="@drawable/ic_phone"
34-
android:title="@string/call"
37+
android:title="@string/action_voice_call"
3538
android:visible="false"
3639
app:iconTint="?colorPrimary"
37-
app:showAsAction="ifRoom"
40+
app:showAsAction="always"
3841
tools:visible="true" />
3942

43+
<!-- We always want to show this item as an icon -->
4044
<item
4145
android:id="@+id/menu_timeline_thread_list"
4246
android:title="@string/action_view_threads"
@@ -86,4 +90,4 @@
8690
android:title="@string/action_share"
8791
app:iconTint="?vctr_content_secondary"
8892
app:showAsAction="withText" />
89-
</menu>
93+
</menu>

0 commit comments

Comments
 (0)