Skip to content
This repository was archived by the owner on Mar 13, 2025. It is now read-only.

Fix issue #621, remove extra scroll bar on 'Add Group Modal'. #623

Closed
wants to merge 1 commit into from
Closed

Fix issue #621, remove extra scroll bar on 'Add Group Modal'. #623

wants to merge 1 commit into from

Conversation

narekcat
Copy link
Contributor

No description provided.

@narekcat narekcat mentioned this pull request Jul 30, 2020
Closed
callmekatootie
callmekatootie suggested changes Jul 30, 2020
View reviewed changes
Copy link
Collaborator

@callmekatootie callmekatootie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In your case, virtual lists are not being shown - the DOM has all the group elements instead of just a handful of them...

@callmekatootie
Copy link
Collaborator

Expected:

image

Actual:

image

You can see that all the DOM nodes are being shown....

@callmekatootie callmekatootie deleted the issue-621 branch July 31, 2020 10:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@callmekatootie callmekatootie callmekatootie requested changes

Assignees
No one assigned
Labels
tcx_Feedback
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@narekcat @callmekatootie