Page MenuHomePhabricator
Differential D13180

[native] Create intro tip
ClosedPublic
Actions

Authored by inka on Tue, Aug 27, 10:06 AM.
Tags
None
Referenced Files
F2769147: D13180.diff
Thu, Sep 19, 6:21 PM
Unknown Object (File)
Wed, Sep 18, 7:54 AM
Unknown Object (File)
Mon, Sep 16, 6:33 PM
Unknown Object (File)
Mon, Sep 16, 1:42 PM
Unknown Object (File)
Sun, Sep 15, 4:58 PM
Unknown Object (File)
Sat, Sep 14, 12:11 PM
Unknown Object (File)
Sat, Sep 14, 11:23 AM
Unknown Object (File)
Sat, Sep 14, 9:32 AM
View All 29 Files
Subscribers
ashoat

Details

Reviewers
tomek
will
Commits
rCOMM77917832dbed: [native] Create intro tip
Summary

issue: ENG-9062

Test Plan

Please see the video:

Diff Detail

Repository
rCOMM Comm
Branch
inka/nux
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

inka created this revision.Tue, Aug 27, 10:06 AM
Herald added a subscriber: ashoat. · View Herald TranscriptTue, Aug 27, 10:06 AM
inka added a parent revision: D13179: [native] Make possible to render tip not pointing to anything.Tue, Aug 27, 10:07 AM
inka edited the test plan for this revision. (Show Details)
inka attached a referenced file: F2559147: Simulator Screen Recording - iPhone 15 Pro Max - 2024-08-27 at 18.55.48.mp4. (Show Details)
Harbormaster completed remote builds in B31301: Diff 43702.Tue, Aug 27, 10:33 AM
inka requested review of this revision.Tue, Aug 27, 10:33 AM
tomek accepted this revision.Wed, Aug 28, 4:08 AM
tomek added inline comments.
native/components/nux-tips-context.react.js
112–114 ↗(On Diff #43702)

Can we check this based on the spec?

This revision is now accepted and ready to land.Wed, Aug 28, 4:08 AM
inka added inline comments.Wed, Aug 28, 4:23 AM
native/components/nux-tips-context.react.js
112–114 ↗(On Diff #43702)

I like that! Thanks!

inka updated this revision to Diff 43735.Wed, Aug 28, 4:23 AM

Address review

inka added inline comments.Wed, Aug 28, 5:40 AM
native/components/nux-tips-context.react.js
112–114 ↗(On Diff #43702)

Actually that doesn't work, because location is the location the key route is supposed to pass in its props.

inka updated this revision to Diff 43736.Wed, Aug 28, 5:43 AM

Revert

Harbormaster completed remote builds in B31328: Diff 43736.Wed, Aug 28, 5:59 AM
inka added a child revision: D13194: [native] Simplify nux tips props design.Wed, Aug 28, 7:05 AM
inka added a comment.Wed, Aug 28, 7:07 AM

Changed some things in the design in D13194, so @tomek's suggestion was possible be applied there

Closed by commit rCOMM77917832dbed: [native] Create intro tip (authored by inka). · Explain WhyThu, Aug 29, 6:44 AM
This revision was automatically updated to reflect the committed changes.
inka added a commit: rCOMM77917832dbed: [native] Create intro tip.

Revision Contents
Changeset List

PathSize
native/
components/
12 lines
navigation/
3 lines
17 lines
2 lines
3 lines

Diff 43735

View Options

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

Loading...
View Options

native/navigation/app-navigator.react.js

Loading...
View Options

native/navigation/intro-tip.react.js

Loading...
View Options

native/navigation/overlay-navigator.react.js

Loading...
View Options

native/navigation/route-names.js

Loading...