brave / brave-ios

Brave iOS Browser
https://brave.com
Mozilla Public License 2.0
1.7k stars 441 forks source link

Buttons loading states / functionality is broken (IUButtonConfiguration) #8555

Open soner-yuksel opened 9 months ago

soner-yuksel commented 9 months ago

Description:

UIButton deprecated various properties and this is causing warning all over the application and in addition this situation is causing some UI discrepancies on some button functionality on the browser.

The major problem is button that has loading state. The addsubview still working on UIButton however dynamically changing subview properties is totally not functioning.

So any custom button that has loading indicator added as subview and loading state is changed accordingly is broken. No loading indicator is shown or functioning. This effects mainly BraveButton class and its functionality and subclasses like ActionButton.

An example to this can be given as when Open search engine is added the open search button OpenSearchEngineButton is not showing its loading state.

Actual result:

Loading states are broken and some other functionality should be thoroughly checked in the app where deprecated button properties and other things are used. Also all the warning related with UIButton generation should be wiped and cleaned.

Expected result:

All functionality should be working properly and and buttons should be converted to using UIButton configuration.

Reproduces how often: [Easily reproduced, Intermittent Issue]

Easily reproduced

Brave Version:

Device details:

Any

Website problems only:

Additional Information