-
Notifications
You must be signed in to change notification settings - Fork 39
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
[Material] Slider.addOnChangeListener binding missing #230
Comments
Binding issue:
|
Reproduced for RangeSlider too. Or maybe I should use other API ? |
Although have the same issue |
Any updates? |
workaround:
|
Please try 1.3.0 and provide feedback here. If it is not fixed automagically I will dive in next week |
@moljac thanks for the new bindings - great stuff! 🥳 |
Thanks. OK. I will need to dig in. Hopefully soon |
Friendly ping :) |
Sadly, this issue is still present with 1.4.0 |
This is a workaround until they fix binding issue:
|
1.4.0.5 still not work |
Moving my comment from #450 here: Ugh, this one is a mess. According to the public documentation public class Slider extends View {
public void addOnChangeListener (L listener) { ... }
} One may be wondering what public class Slider extends BaseSlider<Slider, Slider.OnChangeListener, Slider.OnSliderTouchListener>
{
...
}
abstract class BaseSlider<S extends BaseSlider<S, L, T>, L extends BaseOnChangeListener<S>, T extends BaseOnSliderTouchListener<S>> extends View
{
public void addOnChangeListener(@NonNull L listener) { ... }
} That is, the However, due to the lack of generics support in our tooling, it is unable to copy the
I would suggest continuing to use the workaround listed above. |
Is there no progress with this. Right now it's not very convenient to use these controls |
Still same with "1.8.0" version. Any progress ? |
I'm also affected by this. I understand that updating the tooling to support this edge case would be hard, but is it not possible to add the required code manually in the bindings? I believe something like this is done, for instance, to add method overloads that accept .NET classes like System.Text.StringBuilder to improve interoperability between .NET and Java. |
Just me being nosy here, what's the tooling issue? Is it to do with all generic methods like this? |
Version Information
Describe your Issue:
Slider.IOnChangeListener
is bound, but there is no way to add it to the Slider class, theaddOnChangeListener
method hasn't been boundSteps to Reproduce (with link to sample solution if possible):
The text was updated successfully, but these errors were encountered: