-
Notifications
You must be signed in to change notification settings - Fork 200
Conversation
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.
| Image(systemName: iconName) | ||
| } | ||
| .font(.title2) | ||
| .foregroundColor(Color(red: 0.0, green: 0.18, blue: 0.39)) |
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.
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.
yeah, thanks, fix it
| .background(Color(UIColor.secondarySystemGroupedBackground)) | ||
| .cornerRadius(10) | ||
| .shadow(radius: 2) |
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.
You can use a GroupBox to get a similar look but native to the platform.
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.
yeah, thanks, fix it
| private extension URL { | ||
| static let edrlab = URL(string: "https://www.edrlab.org/")! | ||
| static let license = URL(string: "https://opensource.org/licenses/BSD-3-Clause")! | ||
| } |
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.
Nice!
|
fix iPad layout |
|
fix iPad layout |
|
@mickael-menu I think I fix all issues, can you check this MR one more time plz? |
mickael-menu
left a comment
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.
Thank you for making the changes @FuzzzzyBoy