🐞 Remove justifyContent: 'stretch' from ActionButton

[zachbharris]

Issue

justifyContent: 'stretch' is not supported in react-native and is giving a console error (no crashes). I believe these errors only show while in development as well, no production builds should show these errors.

Here are the available values for justifyContent in react-native: https://reactnative.dev/docs/flexbox#justify-content

Solution

• remove justifyContent: 'stretch'

Additional Notes

Have been seeing this console error in non-adalo org apps while working on a feature.

[nucks]

It won't let me see the full file path from my phone, but aren't they used on web? And that's why the uf statements remove them from mobile devices?

[zachbharris]

It won't let me see the full file path from my phone, but aren't they used on web? And that's why the uf statements remove them from mobile devices?

The property is applied to web, but I don't see it giving any additional functionality. justifiedContent: 'stretch' isn't a supported react-native style according to their docs as well.

[nucks]

It won't let me see the full file path from my phone, but aren't they used on web? And that's why the uf statements remove them from mobile devices?

The property is applied to web, but I don't see it giving any additional functionality. justifiedContent: 'stretch' isn't a supported react-native style according to their docs as well.

I'd be really surprised if it's never been used at all. I'd almost rather keep it in the codebase than remove it without complete certainty that it's never been used and isn't being used.

[scott-massey]

Jacob used it to fix an issue with the ActionButton, I forget what though. The code below was meant to remove the key/value pair on mobile though, so the solution should probably be just to fix that code instead.

[scott-massey]

And sorry for the huge delay I just noticed this was waiting on me in the github integration on slack...

[scott-massey]

should we try to merge this at some point?