Page MenuHomePhabricator
Differential D13917

[native] make the list of NUX tips to be displayed configurable
ClosedPublic
Actions

Authored by varun on Nov 12 2024, 12:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 4:12 PM
Unknown Object (File)
Wed, Dec 18, 2:55 PM
Unknown Object (File)
Mon, Dec 16, 1:10 PM
Unknown Object (File)
Fri, Dec 13, 11:17 PM
Unknown Object (File)
Fri, Dec 13, 11:17 PM
Unknown Object (File)
Fri, Dec 13, 11:15 PM
Unknown Object (File)
Fri, Dec 13, 11:09 PM
Unknown Object (File)
Sun, Nov 24, 2:35 PM
View All 26 Files
Subscribers
None

Details

Reviewers
ashoat
tomek
Commits
rCOMMe8675e276b22: [native] make the list of NUX tips to be displayed configurable
Summary

i'll be introducing a new tip that i want to display from the NUX tips overlay, separate from the tips that display on fist launch.

in order to do this, i need to be able to navigate to my new tip directly from the backdrop, as opposed to going through the 4 existing tips first.

in this diff, i remove nextTip from NUXTipParams. instead, in the NUX handler, we initialize an ordered list of tips and use this list to determine the next tip to display

https://linear.app/comm/issue/ENG-9913/make-the-list-of-nux-tips-to-be-displayed-configurable

Test Plan

on a fresh install, confirmed that the same 4 tips appeared as before, and that navigating through them was unchanged

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

varun created this revision.Nov 12 2024, 12:45 PM
varun edited the summary of this revision. (Show Details)Nov 12 2024, 1:05 PM
Harbormaster completed remote builds in B32628: Diff 45764.Nov 12 2024, 1:47 PM
varun requested review of this revision.Nov 12 2024, 1:47 PM
varun added a child revision: D13920: [native] remove exitingCallback.Nov 12 2024, 1:51 PM
ashoat accepted this revision.Nov 12 2024, 2:01 PM
ashoat added 1 blocking reviewer(s): tomek.

Looks good to me! I'm not familiar with the NUX code (it was introduced while I was away) so would appreciate if @tomek could take a look too

native/components/nux-handler.react.js
24–27 ↗(On Diff #45764)

No reason to define this in the component. Can you move it to the top-level scope?

tomek accepted this revision.Nov 13 2024, 5:18 AM
This revision is now accepted and ready to land.Nov 13 2024, 5:18 AM
Closed by commit rCOMMe8675e276b22: [native] make the list of NUX tips to be displayed configurable (authored by varun). · Explain WhyNov 13 2024, 11:00 AM
This revision was automatically updated to reflect the committed changes.
varun added a commit: rCOMMe8675e276b22: [native] make the list of NUX tips to be displayed configurable.
varun mentioned this in D14054: [native] navigate to community directory nux tip if user declines prompt or closes bottom sheet.Wed, Nov 27, 11:10 AM

Revision Contents
Changeset List

PathSize
native/
components/
16 lines
5 lines
navigation/
26 lines
tooltip/
31 lines

Diff 45799

View Options

native/components/nux-handler.react.js

Loading...
View Options

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

Loading...
View Options

native/navigation/nux-tip-overlay-backdrop.react.js

Loading...
View Options

native/tooltip/nux-tips-overlay.react.js

Loading...