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

Inconsistent title placement on pump configuration screens #974

Closed
itsmojo opened this issue May 27, 2019 · 5 comments
Closed

Inconsistent title placement on pump configuration screens #974

itsmojo opened this issue May 27, 2019 · 5 comments
Labels

Comments

@itsmojo
Copy link
Contributor

itsmojo commented May 27, 2019

Describe the bug
While trying to track down why sometimes there is a partial blue button at the bottom of the Pod Pairing screen when setting up an Omnipod on a 5s or an SE and other times there is nothing of the blue button showing, I think I was able to identify where the problem starts. Any time a user does an Add Pump and has to go thru the RileyLink Setup screen, the titles that are normally on the top middle will end up being in a large bold test on the 2nd line for this screen and all the subsequent screens until the pump is fully configured. This causes everything on the various pump setup pages to scroll down. When configuring an Omnipod on a 5s/SE, what remains of the blue Pair button on the Pod Pairing screen on the bottom scrolls completely off the screen when the title is misplaced. After the pump is fully configured, the screen titles continue to stay in the top middle for all subsequent Pod configurations until the pump is deleted and an Add Pump with a Rileylink Setup screen is run again.

To Reproduce

  1. Build Loop V.10.0DEV for the omnipod-testing branch for the iPhone SE Simulator.
  2. Depending on the initial state, go thru a sequence to force you to select the Omnipod from the Add Pump menu. This can be done by deleting any current pump or starting with a fresh Loop install.

Add Pump

  1. Click on Add Pump and select Omnipod and you will be taken to the RileyLink Setup screen which is the first screen where the title is displayed at the start of the second line in a large bold font. After this, the rest of the pump setup screens will have the screen title at the beginning of the second line in a large bold font instead of in the top middle.

RileyLink Setup on 2nd line

  1. Configure some dummy Pod Settings for Delivery Limits and Basal Rates (if necessary) and then hit Continue. Notice that the "Pod Settings" title is not displayed in the top middle as it should.

Pod Settings on 2nd line to review settings

  1. On the next Pod Pairing screen note that the "Pod Pairing" title is displayed on the beginning of the second line and not in the top middle as it normally is. Note that on a SE/5s, the Pair button at the bottom gets pushed off the screen and cannot be seen without scrolling.

Pod Pairing on 2nd line

  1. Scroll up and hit Pair and continue thru the rest of the simulated Pod setup sequence, scrolling up as needed to click on the bottom button to proceed to the next step. Note that the screen titles will continue to appear at the beginning of the 2nd line instead of the top middle for this Pod setup.

Insert Cannula on 2nd line
Setup Complete on 2nd line
Pod Settings on 2nd line upon completion

  1. Click Done to get back to the Settings screen (which now has its title centered and seems to clear this display problem for the pump configuration menus) and then click on the Omnipod to get the Pod Settings screen that is now properly displayed with the title in the upper middle as it should.

Settings in top middle
Pod Settings in top middle

Expected behavior
The screen titles should be staying in the top middle and not on the 2nd line. You can see all the Pod setup screens with the title in the top middle by simply ending the current Pod session and then setting up a new Pod by continuing on to the Pod Pairing screen as shown in the following screenshots as the RileyLink Setup screen will not be used again until the pump is deleted.

Pod Pairing on top middle
Insert Cannula on top middle
Setup Complete on top middle

If you want to see the Pod setup screens with the titles incorrectly placed on the 2nd line, you will need to do Switch from Omnipod Pumps -> Delete Omnipod -> Add Pump to get to the RileyLink Setup screen which seems to be the first one in the sequence with the misplaced title that then affects all the subsequent Pod setup screens. Note that the same incorrect title placement can be seen when setting up a Medtronic pump if the RileyLink Setup screen is visited in the sequence (i.e., switching between Pod types for each simulated run should continue to show the problem each time a Add Pump operation is done. An Omnipod pump with a SE was used for demo purposes as the effect of the incorrect title placement is most noticeable when it completely pushes the Pair button off the screen.

Phone

  • Hardware: All iPhone tested, but an SE/5s (Simulator or real) show the most dramatic effects
  • OS Version: Reproduced on the Simulator (12.2), SE (12.3.1), and 5s (12.2 and 12.3.1)

Loop Version

  • Version Number: V.10.0DEV
  • Repo: both LoopKit/Loop (omnipod-testing branch) & JoJo

CGM

  • Device: never configured
  • Manager app: never configured

Pump

  • Manufacturer: doesn't matter, but Omnipod used for demonstration purposes
  • Model: NA

Additional context
This issue was the reason why there was no Pair button seen as reported in https://loop.zulipchat.com/#narrow/stream/144120-omnipod_dev/topic/iPhone.20SE.20Pod.20Pairing.20screen/near/166330440 as the new user was switching from Medtronic to Omnipod on his iPhone SE and had to go thru the RileyLink Setup screen when setting up the new Pump.

Issue #973 could potentially be an example of this problem or perhaps related the more generic issue that the table view rows for errors and progress aren’t set up for auto sizing based on content, and doing so creates problems with the method those views use to add padding to keep the button at the bottom.

@ps2
Copy link
Collaborator

ps2 commented May 27, 2019

The expected visual design is actually the one with the bolder text. That is the design implemented for setup screens in Loop, but hasn't been applied to pod replacement flow, it looks like.

@itsmojo
Copy link
Contributor Author

itsmojo commented May 27, 2019

Adjusted the issue title

@itsmojo itsmojo changed the title Misplaced title on pump configuration screens when adding a pump Inconsistent title placement on pump configuration screens May 28, 2019
@itsmojo
Copy link
Contributor Author

itsmojo commented May 28, 2019

Adjusted issue title to match reality

Copy link

github-actions bot commented Nov 4, 2023

This issue is stale because it has been open for 30 days with no activity.

@github-actions github-actions bot added the stale label Nov 4, 2023
Copy link

This issue was closed because it has been inactive for 14 days since being marked as stale.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants