Skip to content

Conversation

@annavik
Copy link
Member

@annavik annavik commented Jul 10, 2023

I realised the mobile version has gotten minimal attention last week! Here comes some quick tweaks:

  • Fix broken session detail view
  • Consider the dynamic view port size when we want "full height"
  • Adjust paddings and margins here and there (32px -> 16px)
  • Adjust typography (go down one step in size and line height)

Still a lot of improvements to made for mobile devices, for example find an alternative to our hover interactions and make table views even more compact? Also the session detail might need some alternative designs? I hope this will take care of most urgent issues though :)

@annavik annavik requested a review from mihow July 10, 2023 18:42
@netlify
Copy link

netlify bot commented Jul 10, 2023

Deploy Preview for ami-storybook ready!

Name Link
🔨 Latest commit d91f816
🔍 Latest deploy log https://app.netlify.com/sites/ami-storybook/deploys/64ac5129e31b9f000872fb6c
😎 Deploy Preview https://deploy-preview-197--ami-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Jul 10, 2023

Deploy Preview for ami-web ready!

Name Link
🔨 Latest commit d91f816
🔍 Latest deploy log https://app.netlify.com/sites/ami-web/deploys/64ac5129ec793b0007abef34
😎 Deploy Preview https://deploy-preview-197--ami-web.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mihow
Copy link
Collaborator

mihow commented Jul 10, 2023

Heck yeah! Fantastic, thank you. I didn't want to say anything but I really wanted the session detail to work on my phone.

@mihow mihow merged commit b9873f9 into main Jul 10, 2023
@annavik annavik deleted the web-ui-small-screen-tweaks branch October 11, 2023 06:21
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.

3 participants