Skip to content
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 Hacker News guidelines button to login screen #354

Merged
merged 2 commits into from
Feb 5, 2025
Merged

Conversation

Itaybre
Copy link
Collaborator

@Itaybre Itaybre commented Feb 4, 2025

Added button to open Hacker New's Guidelines
Simulator Screenshot - iPhone 16 Pro - 2025-02-04 at 17 52 18

@@ -96,6 +110,11 @@ struct LoginScreen: View {
.padding(32)
.background(.surface)
}

func openURL(_ urlString: String) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

think you forgot to use this? same why is above using "https://apple.com"?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤦‍♂️

@@ -64,6 +64,20 @@ struct LoginScreen: View {
.font(.ibmPlexMono(.regular, size: 14))
}


HStack(spacing: 0) {
Text("By signin in, you agree to the ")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"By signing in"

Copy link

emerge-tools bot commented Feb 4, 2025

📸 Snapshot Test

2 modified, 86 unchanged

Name Added Removed Modified Renamed Unchanged Errored Approval
HackerNews
com.emergetools.hackernews.snapshots
0 0 2 0 68 0 ⏳ Needs approval
HackerNews Swift-Snapshot-Testing
com.emerge.hn.Hacker-News.swiftsnapshottesting
0 0 0 0 18 0 N/A
HackerNews
com.emergetools.hackernews
0 0 0 0 0 0 N/A
HackerNews
com.emergetools.hackernews.adhoc
0 0 0 0 0 0 N/A

🛸 Powered by Emerge Tools

@Itaybre Itaybre merged commit 7df8ff5 into main Feb 5, 2025
6 checks passed
@Itaybre Itaybre deleted the feature/guidelines branch February 5, 2025 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants