[native] remove exitingCallback
ClosedPublic
Actions

Authored by varun on Tue, Nov 12, 1:51 PM.

Tags

None

Referenced Files

	F3404369: D13920.diff
	Tue, Dec 3, 8:51 AM

	Unknown Object (File)
	Thu, Nov 28, 2:14 AM

	Unknown Object (File)
	Tue, Nov 26, 12:09 PM

	Unknown Object (File)
	Mon, Nov 25, 12:10 AM

	Unknown Object (File)
	Sun, Nov 24, 10:12 PM

	Unknown Object (File)
	Sat, Nov 23, 1:40 AM

	Unknown Object (File)
	Thu, Nov 21, 10:16 AM

	Unknown Object (File)
	Wed, Nov 20, 4:56 PM

View All 21 Files

Subscribers

None

Details

Reviewers

ashoat
tomek

Commits

rCOMM485956d0276d: [native] remove exitingCallback

Summary

we should just call navigation.goBack when we reach the end of the ordered list of tips. there's no need for the exitingCallback

Depends on D13917

Test Plan

confirmed that when i pressed ok on the final tip, i was backed out from the backdrop

Diff Detail

Repository

rCOMM Comm

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

varun created this revision.Tue, Nov 12, 1:51 PM

varun added 1 blocking reviewer(s): tomek.Tue, Nov 12, 2:05 PM

Harbormaster completed remote builds in B32637: Diff 45775.Tue, Nov 12, 2:59 PM

varun requested review of this revision.Tue, Nov 12, 2:59 PM

tomek accepted this revision.Wed, Nov 13, 5:20 AM

This revision is now accepted and ready to land.Wed, Nov 13, 5:20 AM

Closed by commit rCOMM485956d0276d: [native] remove exitingCallback (authored by varun). · Explain WhyWed, Nov 13, 11:00 AM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMM485956d0276d: [native] remove exitingCallback.

Revision Contents
Changeset List

Path

Size

native/

components/

nux-tips-context.react.js

5 lines

tooltip/

nux-tips-overlay.react.js

8 lines

Diff 45775

View Options

native/components/nux-tips-context.react.js