Titanium JIRA Archive
Titanium SDK/CLI (TIMOB)

[TIMOB-26834] Android: NavigationView root window displays back button

GitHub Issuen/a
TypeBug
PriorityCritical
StatusClosed
ResolutionFixed
Resolution Date2019-04-03T09:13:52.000+0000
Affected Version/sRelease 8.0.0
Fix Version/sRelease 8.0.1
ComponentsAndroid
Labelsn/a
ReporterRene Pot
AssigneeYordan Banev
Created2019-02-18T11:26:01.000+0000
Updated2019-04-03T09:13:52.000+0000

Description

When creating a basic NavigationWindow in Android it displays a back button (see attachment) where it shouldn't, since it is the root window.

	<NavigationWindow>
		<Window class="container">
			<Label id="label" onClick="doClick">Hello, World</Label>
		</Window>
	</NavigationWindow>

Basically, the hello world app with a wrapping NavigationWindow *Expected result* It should not display a back button on root window

Attachments

FileDateSize
Screenshot_1550489020.png2019-02-18T11:25:51.000+000034484

Comments

  1. Yordan Banev 2019-02-19

    PR: https://github.com/appcelerator/titanium_mobile/pull/10710
  2. Rene Pot 2019-02-19

    I can confirm my test case is now working with this PR
  3. Joshua Quick 2019-02-19

    This sounds like a major parity issue. We should probably make this change in 8.0.0.GA too.
  4. Yordan Banev 2019-02-20

    We talked with [~topener] about it and agreed on the same - it will be good to have it in 8.0.0.GA. I will create a backport.
  5. Rene Pot 2019-02-20

    Yep, it basically is a bug in the implementation of NavigationWindow on Android. So for sure 8.0
  6. Yordan Banev 2019-03-21

    I have missed to backport this for the GA release of 8.0.0, but lets get it into 8.0.1. PR (8_0_X): https://github.com/appcelerator/titanium_mobile/pull/10791
  7. Samir Mohammed 2019-04-02

    FR Passed, Waiting on Jenkins builds.
  8. Samir Mohammed 2019-04-03

    *Closing ticket* Fix verified in SDK Version 8.0.1.v20190402094527 and SDK Version 8.1.0.v20190402113644 Test and other information can be found at: Master: https://github.com/appcelerator/titanium_mobile/pull/10710 8_0_X: https://github.com/appcelerator/titanium_mobile/pull/10791

JSON Source