blallo
/
fenix
1
0
Fork 0
Code Releases Activity

For #13390 - Fix for stuck readerMode appearance controls

master
codrut.topliceanu 2020-08-13 10:01:55 +03:00 committed by Mugurell
parent 0356d86784
commit f0a045f9b9
2 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
app/src/main/java/org/mozilla/fenix/browser/readermode/ReaderModeController.kt
View File

@ -27,7 +27,10 @@ class DefaultReaderModeController(
private val isPrivate: Boolean = false
) : ReaderModeController {
override fun hideReaderView() {
readerViewFeature.withFeature { it.hideReaderView() }
readerViewFeature.withFeature {
it.hideReaderView()
it.hideControls()
}
}
override fun showReaderView() {

2
app/src/test/java/org/mozilla/fenix/browser/readermode/DefaultReaderModeControllerTest.kt
View File

@ -54,6 +54,7 @@ class DefaultReaderModeControllerTest {
every { readerViewFeature.hideReaderView() } returns Unit
every { readerViewFeature.showReaderView() } returns Unit
every { readerViewFeature.showControls() } returns Unit
every { readerViewFeature.hideControls() } returns Unit
}
@Test
@ -61,6 +62,7 @@ class DefaultReaderModeControllerTest {
val controller = DefaultReaderModeController(featureWrapper, readerViewControlsBar)
controller.hideReaderView()
verify { readerViewFeature.hideReaderView() }
verify { readerViewFeature.hideControls() }
}
@Test