wordpress-mobile / WordPress-iOS

WordPress for iOS - Official repository
http://ios.wordpress.org/
GNU General Public License v2.0
3.69k stars 1.12k forks source link

Me view: Table expands to full width when the app is restarted from the Settings app #19561

Open twstokes opened 1 year ago

twstokes commented 1 year ago

Expected behavior

The Me view to remain visually unchanged.

Actual behavior

The rows on the Me view expand to full width.

https://user-images.githubusercontent.com/2092798/200063686-11cb367b-42e7-4b14-80b9-a697b90f4040.MOV

Steps to reproduce the behavior

  1. In the Jetpack or WordPress app, go to My Site -> Me
  2. Go to the Settings app and toggle an app setting such as the Camera permission
  3. Return to the Jetpack or WordPress app
  4. Observe the table layout
Tested on iPhone 12 Pro, iOS 16.1, WPiOS 21.0
momo-ozawa commented 1 year ago

Hey @twstokes! 👋 I can't reproduce this issue on WPiOS 21.6 - lmk if I can close this issue. Thanks!

twstokes commented 1 year ago

Hey @momo-ozawa! Thanks for checking in on this. It appears to me like the behavior of the app has changed and I'm unsure if it was intentional.

Before

  1. Changing a setting for WPiOS / JPiOS in the Settings App causes a restart of the app (this is expected iOS functionality)
  2. Returning to WPiOS / JPiOS should resume from the previous screen (e.g. if the user was on My Site -> Me, they should still be on this view when the app restarts)

Version: JP 21.1

After

  1. Changing a setting for WPiOS / JPiOS in the Settings App causes a restart of the app (this is expected iOS functionality)
  2. Returning to WPiOS / JPiOS shows the My Site view

trunk 83bfa59fdbccf27f65529ec3f5b4ba655df70161

Does this seem like a regression to you? I would expect the app to (for the most part) always try to resume from the screen the user was on.

twstokes commented 1 year ago

I created an issue for this: https://github.com/wordpress-mobile/WordPress-iOS/issues/20082

momo-ozawa commented 1 year ago

Thank you @twstokes!