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

Dataset Files - DataTable LiveScroll w/ Duplicate Files #2699

Closed
mheppler opened this issue Oct 27, 2015 · 8 comments
Closed

Dataset Files - DataTable LiveScroll w/ Duplicate Files #2699

mheppler opened this issue Oct 27, 2015 · 8 comments
Labels
Feature: File Upload & Handling Type: Bug a defect UX & UI: Design This issue needs input on the design of the UI and from the product owner

Comments

@mheppler
Copy link
Contributor

In a local dataset pg, I have files 026.txt - 050.txt duplicated multiple times, due to the table reloading the same 25 files it loaded previously.

To replicate it, load the dataset pg, scroll the page half way down, then hover over the files table, to scroll the files and trigger a liveScroll refresh for additional files. While the blue spinner is on the page, quickly start scrolling the table further. I was able to replicate it multiple times with this approach.

This was in Firefox (v 41.0.2). Dataset is published. I wasn't logged in.

(Pay no attention to the icons being cut off in this screenshot. That layout issue was resolved in another ticket.)

screen shot 2015-10-27 at 12 56 15 pm

@mheppler mheppler added UX & UI: Design This issue needs input on the design of the UI and from the product owner Feature: File Upload & Handling Type: Bug a defect labels Oct 27, 2015
@mheppler mheppler added this to the 4.2.1 Performance patch milestone Oct 27, 2015
@landreev
Copy link
Contributor

Hmm. This sounds like PF's scroll is just buggy (allowing to activate scrolling while it's already scrolling...)
Unless of course this is something we are doing wrong. Will investigate.

@sekmiller
Copy link
Contributor

For what it's worth, I was able to get it to happen, but when I selected the same file twice and clicked "download all" I only got it once.

@landreev
Copy link
Contributor

I'm at a loss as to what to do with this one... Doesn't seem like anything we are doing (i.e., this is the behavior of the PF component). There could still be a way to disable extra scrolling while another scrolling is in progress... I just don't know how to do that .
@sekmiller - any chance you have time to take this one for investigation?

@pdurbin
Copy link
Member

pdurbin commented Oct 28, 2015

I spoke with @sekmiller this morning and he said he's able to replicate the bug. I just tried locally (I'm on commit a593b40 ) and couldn't. I walked over to @mheppler and we couldn't replicate it on his machine either.

@kcondon
Copy link
Contributor

kcondon commented Oct 28, 2015

@mheppler @pdurbin @sekmiller @landreev

I can reproduce the bug. It creates duplicate blocks of files, not two of the same next to each other.

@kcondon
Copy link
Contributor

kcondon commented Oct 28, 2015

@mheppler @pdurbin @sekmiller @landreev

I did more testing and confirmed edit, delete, restrict, download only affect the file chosen, even if both copies are chosen.

@landreev landreev assigned scolapasta and unassigned landreev Oct 29, 2015
@landreev landreev modified the milestones: 4.3, 4.2.1 Performance patch Oct 29, 2015
@landreev
Copy link
Contributor

Putting the ticket into 4.3. Also, removing my name. (It ended up in my queue kinda randomly - I asked for it thinking it could be something I was doing with the filemetadatas; but it quickly became apparent this had to do with the idiosyncrasies of the PrimeFaces' scroller).
Let's decide who'll be working on this for 4.3.

@mheppler
Copy link
Contributor Author

A new issue, Dataset - File UI Improvements #3404, was created to track the UI improvements related to the files view on the dataset pg. Older feature requests and bug reports like this one will be closed in favor of that new issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature: File Upload & Handling Type: Bug a defect UX & UI: Design This issue needs input on the design of the UI and from the product owner
Projects
None yet
Development

No branches or pull requests

8 participants