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

In TimetableGridItem, Schedule icon align is misaligned when changing the font size on the device #900

Conversation

kosenda
Copy link
Contributor

@kosenda kosenda commented Aug 31, 2024

Issue

none

Overview

  • It is quite detailed...
  • The schedule has icons and text surrounded by a Row, but the height of the icons is fixed, so if the font size on the device is changed, the height will not match and the alignment will be shifted.

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

Title Before After
small font
medium font
large font

@github-actions github-actions bot temporarily deployed to deploygate-distribution August 31, 2024 07:43 Inactive
Copy link

Snapshot diff report

File name Image
TimetablePreview_com
pare.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
TimetableVerticalSca
le60PercentPreview_c
ompare.png
PreviewTimetableGrid
ItemWelcomeTalk_comp
are.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 datet
ime is 2024-09-12 10
-30 - it should show
an indicator of the
current time at 10-
30]_compare.png
PreviewTimetableGrid
LongTitleItem_compar
e.png
TimetableVerticalSca
le40PercentPreview_c
ompare.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 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 the current date
is 2024-09-13 - swit
ch to grid timetable
- it should show ti
metable items for Co
nferenceDay2]_compar
e.png

@github-actions github-actions bot temporarily deployed to deploygate-distribution August 31, 2024 07:59 Inactive
@kosenda kosenda marked this pull request as ready for review August 31, 2024 08:05
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 b432688 into DroidKaigi:main Sep 1, 2024
7 checks passed
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