feathersui / feathersui-starling

User interface components for Starling Framework and Adobe AIR
https://feathersui.com/learn/as3-starling/
Other
914 stars 386 forks source link

Buggy Alert with TopcoatLightMobileTheme #1683

Closed CrazyFlasher closed 6 years ago

CrazyFlasher commented 6 years ago

Alert layout seems to be broken in TopcoatLightMobileTheme. Fine with MetalWorksMobileTheme Feathers 3.3.1

See screens: http://joxi.ru/nAyzdViX0555rZ - broken http://joxi.ru/Dr8Kx8CkaEE5A6 - ok

joshtynjala commented 6 years ago

Already fixed in the latest source here on GitHub. 😄

CrazyFlasher commented 6 years ago

Took 3.4.1 release and issue is still there: image

joshtynjala commented 6 years ago

I did not include the fix in 3.4.1. It will be included in 3.5.

CrazyFlasher commented 6 years ago

ok, cool

CrazyFlasher commented 6 years ago

Was it fix in theme, feathers or just in assets fla? I need to know what component to rebuild

joshtynjala commented 6 years ago

It was fixed in the theme.

CrazyFlasher commented 6 years ago

in theme code or just assets design fixes?

joshtynjala commented 6 years ago

I know for sure that I changed the assets a bit, but I don't recall if I changed the code too.

joshtynjala commented 6 years ago

Commit eb941d7ebcdeccddbdc166882a79cb2067abadf3

I made a change to the code, but it doesn't appear to be required for the fix to alert.

CrazyFlasher commented 6 years ago

is it possible to download topcoat_light_mobile.fla somewhere before release 3.5?

joshtynjala commented 6 years ago

topcoat_light_mobile.zip

CrazyFlasher commented 6 years ago

thank you very much!