totalcross icon indicating copy to clipboard operation
totalcross copied to clipboard

Spinner isnt cropped correctly

Open Iaggoq opened this issue 4 years ago • 11 comments

Totalcross component spinner isnt cropped correctly

Describe the bug

There is a clear cut in the left and the what is missing is getting in the right part of the component.

To Reproduce

spinner

		Spinner spinner = new Spinner();
		add(spinner,400,400,600,600);
		spinner.setBackForeColors(Color.getRGB("000000"),Color.getRGB("BC26FF"));
		spinner.start();

Expected behavior

of course is to not to have that cut.

Devices:

Please complete the following information:

  • OS Version: [Windows 10]
  • TotalCross Version: [6.1.0]

Iaggoq avatar Aug 18 '20 19:08 Iaggoq

Hey @Iaggoq is this bug reproducible only in the simulator or in some other device is reproducible?

acmlira avatar Aug 24 '20 13:08 acmlira

In my android also show like that

Device:

Device: Asus ROG II OS: Android OS Version: 9 TotalCross Version: 6.1.0

Iaggoq avatar Aug 24 '20 13:08 Iaggoq

Can you try to package for windows and test on your host?

acmlira avatar Aug 24 '20 13:08 acmlira

I tested the package on windows and linux arm and still has the same problem

Iaggoq avatar Aug 24 '20 17:08 Iaggoq

Ok, I think it's a problem with the component. @ricardobna you can look at something about it?

acmlira avatar Aug 25 '20 16:08 acmlira

This looks like a bug on skia, does this behavior also happen in the simulator?

ricardobna avatar Sep 17 '20 12:09 ricardobna

The bug also happens in simulator. image

teras avatar Nov 25 '20 15:11 teras

@teras , issue #200 suggests a new implementation for this spinner component (called loader there). what do you think?

nmarquesin avatar Dec 08 '20 16:12 nmarquesin

@teras , issue #200 suggests a new implementation for this spinner component (called loader there). what do you think?

On #200 is a different kind of spinner. There the spinner has defined start and finish. This one is indefinitely. We probably need both, but at minimum we need this.

teras avatar Dec 09 '20 09:12 teras

what about having a single component with a parameter like "finish=true" that defines if it runs indefinitely or has start and finish points? would that introduce too much complexity?

nmarquesin avatar Dec 10 '20 16:12 nmarquesin

it seems there are issues with transparency and padding on the gif used for this animation. we'll work on a new gif.

nmarquesin avatar Jan 08 '21 11:01 nmarquesin