Skip to content
This repository was archived by the owner on Sep 3, 2024. It is now read-only.

Commit 9b9aad7

Browse files
authored
app/PopupMenu: minor ui fixes (#63)
Signed-off-by: BlackMesa123 <[email protected]>
1 parent 7d82261 commit 9b9aad7

File tree

3 files changed

+17
-11
lines changed

3 files changed

+17
-11
lines changed

yanndroid/oneui/src/main/java/de/dlyt/yanndroid/oneui/menu/PopupMenu.java

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@
22

33
import android.content.Context;
44
import android.graphics.Typeface;
5-
import android.text.TextUtils;
65
import android.util.TypedValue;
6+
import android.view.Gravity;
77
import android.view.KeyEvent;
88
import android.view.MotionEvent;
99
import android.view.View;
@@ -156,16 +156,16 @@ public boolean onTouch(View view, MotionEvent motionEvent) {
156156

157157
private TextView createTitleView() {
158158
TextView titleView = new TextView(context);
159-
titleView.setPadding(context.getResources().getDimensionPixelSize(R.dimen.sesl_popup_menu_item_start_padding),
160-
context.getResources().getDimensionPixelSize(R.dimen.sesl_popup_menu_item_top_padding),
161-
context.getResources().getDimensionPixelSize(R.dimen.sesl_popup_menu_item_end_padding),
162-
context.getResources().getDimensionPixelSize(R.dimen.sesl_menu_popup_bottom_padding));
163-
titleView.setTextDirection(View.TEXT_DIRECTION_LOCALE);
159+
titleView.setGravity(Gravity.CENTER_VERTICAL);
160+
titleView.setPaddingRelative(context.getResources().getDimensionPixelSize(R.dimen.sesl_context_menu_title_start_padding),
161+
context.getResources().getDimensionPixelSize(R.dimen.sesl_context_menu_title_top_padding),
162+
context.getResources().getDimensionPixelSize(R.dimen.sesl_context_menu_title_end_padding),
163+
context.getResources().getDimensionPixelSize(R.dimen.sesl_context_menu_title_bottom_padding));
164164
titleView.setTextColor(context.getResources().getColor(R.color.item_color));
165165
titleView.setTypeface(ResourcesCompat.getFont(context, R.font.sesl_roboto), Typeface.BOLD);
166-
titleView.setEllipsize(TextUtils.TruncateAt.END);
167-
titleView.setMaxLines(1);
168-
titleView.setTextSize(16);
166+
titleView.setTextAlignment(View.TEXT_ALIGNMENT_VIEW_START);
167+
titleView.setMaxLines(2);
168+
titleView.setTextSize(TypedValue.COMPLEX_UNIT_PX, context.getResources().getDimensionPixelSize(R.dimen.sesl_context_menu_title_text_size));
169169
titleView.setText(title);
170170
return titleView;
171171
}

yanndroid/oneui/src/main/res/layout/menu_item_popup_item.xml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,8 @@
5252

5353
<CheckBox
5454
android:id="@+id/menu_item_check_box"
55-
android:layout_width="wrap_content"
56-
android:layout_height="wrap_content"
55+
android:layout_width="32.0dip"
56+
android:layout_height="24.0dip"
5757
android:background="@null"
5858
android:clickable="false"
5959
android:focusable="false"

yanndroid/oneui/src/main/res/values/dimens.xml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,6 +66,12 @@
6666
<dimen name="sesl_popup_menu_item_start_padding">24.0dip</dimen>
6767
<dimen name="sesl_popup_menu_item_top_padding">13.0dip</dimen>
6868

69+
<dimen name="sesl_context_menu_title_bottom_padding">6.0dip</dimen>
70+
<dimen name="sesl_context_menu_title_end_padding">24.0dip</dimen>
71+
<dimen name="sesl_context_menu_title_start_padding">24.0dip</dimen>
72+
<dimen name="sesl_context_menu_title_text_size">17.0sp</dimen>
73+
<dimen name="sesl_context_menu_title_top_padding">14.0dip</dimen>
74+
6975
<dimen name="sesl_checked_text_padding">18.0dip</dimen>
7076

7177

0 commit comments

Comments
 (0)