Paths

Table of Contentst

Differential D14066

[native] Extract the buttons from the logged out modal
ClosedPublic
Actions

Authored by tomek on Fri, Nov 29, 2:51 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Dec 22, 3:55 AM

	Unknown Object (File)
	Fri, Dec 20, 4:11 AM

	Unknown Object (File)
	Mon, Dec 16, 2:17 PM

	Unknown Object (File)
	Sun, Dec 15, 2:14 PM

	Unknown Object (File)
	Fri, Dec 13, 9:42 PM

	Unknown Object (File)
	Fri, Dec 13, 2:05 AM

	Unknown Object (File)
	Thu, Dec 12, 4:00 PM

	Unknown Object (File)
	Thu, Dec 12, 3:54 AM

View All 26 Files

Subscribers

Details

Reviewers

angelika
ashoat
kamil

Commits

rCOMMb298be8faf64: [native] Extract the buttons from the logged out modal

Summary

These buttons will be used later in the stack. Use the primary buttons instead of the old styling.

Before

old-buttons.png (2×1 px, 1 MB)

After

new-buttons.png (2×1 px, 1 MB)

Depends on D14064

Test Plan

Check if the logged out modal still looks correct.

Diff Detail

Repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

tomek created this revision.Fri, Nov 29, 2:51 AM

Herald added a subscriber: ashoat. · View Herald TranscriptFri, Nov 29, 2:51 AM

tomek added inline comments.Fri, Nov 29, 2:57 AM

native/account/logged-out-modal.react.js
181 ↗	(On Diff #46126)	This prop isn't needed - the `button` style already specifies it

tomek edited the summary of this revision. (Show Details)Fri, Nov 29, 2:58 AM

tomek added inline comments.Fri, Nov 29, 3:01 AM

native/account/logged-out-modal.react.js
561 ↗	(On Diff #46126)	We don't have a list of register buttons anymore so we don't need to set this `key`

Harbormaster completed remote builds in B32895: Diff 46126.Fri, Nov 29, 3:10 AM

tomek requested review of this revision.Fri, Nov 29, 3:10 AM

tomek removed a parent revision: D14064: [native] Make QR screen style consistent with the registration flow.Fri, Nov 29, 4:05 AM

tomek edited the summary of this revision. (Show Details)

tomek edited the summary of this revision. (Show Details)

tomek added a parent revision: D14064: [native] Make QR screen style consistent with the registration flow.

tomek added a child revision: D13907: [native] Introduce a restore screen.

kamil accepted this revision.Mon, Dec 2, 1:09 AM

This revision is now accepted and ready to land.Mon, Dec 2, 1:09 AM

Use the primary button

tomek edited the summary of this revision. (Show Details)Wed, Dec 4, 7:21 AM

tomek added inline comments.

native/account/logged-out-modal.react.js
478–485	We could use `gap` if we had RN 0.71.

tomek requested review of this revision.Wed, Dec 4, 7:23 AM

tomek added inline comments.Wed, Dec 4, 7:26 AM

native/account/logged-out-modal.react.js
458–463	I wasn't sure if we should use the new design (consistent with the registration flow) or the old. If we decide to keep the old one, we can bring back the styling here, and use the `PromptButton` in the restoration flow.

Harbormaster completed remote builds in B32925: Diff 46209.Wed, Dec 4, 7:37 AM

Specify the keys

Harbormaster completed remote builds in B32929: Diff 46213.Thu, Dec 5, 1:56 AM

tomek added a reviewer: ashoat.Fri, Dec 6, 2:40 AM

In this revision, styles are updated, so leaving this @ashoat for review

ashoat accepted this revision.Sat, Dec 7, 12:54 PM

This revision is now accepted and ready to land.Sat, Dec 7, 12:54 PM

tomek edited the summary of this revision. (Show Details)Tue, Dec 10, 9:33 AM

Closed by commit rCOMMb298be8faf64: [native] Extract the buttons from the logged out modal (authored by tomek). · Explain WhyTue, Dec 10, 10:12 AM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMMb298be8faf64: [native] Extract the buttons from the logged out modal.

Revision Contents
Changeset List

Path

Size

native/

account/

logged-out-modal.react.js

131 lines

prompt-button.react.js

58 lines

components/

primary-button.react.js

30 lines

Diff 46209

native/account/logged-out-modal.react.js

Loading...

native/account/prompt-button.react.js

Loading...

native/components/primary-button.react.js

Loading...