Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Bugfix FXIOS-5112 [v108] Fix Learn More button in ETP settings showing up in wrong places #12227

Merged
merged 1 commit into from
Oct 26, 2022

Conversation

nayushi
Copy link
Contributor

@nayushi nayushi commented Oct 26, 2022

Issue #12179

This bug was happening because when you tapped Standard option it would lose the label constraint reference for the button and it would still shows up. The only way that I thought that would be a quick and effective solution would take the creation of the UIButton out of the footer function and let it be private inside of the class so I could hide and show the button when check the Standard and Strict button inside of the generateSettings function.
A few other things that I tried was doing some rules to just create the button based on the currentBlockingStrength and/or section but would still happen.

@lmarceau lmarceau self-requested a review October 26, 2022 13:08
@lmarceau lmarceau changed the title Bugfix FXIOS-12179 v108 Fix Learn More button in ETP settings showing up in wrong places Bugfix FXIOS-5112 [v108] Fix Learn More button in ETP settings showing up in wrong places Oct 26, 2022
Copy link
Contributor

@lmarceau lmarceau left a comment

Choose a reason for hiding this comment

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

Thanks a lot @nayushi ! Great work!

@lmarceau lmarceau merged commit 4bf1945 into mozilla-mobile:main Oct 26, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants