Skip to content

Commit

Permalink
Merge pull request #3743 from kiwix/Fix#3741
Browse files Browse the repository at this point in the history
Fixed: After restoring the tab it hides a few area of the article from the top.
  • Loading branch information
kelson42 authored Mar 14, 2024
2 parents 341ce65 + 060e410 commit f7a34c1
Showing 1 changed file with 13 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,6 @@ import android.os.IBinder
import android.os.Looper
import android.provider.Settings
import android.util.AttributeSet
import org.kiwix.kiwixmobile.core.utils.files.Log
import android.view.ActionMode
import android.view.LayoutInflater
import android.view.Menu
Expand All @@ -57,6 +56,7 @@ import android.widget.Button
import android.widget.FrameLayout
import android.widget.ImageButton
import android.widget.ImageView
import android.widget.LinearLayout
import android.widget.TextView
import android.widget.Toast
import androidx.activity.result.ActivityResultLauncher
Expand Down Expand Up @@ -159,6 +159,7 @@ import org.kiwix.kiwixmobile.core.utils.dialog.DialogShower
import org.kiwix.kiwixmobile.core.utils.dialog.KiwixDialog
import org.kiwix.kiwixmobile.core.utils.files.FileUtils.deleteCachedFiles
import org.kiwix.kiwixmobile.core.utils.files.FileUtils.readFile
import org.kiwix.kiwixmobile.core.utils.files.Log
import org.kiwix.kiwixmobile.core.utils.titleToUrl
import org.kiwix.kiwixmobile.core.utils.urlSuffixToParsableUrl
import org.kiwix.libkiwix.Book
Expand Down Expand Up @@ -1226,6 +1227,16 @@ abstract class CoreReaderFragment :
reopenBook()
}
tempWebViewForUndo?.let {
if (tabSwitcherRoot?.visibility == View.GONE) {
// Remove the top margin from the webView when the tabSwitcher is not visible.
// We have added this margin in `TabsAdapter` to not show the top margin in tabs.
// `tempWebViewForUndo` saved with that margin so before showing it to the `contentFrame`
// We need to set full width and height for properly showing the content of webView.
it.layoutParams = LinearLayout.LayoutParams(
LinearLayout.LayoutParams.MATCH_PARENT,
LinearLayout.LayoutParams.MATCH_PARENT
)
}
zimReaderContainer?.setZimFile(tempZimFileForUndo)
webViewList.add(index, it)
tabsAdapter?.notifyDataSetChanged()
Expand All @@ -1248,10 +1259,7 @@ abstract class CoreReaderFragment :
contentFrame?.let {
it.removeAllViews()
val webView = safelyGetWebView(position) ?: return@selectTab
webView.parent?.let {
(webView.parent as ViewGroup).removeView(webView)
}
it.addView(webView)
safelyAddWebView(webView)
tabsAdapter?.selected = currentWebViewIndex
updateBottomToolbarVisibility()
loadPrefs()
Expand Down

0 comments on commit f7a34c1

Please sign in to comment.