-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Change deprecated method setButton #546
Comments
I would eagerly like to work on this issue and have already started. :) |
@UrjaPawar does it make sense now? As you know we want to refactor our dialogs #541 |
@grzesiek2010 @UrjaPawar I haven't looked at this in detail. Could doing this first be a way to break #541 more incremental? In other words, will this change be useful for #541 and reduce the scope of what then needs to be done there? I guess it depends on whether an external library gets used, right? |
@lognaturel @grzesiek2010 I did think about #541 and noticed that if we use this-> https://github.com/michael-rapp/AndroidMaterialDialog library or simply update the original one, the syntax changes will be similar! This is what will be used in case of libraries This is what alternative of deprecated setButton came out to be in one of the function of the app: Please consider this and let me know what are your thoughts! |
@UrjaPawar I see. I just thought that we can do that as a part of #541 but generally I'm in favour of small and focused prs so maybe it makes sense. Let @lognaturel decide. |
@grzesiek2010 I understand your preference and will make sure not to repeat this. I will close this issue as soon as @lognaturel confirms it |
I'm sorry I let this sit for so long! Yes, I think it's best to close for now, make a decision on dialogs more generally and then decide how we want to implement those changes. |
There are a bunch of setButton methods used in Alert Dialogs which are deprecated and need to be replaced for a smoother transition to updated version of app's UI.
All the methods used previously in alert dialogs are available in similar formats in the updated version where the builder is customized first and then is created.
These methods need to get changed
The text was updated successfully, but these errors were encountered: