-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add FXIOS-10007 [Bookmarks Evolution] Add sign in button to empty state #23442
Add FXIOS-10007 [Bookmarks Evolution] Add sign in button to empty state #23442
Conversation
@@ -182,6 +221,26 @@ class BookmarksCoordinator: BaseCoordinator, | |||
return controller | |||
} | |||
|
|||
private func makeSignInController() -> UIViewController { | |||
let fxaParams = FxALaunchParams(entrypoint: .libraryPanel, query: [:]) | |||
// NEED TO CHANGE PARENT TYPE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still todo?
// NEED TO CHANGE PARENT TYPE |
key: "Bookmarks.EmptyState.Root.ButtonTitle.v135", | ||
tableName: "Bookmarks", | ||
value: "Sign in to Sync", | ||
comment: "The button title for the sign in button on the placeholder screen shown when there are no saved bookmarks, located at the root level of the bookmarks panel within the libray modal. This button starts the initiates allowing users to sign in to their Mozilla Account to sync data") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo + I am not sure I get the sentence "This button starts the initiates [...]"
comment: "The button title for the sign in button on the placeholder screen shown when there are no saved bookmarks, located at the root level of the bookmarks panel within the libray modal. This button starts the initiates allowing users to sign in to their Mozilla Account to sync data") | |
comment: "The button title for the sign in button on the placeholder screen shown when there are no saved bookmarks, located at the root level of the bookmarks panel within the library modal. This button starts the flow allowing users to sign in to their Mozilla Account to sync data") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
woah, yea I don't understand what I was going for there either 😂
0797c41
to
e41f37a
Compare
This reverts commit e151b0a.
Client.app: Coverage: 31.46
Generated by 🚫 Danger Swift against 6d5d1bd |
📜 Tickets
Jira ticket
Github issue
💡 Description
📝 Notes:
FirefoxAccountSignInViewController
is presented as a fullscreen modal for consistency with the remote tabs panel sign in flow📷 Screenshots
📝 Checklist
You have to check all boxes before merging
@Mergifyio backport release/v120
)