Skip to content

Conversation

@AkkeyLab
Copy link
Member

@AkkeyLab AkkeyLab commented Dec 20, 2025

Before After After(Phone)
Screenshot 2025-12-20 at 16 41 34 Screenshot 2025-12-20 at 16 40 59 Simulator Screenshot - iPhone 17 Pro - 2025-12-20 at 16 49 13
Screenshot 2025-12-20 at 16 38 21 Screenshot 2025-12-20 at 16 37 41 Simulator Screenshot - iPhone 17 Pro - 2025-12-20 at 16 49 01

@AkkeyLab AkkeyLab self-assigned this Dec 20, 2025
@AkkeyLab AkkeyLab requested a review from psbss December 20, 2025 07:44
"stringUnit" : {
"state" : "translated",
"value" : "開会の挨拶"
"value" : "Opening"
Copy link
Contributor

Choose a reason for hiding this comment

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

[nits]
I know "Closing" is being translated as "クロージング" so how about it might be a good idea to translate "Opening" as "オープニング" as well.

VStack(alignment: .leading, spacing: 8) {
ForEach(speakers) { speaker in
Image(speaker.imageFilename, description: speaker.name)
Table {
Copy link
Contributor

Choose a reason for hiding this comment

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

[discuss]
I think this is better than the previous one, but how about displaying a "sticky header" + "tab view" when the user is using a smartphone? 👀

Copy link
Contributor

Choose a reason for hiding this comment

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

I think the quality of this PR is already good enough to merge. 🙆‍♂️

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