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

ScrollArea not working properly (MacOS Big Sur 11.2.2) #342

Open
dshomoye opened this issue Mar 7, 2021 · 4 comments
Open

ScrollArea not working properly (MacOS Big Sur 11.2.2) #342

dshomoye opened this issue Mar 7, 2021 · 4 comments

Comments

@dshomoye
Copy link

dshomoye commented Mar 7, 2021

Describe the bug

ScrollArea does not seem to work with nested view (no scrollbars).

It's a little confusing because rendering a single Text component inside the scroll area works.

Like so:

  <Window
    windowIcon={winIcon}
    windowTitle="Look Back"
    styleSheet={styleSheet}
    minSize={minSize}
  >
    <ScrollArea>
      <Text>
      {`Lorem ipsum dolor sit amet, consectetur adipiscing elit, 
      sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. 
      ...
      Duis aute irure dolor in reprehenderit in voluptate velit 
      ...
      esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia 
      deserunt mollit anim id est laborum
      `}
      </Text>
    </ScrollArea>
  </Window>

Which yields:

image

But nesting the Text in a View, inside a ScrollArea (which is required for multiple children) doesn't work:

  <Window
    windowIcon={winIcon}
    windowTitle="Look Back"
    styleSheet={styleSheet}
    minSize={minSize}
  >
    <ScrollArea>
      <View style={` height: '100%'; width: '100%'; `}>
        <Text>
        {`Lorem ipsum dolor sit amet, consectetur adipiscing elit, 
        ...
        nisi ut aliquip ex ea commodo consequat. 
        Duis aute irure dolor in reprehenderit in voluptate velit 
        ...
        esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia 
        deserunt mollit anim id est laborum
        `}
        </Text>
      </View>
    </ScrollArea>
  </Window>

Which yields (text clipped, no scroll bar):

image

Expected behavior
Nesting View should behave like any other component, especially since a container is required for nesting children in a ScrollArea.

Desktop (please complete the following information):

@KRTirtho
Copy link
Contributor

KRTirtho commented Mar 8, 2021

Just upgrade to the latest versions of nodegui & react-nodegui
See issue #341 for better understanding, I was having this issue to

@dshomoye
Copy link
Author

dshomoye commented Mar 8, 2021

I updated to latest but I'm still seeing the issue. It doesn't seem to be marked as fixed either, just going by this.

@a7ul
Copy link
Collaborator

a7ul commented Mar 8, 2021

@dshomoye Have you updated the react nodegui and nodegui versions ?

I tried your example and it looks like its working as expected

image

That issue is closed and fixed.

@dshomoye
Copy link
Author

dshomoye commented Mar 9, 2021

You're right, I'm not sure what happened when I had initially tested, but it seems this is not the problem I have.
Does ScrollArea adapt to View size changes after first render?

In my case, I'm getting an array of objects and rendering them into a View after rendering (in a useEffect). If I use static values the scroll area actually works (from the latest update).

I have a sample code here: https://github.com/dshomoye/react-nodegui-starter/blob/ff14af28d9b207e9ba4c377ed23b2f3be4491dac/src/components/NotesContainer.tsx#L7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants