site-kit-wp icon indicating copy to clipboard operation
site-kit-wp copied to clipboard

Android - Galaxy 22 and 22 ultra UI issues

Open eclarke1 opened this issue 3 years ago • 3 comments

Bug Description

Bug bash issue: https://app.asana.com/0/1202258919887896/1202424752772575 please see Asana for background

We should hide the "Site Kit" part of the logo in screens that small.

Steps to reproduce

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Screenshots

image.png image.png image.png

Additional Context

  • PHP Version:
  • OS: [e.g. iOS]
  • Browser: [e.g. chrome, safari]
  • Plugin Version: [e.g. 22]
  • Device: [e.g. iPhone6]

Do not alter or remove anything below. The following sections will be managed by moderators only.

Acceptance criteria

Implementation Brief

Test Coverage

QA Brief

Changelog entry

eclarke1 avatar Jun 27 '22 13:06 eclarke1

@jimmymadon I think most of the desired outcome is probably obvious here where things are overflowing. We should consider reducing the logo to just the G in portrait-phone viewports though. Last I recall checking this I wasn't able to reproduce, but I also didn't see it with a Galaxy device 😄

aaemnnosttv avatar Oct 11 '22 23:10 aaemnnosttv

@aaemnnosttv I checked this on my personal Galaxy S21 Ultra too (which has the same screen width as the S22 Ultra). Everything looks good as per the below screenshots. The logo does overflow on smaller screens when testing on the browser.

Screenshots

Screenshot_20221019-121601_Chrome Screenshot_20221019-122321_Chrome Screenshot_20221019-122332_Chrome Screenshot_20221019-122402_Chrome

@mohitwp I can reproduce the Logo overflows on smaller screen widths (< 375px). However, I cannot reproduce the overflows in the second screenshot anymore - I believe the Dashboard Sharing popup overflows may have been resolved in another similar issue. Can you please check again?

jimmymadon avatar Oct 19 '22 11:10 jimmymadon

@aaemnnosttv We don't have a breakpoint smaller than 450px ($width-xsmall). So should we change the logo for all screens below this width? The affected screen widths are approx. < 375px.

jimmymadon avatar Oct 19 '22 12:10 jimmymadon

@aaemnnosttv just following up here so we can potentially get this one moving. Also ccing @eugene-manuilov if you can help?

FlicHollis avatar Nov 09 '22 10:11 FlicHollis

@jimmymadon is this the only issue remaining here?

Screenshot image

If anything, I wonder if we should hide the "Site Kit" part of the logo in mobile. Thoughts @marrrmarrr ?

aaemnnosttv avatar Nov 11 '22 23:11 aaemnnosttv

@jimmymadon

Actually there are different versions of Galaxy device available on BrowserStack. I checked in Galaxy S21 device which viewports is 360X649. Issue not appearing in Galaxy S22+ which have width >375. So, yes issue coming only in devices <375. Regarding 2nd issue related to Dashboard sharing modal - I'm also not able to reproduce it now. Thank you !

image

image

image

mohitwp avatar Nov 14 '22 08:11 mohitwp

Recapping discussion about this one earlier it seems the only thing left here is the "Site Kit" portion of the logo getting cut off in viewports with a width < 375 px. For our smallest breakpoint we should hide this part of the logo.

Anything else to mention here @jimmymadon ?

aaemnnosttv avatar Nov 14 '22 23:11 aaemnnosttv

ACs here are good 👍🏻

tofumatt avatar Dec 06 '22 20:12 tofumatt

IB ✅

aaemnnosttv avatar Dec 08 '22 01:12 aaemnnosttv

QA Update: ✅

Verified:

  • Opened SK on mobile with a width <449px and the text part of a logo does not appear.
  • Tested on the devices in the ticket and a few other Android and iPhone devices
  • Tested on devices with different browsers, i.e. chrrome, firefox, and safari.

image

wpdarren avatar Dec 22 '22 11:12 wpdarren