Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion feature/search/api/src/main/res/values/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
<string name="feature_search_api_clear_search_text_content_desc">Clear search text</string>
<string name="feature_search_api_result_not_found">Sorry, there is no content found for your search \"%1$s\"</string>
<string name="feature_search_api_not_ready">Sorry, we are still processing the search index. Please come back later</string>
<string name="feature_search_api_try_another_search">Try another search or explorer </string>
<string name="feature_search_api_try_another_search">Try another search or explore </string>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The string resource feature_search_api_try_another_search contains a trailing space. However, in SearchScreen.kt (line 233), an explicit space is already appended:

append(stringResource(id = searchR.string.feature_search_api_try_another_search))
append(" ")

This results in a double space in the rendered UI (i.e., Try another search or explore Interests). Removing the trailing space from the string resource resolves this issue and keeps the resource clean.

Suggested change
<string name="feature_search_api_try_another_search">Try another search or explore </string>
<string name="feature_search_api_try_another_search">Try another search or explore</string>

<string name="feature_search_api_interests">Interests</string>
<string name="feature_search_api_to_browse_topics"> to browse topics</string>
<string name="feature_search_api_topics">Topics</string>
Expand Down