-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[TIMOB-16622] iOS:Replace UISearchDisplayController with UISearchController #9066
Changes from 13 commits
5d2ee66
65e8ec4
8f0bda9
326854a
cc5626d
92c5bd0
715fd6a
82777b6
da16438
11151f1
9dcf45d
8276351
32ad6f8
7c492f6
0c21389
019f1f5
0870b11
9089e31
7b84063
ad81b10
333ac7d
d3eacd2
4396c00
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 |
---|---|---|
|
@@ -1069,6 +1069,10 @@ properties: | |
since: 6.1.0 | ||
platforms: [iphone, ipad] | ||
availability: creation | ||
removed: | ||
since: "6.2.0" | ||
notes: | | ||
The background color of the search table-view is determined by it's table-view, so there is no additional property required. | ||
|
||
- name: resultsSeparatorColor | ||
summary: | | ||
|
@@ -1083,6 +1087,10 @@ properties: | |
since: 6.1.0 | ||
platforms: [iphone, ipad] | ||
availability: creation | ||
removed: | ||
since: "6.2.0" | ||
notes: | | ||
The separator color of the search table-view is determined by it's table-view, so there is no additional property required. | ||
|
||
- name: resultsSeparatorStyle | ||
summary: The separator style of the search results (iOS-only). | ||
|
@@ -1091,6 +1099,10 @@ properties: | |
since: 6.1.0 | ||
platforms: [iphone, ipad] | ||
availability: creation | ||
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. Removing them is not an option. Can't we just grab the values and style the search-tableview accordingly? 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. Is it really necessary? Problem mentioned in ticket will not arise. 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. Agree, but we need to state that, by using: removed:
since: "6.2.0"
notes: |
The styling of the search table-view is determined by it's table-view, so there
is no additional styling required. |
||
removed: | ||
since: "6.2.0" | ||
notes: | | ||
The styling of the search table-view is determined by it's table-view, so there is no additional styling required. | ||
|
||
- name: resultsSeparatorInsets | ||
summary: | | ||
|
@@ -1104,7 +1116,11 @@ properties: | |
since: 6.1.0 | ||
platforms: [iphone, ipad] | ||
availability: creation | ||
|
||
removed: | ||
since: "6.2.0" | ||
notes: | | ||
The separator insets of the search table-view is determined by it's table-view, so there is no additional property required. | ||
|
||
methods: | ||
- name: scrollToItem | ||
summary: Scrolls to a specific item. | ||
|
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.
This is not the way the docs are written. Please check how other docs handle removal, also remember to break long lines into separate ones.