Skip to content

Commit 48680c3

Browse files
authored
remove spaces feedback from left panel (#5686)
1 parent 948566c commit 48680c3

File tree

6 files changed

+5
-48
lines changed

6 files changed

+5
-48
lines changed

changelog.d/5486.misc

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Spaces feedback section is removed from left panel

vector/src/main/java/im/vector/app/features/spaces/SpaceBetaHeaderItem.kt

Lines changed: 1 addition & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -16,27 +16,12 @@
1616

1717
package im.vector.app.features.spaces
1818

19-
import android.view.View
20-
import com.airbnb.epoxy.EpoxyAttribute
2119
import com.airbnb.epoxy.EpoxyModelClass
2220
import im.vector.app.R
23-
import im.vector.app.core.epoxy.ClickListener
2421
import im.vector.app.core.epoxy.VectorEpoxyHolder
2522
import im.vector.app.core.epoxy.VectorEpoxyModel
26-
import im.vector.app.core.epoxy.onClick
2723

2824
@EpoxyModelClass(layout = R.layout.item_space_beta_header)
2925
abstract class SpaceBetaHeaderItem : VectorEpoxyModel<SpaceBetaHeaderItem.Holder>() {
30-
31-
@EpoxyAttribute(EpoxyAttribute.Option.DoNotHash)
32-
var clickAction: ClickListener? = null
33-
34-
override fun bind(holder: Holder) {
35-
super.bind(holder)
36-
holder.feedBackAction.onClick(clickAction)
37-
}
38-
39-
class Holder : VectorEpoxyHolder() {
40-
val feedBackAction by bind<View>(R.id.spaceBetaFeedbackAction)
41-
}
26+
class Holder : VectorEpoxyHolder()
4227
}

vector/src/main/java/im/vector/app/features/spaces/SpaceSummaryController.kt

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -109,9 +109,6 @@ class SpaceSummaryController @Inject constructor(
109109
val host = this
110110
spaceBetaHeaderItem {
111111
id("beta_header")
112-
clickAction {
113-
host.callback?.sendFeedBack()
114-
}
115112
}
116113

117114
// show invites on top

vector/src/main/res/drawable/ic_feedback.xml

Lines changed: 0 additions & 14 deletions
This file was deleted.

vector/src/main/res/layout/item_space_beta_header.xml

Lines changed: 1 addition & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
android:paddingStart="16dp"
99
android:paddingTop="8dp"
1010
android:paddingEnd="16dp"
11+
android:paddingBottom="16dp"
1112
tools:viewBindingIgnore="true">
1213

1314
<TextView
@@ -27,18 +28,4 @@
2728
android:text="@string/spaces_beta_welcome_to_spaces_desc"
2829
android:textColor="?vctr_content_secondary" />
2930

30-
<TextView
31-
android:id="@+id/spaceBetaFeedbackAction"
32-
style="@style/Widget.Vector.TextView.Body"
33-
android:layout_width="wrap_content"
34-
android:layout_height="wrap_content"
35-
android:drawablePadding="8dp"
36-
android:paddingStart="0dp"
37-
android:paddingTop="8dp"
38-
android:paddingEnd="16dp"
39-
android:paddingBottom="8dp"
40-
android:text="@string/give_feedback"
41-
android:textColor="@color/vector_info_color"
42-
app:drawableStartCompat="@drawable/ic_feedback" />
43-
4431
</LinearLayout>

vector/src/main/res/values/strings.xml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1660,7 +1660,8 @@
16601660
<string name="you_may_contact_me">You may contact me if you have any follow up questions</string>
16611661
<string name="feedback_sent">Thanks, your feedback has been successfully sent</string>
16621662
<string name="feedback_failed">The feedback failed to be sent (%s)</string>
1663-
<string name="give_feedback">Give Feedback</string>
1663+
<!-- TODO TO BE REMOVED -->
1664+
<string name="give_feedback" tools:ignore="UnusedResources">Give Feedback</string>
16641665
<string name="give_feedback_threads">Give Feedback</string>
16651666
<string name="send_feedback_threads_title">Threads Beta feedback</string>
16661667
<string name="send_feedback_threads_info">Threads are a work in progress with new, exciting upcoming features, such as improved notifications. We’d love to hear your feedback!</string>

0 commit comments

Comments
 (0)