Skip to content
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

CSS "direction" property seems to have no effect on GridLayout #13

Open
steinerj opened this issue Jul 3, 2020 · 5 comments
Open

CSS "direction" property seems to have no effect on GridLayout #13

steinerj opened this issue Jul 3, 2020 · 5 comments
Labels
bug Something isn't working help wanted Extra attention is needed Platform: Android Platform: iOS

Comments

@steinerj
Copy link

steinerj commented Jul 3, 2020

Hey @xlmnxp - great plugin!

The "direction" CSS property does not seem to have any effect on GridLayout.

The displayed content for the sample below is always: "RIGHT LEFT". It does not make a difference if the style is inline or in app.css.

The isRtl property works fine and swaps the output, if set to false.

Can you reproduce or am I missing something? Thank you for any help.

Sample xml

<Page xmlns="http://schemas.nativescript.org/tns.xsd" xmlns:rtl="@nativescript-rtl/ui" navigatingTo="navigatingTo">
    <ActionBar title="My App" icon=""></ActionBar>
    <rtl:GridLayout columns="*, *" class="testclass" style="direction:ltr;">
        <Label col="0" text="LEFT" class="h1"/>
        <Label col="1" text="RIGHT" class="h1"/>
    </rtl:GridLayout>
</Page>

app.css

.testclass {
    direction: ltr;
}

package.json

"dependencies": {
    "@nativescript/core": "~6.5.0",
    "@nativescript/theme": "~2.3.0",
    "tns-core-modules": "6.5.8",
    "@nativescript-rtl/ui": "~0.1.6"
  },

EDIT: Test platform is Android 29 Emulator.

@xlmnxp
Copy link
Member

xlmnxp commented Jul 3, 2020

Is it show any invalid message on console?


Edit: I will fix it

@xlmnxp xlmnxp added bug Something isn't working Platform: Android Platform: iOS and removed bug Something isn't working labels Jul 3, 2020
@steinerj
Copy link
Author

steinerj commented Jul 3, 2020

Sorry, should've mentioned in original post. There is no console output and the valueConverter in the directionProperty is never called. This might well be an upstream issue.

@xlmnxp
Copy link
Member

xlmnxp commented Jul 4, 2020

@steinerj It works on older versions of NativeScript
I don't know what they changed in NativeScript, which doesn't make it work!

This might well be an upstream issue

I think so, I will work to fix it

@steinerj
Copy link
Author

steinerj commented Jul 4, 2020

@xlmnxp Thank you! Do you have any rough idea which commit/core modules version it was working with last?

@xlmnxp
Copy link
Member

xlmnxp commented Jul 4, 2020

@xlmnxp Thank you! Do you have any rough idea which commit/core modules version it was working with last?

3.2.0

@xlmnxp xlmnxp added the help wanted Extra attention is needed label Jun 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed Platform: Android Platform: iOS
Projects
None yet
Development

No branches or pull requests

2 participants