Skip to content
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

Add placeholder to SearchScreen text field and focus on when launch screen #1002

Merged

Conversation

iiinaiii
Copy link

@iiinaiii iiinaiii commented Sep 8, 2024

Issue

  • close #ISSUE_NUMBER

Overview (Required)

  • Focus on Textfield when SearchScreen is launched. Keyboard for input is appear when this timing.
  • Add placeholder to SearchScreen Textfield.

Links

Screenshot (Optional if screenshot test is present or unrelated to UI)

Before After

Movie (Optional)

Before After
movie-before.mp4
movie-after.mp4

@iiinaiii
Copy link
Author

iiinaiii commented Sep 8, 2024

As for the wording of the Placeholder, it was “Search/検索” on iOS, so I made it the same as that.
I thought it would be better to use the phrase “Search sessions/セッションを検索”, what do you think ?

@github-actions github-actions bot temporarily deployed to deploygate-distribution September 8, 2024 06:26 Inactive
@DroidKaigi DroidKaigi deleted a comment from github-actions bot Sep 8, 2024
@github-actions github-actions bot temporarily deployed to deploygate-distribution September 8, 2024 07:18 Inactive
@takahirom
Copy link
Member

"Search sessions/セッションを検索" sounds great. Could you change it? 🙏

Copy link

github-actions bot commented Sep 8, 2024

Snapshot diff report

File name Image
SearchScreenTest[Sea
rchScreen - when ser
ver is operational -
when filter categor
y chip click - when
click category App A
rchitecture en - it
should selected cate
gory App Architectur
e en]_compare.png
SearchScreenTest[Sea
rchScreen - when ser
ver is operational -
when filter day chi
p click - when click
conference day 2 -
it should selected d
ay 2]_compare.png
SearchScreenTest[Sea
rchScreen - when ser
ver is operational -
when filter languag
e chip click - it sh
ould show drop down
menu]_compare.png
SearchScreenTest[Sea
rchScreen - when dev
ice is tablet - it s
hould no timetable i
tems are displayed]_
compare.png
SearchScreenTest[Sea
rchScreen - when ser
ver is operational -
when filter categor
y chip click - when
click category Other
en - it should sele
cted category Other
en]_compare.png
SearchTextFieldAppBa
rPreview_FilledSearc
hWord_compare.png
SearchScreenTest[Sea
rchScreen - when ser
ver is operational -
when filter languag
e chip click - when
click language MIXED
- it should selecte
d language MIXED]_co
mpare.png
SearchScreenTest[Sea
rchScreen - when ser
ver is operational -
when filter categor
y chip click - it sh
ould show drop down
menu]_compare.png
SearchScreenTest[Sea
rchScreen - when ser
ver is operational -
when filter languag
e chip click - when
click language ENGLI
SH - it should selec
ted language ENGLISH
]_compare.png
SearchScreenTest[Sea
rchScreen - when ser
ver is operational -
input search word t
o TextField - it sho
uld show search word
and filtered items]
_compare.png
SearchScreenTest[Sea
rchScreen - when ser
ver is operational -
when filter categor
y chip click - when
click category Jetpa
ck Compose en - it s
hould selected categ
ory Jetpack Compose
en]_compare.png
SearchScreenTest[Sea
rchScreen - when ser
ver is operational -
when filter day chi
p click - when click
conference day 1 -
it should selected d
ay 1]_compare.png
SearchScreenTest[Sea
rchScreen - when ser
ver is operational -
when filter day chi
p click - it should
show drop down menu]
_compare.png
SearchTextFieldAppBa
rPreview_EmptySearch
Word_compare.png
SearchScreenTest[Sea
rchScreen - when ser
ver is operational -
it should no timeta
ble items are displa
yed]_compare.png
SearchScreenTest[Sea
rchScreen - when dev
ice is tablet - inpu
t search word to Tex
tField - it should s
how search word and
filtered items]_comp
are.png
EmptySearchScreenPre
view_compare.png
SearchScreenTest[Sea
rchScreen - when ser
ver is operational -
when filter languag
e chip click - when
click language JAPAN
ESE - it should sele
cted language JAPANE
SE]_compare.png

@iiinaiii
Copy link
Author

iiinaiii commented Sep 8, 2024

@takahirom

"Search sessions/セッションを検索" sounds great. Could you change it? 🙏

OK!
I can also make corrections on the iOS side, but is it okay to handle it in this pull request?

@takahirom
Copy link
Member

You can do it in this PR. But separated pull request would be better because there are different reviewers.

@iiinaiii
Copy link
Author

iiinaiii commented Sep 8, 2024

Understood.
I will only modify the Android side in this pull request

@iiinaiii
Copy link
Author

iiinaiii commented Sep 8, 2024

I modified the words of placeholder.

en ja

@github-actions github-actions bot temporarily deployed to deploygate-distribution September 8, 2024 08:49 Inactive
@@ -56,7 +75,32 @@ fun SearchTextFieldAppBar(
focusManager.clearFocus()
},
),
decorationBox = { innerTextField ->
TextFieldDefaults.DecorationBox(
Copy link
Member

Choose a reason for hiding this comment

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

DecorationBox 📝

Copy link
Member

@takahirom takahirom left a comment

Choose a reason for hiding this comment

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

Looks great! Thank you for your contribution!

@takahirom takahirom merged commit 2d83ed6 into DroidKaigi:main Sep 8, 2024
7 checks passed
@iiinaiii
Copy link
Author

iiinaiii commented Sep 8, 2024

Sorry, I forgot to write the #ISSUE_NUMBER of close in the PR.
The issue number is #992 .

Please close the issue.

@iiinaiii iiinaiii deleted the feature/add-search-screen-placeholder branch September 9, 2024 00:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants