-
Notifications
You must be signed in to change notification settings - Fork 6
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
Fix safe area observing for example query #124 #126
Fix safe area observing for example query #124 #126
Conversation
…cell to denote navigation
…slate#112 - Fork's code review feedback
* Issue 80,81 Delete all option * Added confirmation dialog and changed the placement
* Issue 104 * Updates in Device info section and connectivity tab * Added a filter as per netUtils interface function * Updates 104 Removed the data usage from interface and added 3 rows in data usage in device tab.
Add WHOIS XML API geolocation endpoint twodayslate#76
ec3730/Views/Host/HostBarView.swift
Outdated
.ignoresSafeArea(.all, edges: .horizontal) | ||
) | ||
.ignoresSafeArea() | ||
}.ignoresSafeArea() |
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.
Let's just have the relevant changes in the PR
Please rebase |
https://we.tl/t-OjRjdFyePK already up to date |
@@ -10,7 +10,7 @@ struct HostViewSectionFocusView: View { | |||
ScrollView { | |||
HostViewSectionContent(sectionModel: model, canQuery: true) | |||
} | |||
HostBarView(url: url, date: date) | |||
HostBarView(url: url, date: date).background(Color(UIColor.systemGroupedBackground)) |
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.
Why is this change necessary?
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.
HostBarView(url: url, date: date).background(Color(UIColor.systemGroupedBackground))
https://we.tl/t-FPltFCM832 Without background color , it will show like the given screenshot, there will be no background color.
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.
Shouldn't this just ignore the safe area?
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.
https://we.tl/t-2KPswENwxO attached screenshots , in case we use only the ignore safe area , or even without that there is no background color, but if we want background color , then we need to give background color
If you check this link , Background color also given same as I implemented in snippet. Please check these screenshots: https://we.tl/t-ujdUL7nHMj If background color is not provided it will show as: https://we.tl/t-SS3ovvpGF7 |
This seemed to work for me
|
It's also used in HostResult.swift |
Please Suggest , on which ticket we can work upon next 1.#108 |
https://we.tl/t-QxHhGQZNh6 This view has tabbar below , please check screenshot, If any updation required let me know. |
Can that use the safe area unset as well? |
No description provided.