Page MenuHomePhabricator
Differential D13136

[native] Fix blinking overlay background
ClosedPublic
Actions

Authored by inka on Thu, Aug 22, 2:37 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Sep 17, 2:10 AM
Unknown Object (File)
Mon, Sep 16, 8:56 PM
Unknown Object (File)
Sun, Sep 15, 6:29 PM
Unknown Object (File)
Sun, Sep 15, 1:30 PM
Unknown Object (File)
Mon, Sep 9, 6:12 PM
Unknown Object (File)
Sat, Sep 7, 9:57 AM
Unknown Object (File)
Fri, Sep 6, 2:56 PM
Unknown Object (File)
Fri, Sep 6, 11:28 AM
View All 27 Files
Subscribers
ashoat

Details

Reviewers
tomek
will
Commits
rCOMM7519541150d4: [native] Fix blinking overlay background
Summary

issue: ENG-9058

Test Plan

Please see the video:

(this is with a sleep(2000) in the nux handler, because otherwise the tip gets dismissed for some reason. Created https://linear.app/comm/issue/ENG-9085/fix-tip-getting-dismissed to track

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

inka created this revision.Thu, Aug 22, 2:37 AM
Herald added a subscriber: ashoat. · View Herald TranscriptThu, Aug 22, 2:37 AM
inka edited the test plan for this revision. (Show Details)Thu, Aug 22, 2:42 AM
inka attached a referenced file: F2510688: Simulator Screen Recording - iPhone 15 Pro Max - 2024-08-22 at 11.12.27.mp4. (Show Details)
inka updated this revision to Diff 43567.Thu, Aug 22, 2:48 AM

Rebase

Harbormaster completed remote builds in B31209: Diff 43565.Thu, Aug 22, 2:55 AM
Harbormaster completed remote builds in B31211: Diff 43567.Thu, Aug 22, 3:04 AM
inka requested review of this revision.Thu, Aug 22, 3:04 AM
inka added a comment.Thu, Aug 22, 3:31 AM

Turns out that the tip being dismissed is only happening after this diff. Trying to figure out why

inka added a parent revision: D13127: [native] Update styling.Fri, Aug 23, 5:09 AM

Moved the handler in D13085, the tip is not dismissed anymore

tomek accepted this revision.Fri, Aug 23, 5:17 AM
This revision is now accepted and ready to land.Fri, Aug 23, 5:17 AM
inka added a child revision: D13176: [native] Fix light mode highlighting.Tue, Aug 27, 7:39 AM
Closed by commit rCOMM7519541150d4: [native] Fix blinking overlay background (authored by inka). · Explain WhyThu, Aug 29, 6:43 AM
This revision was automatically updated to reflect the committed changes.
inka added a commit: rCOMM7519541150d4: [native] Fix blinking overlay background.

Revision Contents
Changeset List

Diff 43802

View Options

native/components/nux-handler.react.js

Loading...
View Options

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

Loading...
View Options

native/navigation/app-navigator.react.js

Loading...
View Options

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

Loading...
View Options

native/navigation/overlay-navigator.react.js

Loading...
View Options

native/navigation/route-names.js

Loading...
View Options

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

Loading...