-
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
Changes from 108 commits
e102d56
0252719
7748f80
2db8bbf
ab371ca
0297132
909b279
a9dd7f3
a0493c5
34d7072
a61ffe8
453daa8
f6a81ab
1d64f9e
e256236
b90f346
43f3ad3
ce8c869
ffc3f7c
f809802
71ebafa
5326a2b
9bd7e13
88bec38
79709a4
02e99b2
9c4bb27
262be46
b0ce950
f0bca54
58b8cb4
ce8f294
9d44c95
d35fc6d
f684104
3b3cc8b
63d7a17
d7709d1
0de1df8
68b4b51
04de087
dc6e671
904c877
a6a48e1
49050fb
82aaefb
828f981
d050837
5a860c2
0c0fad9
dc50f15
b325190
b8f7490
6593ef6
607f3c1
8e03bbb
c2a21ca
5734360
05f4650
c628d1c
deafa74
939a213
c1c52c4
fb4ae86
15dc4b6
8ad4fc9
e306852
df90a74
778502d
d6b77a4
379b940
0841bbd
efa8131
698d5da
729c419
c235e29
3cc5507
6c8fe48
be4522d
d7d982a
24d842a
2beb4f1
967ae0e
6905733
8ad867f
cc5f410
437258e
360de76
aad51ec
3ef81d9
e7ee534
d9dd2bf
8c9fc5d
ba5ed71
fdc8bd8
ab01057
1f66def
154b23d
fbeb371
6517730
732e123
066b51e
f19795a
9e7629d
ff0c0f8
f97a4e6
f191068
73dc554
5d9b463
576a420
2d0cfe9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -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 commentThe 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 commentThe 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 commentThe 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 commentThe 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 |
||
} | ||
.navigationTitle(model.service.name) | ||
.navigationBarTitleDisplayMode(.inline) | ||
|
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