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

Update the touch targets for the header icons on TimetableScreen #821

Merged
merged 1 commit into from
Aug 27, 2024

Conversation

ritsukiotsuka
Copy link
Contributor

Issue

Overview (Required)

  • Update the touch targets for the header icons on TimetableScreen to 48x48dp, as specified in the Figma design
  • Update the ripple effect for the icons to be circular in shape

Links

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

Movie (Optional)

Before After
2024-08-26.1.25.40.mov
Screen.Recording.2024-08-27.at.6.33.07.AM.mov

@github-actions github-actions bot temporarily deployed to deploygate-distribution August 26, 2024 23:15 Inactive
Copy link

Snapshot diff report

File name Image
TimetableScreenTest[
TimetableScreen - wh
en the current date
is 2024-09-13 - it s
hould show timetable
items for Conferenc
eDay2]_compare.png
TimetableScreenTest[
TimetableScreen - wh
en the current date
is 2024-09-12 - swit
ch to grid timetable
- it should show ti
metable items for Co
nferenceDay1]_compar
e.png
PreviewTimetableScre
enDark_compare.png
TimetableScreenTest[
TimetableScreen - wh
en server is operati
onal - click timetab
le ui type change -
it should change tim
etable ui type]_comp
are.png
TimetableScreenTest[
TimetableScreen - wh
en the current date
is 2024-09-12 - it s
hould show timetable
items for Conferenc
eDay1]_compare.png
TimetableScreenTest[
TimetableScreen - wh
en the current datet
ime is 2024-09-12 10
-30 - it should show
an indicator of the
current time at 10-
30]_compare.png
TimetableScreenTest[
TimetableScreen - wh
en server is operati
onal - it should sho
w timetable items]_c
ompare.png
TimetableScreenTest[
TimetableScreen - wh
en device is tablet
- it should show tim
etable items]_compar
e.png
TimetableScreenTest[
TimetableScreen - wh
en server is down -
it should show error
message]_compare.pn
g
TimetableScreenTest[
TimetableScreen - wh
en server is operati
onal - click confere
nce day2 tab - it sh
ould change displaye
d day]_compare.png
TimetableScreenTest[
TimetableScreen - wh
en the current date
is 2024-09-11 - it s
hould show timetable
items for Conferenc
eDay1]_compare.png
TimetableScreenTest[
TimetableScreen - wh
en server is operati
onal - scroll timeta
ble - it should firs
t session is not dis
played]_compare.png
TimetableScreenTest[
TimetableScreen - wh
en server is operati
onal - click timetab
le ui type change -
click conference day
2 tab - it should ch
ange displayed day]_
compare.png
TimetableScreenTest[
TimetableScreen - wh
en the current datet
ime is 2024-09-11 10
-00 - it should not
show an indicator of
the current time]_c
ompare.png
TimetableScreenTest[
TimetableScreen - wh
en the current date
is 2024-09-11 - swit
ch to grid timetable
- it should show ti
metable items for Co
nferenceDay1]_compar
e.png
TimetableScreenTest[
TimetableScreen - wh
en server is operati
onal - click timetab
le ui type change -
scroll timetable - i
t should first sessi
on is not displayed]
_compare.png
TimetableScreenTest[
TimetableScreen - wh
en the current datet
ime is 2024-09-13 11
-00 - it should show
an indicator of the
current time at 11-
00]_compare.png
TimetableScreenTest[
TimetableScreen - wh
en server is operati
onal - click first s
ession bookmark - it
should show bookmar
ked session]_compare
.png
TimetableScreenTest[
TimetableScreen - wh
en the current date
is 2024-09-13 - swit
ch to grid timetable
- it should show ti
metable items for Co
nferenceDay2]_compar
e.png
KaigiAppTest[KaigiAp
p - when app is star
ting - it should sho
w timetable items]_c
ompare.png

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 fixing this!

@takahirom takahirom merged commit e85458a into DroidKaigi:main Aug 27, 2024
6 checks passed
@ritsukiotsuka ritsukiotsuka deleted the ritsukiotsuka/issue_804 branch August 27, 2024 13:42
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.

In TimetableScreen, buttons placement is different from Figma and tap area is narrower
2 participants