Skip to content
This repository has been archived by the owner on Dec 14, 2021. It is now read-only.

Fixed Issue #1193: Added space below Lock Now button #1228

Closed
wants to merge 4 commits into from

Conversation

KrisRJack
Copy link
Contributor

@KrisRJack KrisRJack commented May 19, 2020

Fixes #1193

Added space below Lock Now button in settings.

Testing and Review Notes

Go to Settings page. Scroll to bottom on TableView. There should be a space of 18px below the last cell.

Screenshots or Videos

Simulator Screen Shot - iPhone 8 - 2020-05-18 at 23 37 49

@KrisRJack KrisRJack requested a review from a team as a code owner May 19, 2020 04:39
Comment on lines 197 to 198
// Font is not consistent with app. Uncomment this line out if this was intentional
// cell.textValue.font = UIFont(name: "Menlo-Regular", size: 16)

Choose a reason for hiding this comment

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

I think these lines can just be deleted 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@eliserichards For some reason I keep getting "build error" for the uispecs test in GitHub. Is that error on my side?

Copy link
Contributor

Choose a reason for hiding this comment

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

@Krisjackson no it isn't - figuring it out now. It's an fxa error with our python script that creates fake logins for the main list, I believe

@st3fan st3fan closed this Jan 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Lock now" button is too close to the screen margin
4 participants