Skip to content

Commit 453c448

Browse files
authored
Merge pull request #677 from esensar/fix/676-big-text-size-note-titles
Take all text scaling into account when determining pagerTabStrip height
2 parents c3278ea + cb152bc commit 453c448

File tree

1 file changed

+7
-3
lines changed
  • app/src/main/kotlin/com/simplemobiletools/notes/pro/activities

1 file changed

+7
-3
lines changed

app/src/main/kotlin/com/simplemobiletools/notes/pro/activities/MainActivity.kt

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -100,9 +100,10 @@ class MainActivity : SimpleActivity() {
100100

101101
initViewPager(intent.getLongExtra(OPEN_NOTE_ID, -1L))
102102
binding.pagerTabStrip.drawFullUnderline = false
103-
binding.pagerTabStrip.setTextSize(TypedValue.COMPLEX_UNIT_PX, getPercentageFontSize())
103+
val textSize = getPercentageFontSize()
104+
binding.pagerTabStrip.setTextSize(TypedValue.COMPLEX_UNIT_PX, textSize)
104105
binding.pagerTabStrip.layoutParams.height =
105-
(binding.pagerTabStrip.height + resources.getDimension(com.simplemobiletools.commons.R.dimen.activity_margin) * 2 * (config.fontSizePercentage / 100f)).toInt()
106+
(textSize + resources.getDimension(com.simplemobiletools.commons.R.dimen.medium_margin) * 2).toInt()
106107
(binding.pagerTabStrip.layoutParams as ViewPager.LayoutParams).isDecor = true
107108
checkWhatsNewDialog()
108109
checkIntents(intent)
@@ -138,7 +139,10 @@ class MainActivity : SimpleActivity() {
138139

139140
refreshMenuItems()
140141
binding.pagerTabStrip.apply {
141-
setTextSize(TypedValue.COMPLEX_UNIT_PX, getPercentageFontSize())
142+
val textSize = getPercentageFontSize()
143+
setTextSize(TypedValue.COMPLEX_UNIT_PX, textSize)
144+
layoutParams.height =
145+
(textSize + resources.getDimension(com.simplemobiletools.commons.R.dimen.medium_margin) * 2).toInt()
142146
setGravity(Gravity.CENTER_VERTICAL)
143147
setNonPrimaryAlpha(0.4f)
144148
setTextColor(getProperPrimaryColor())

0 commit comments

Comments
 (0)