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

Files Tab: File count does not indicate how many files are currently displayed from total. #2681

Closed
kcondon opened this issue Oct 23, 2015 · 10 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

@kcondon
Copy link
Contributor

kcondon commented Oct 23, 2015

Infinite scroller maybe should be showing how many files are currently being displayed out of total. I have 1023 files, I am on the first or second page, how many are currently displayed?

@kcondon kcondon added this to the 4.2.1 Performance patch milestone Oct 23, 2015
@eaquigley
Copy link
Contributor

Additionally, there needs to be a way to see how many files have been selected using the select all checkbox. Currently the only way to find out is to wait till you have selected files and then clicked Edit Metadata.

@sekmiller
Copy link
Contributor

As we discussed in the meeting we're not going to deal with the number selected for now, but if there's some relatively easy way to get the number of files currently in the scroller to display that number to the user, that should be enough for now

@sekmiller sekmiller assigned pdurbin and unassigned sekmiller Oct 26, 2015
@eaquigley
Copy link
Contributor

why aren't we dealing with this now? with the scroller, i think it is necessary to show the number selected because you can't see all of the files anymore

On Oct 26, 2015, at 11:48 AM, Stephen Kraffmiller <notifications@git.luolix.topmailto:notifications@github.com> wrote:

As we discussed in the meeting we're not going to deal with the number selected for now, but if there's some relatively easy way to get the number of files currently in the scroller to display that number to the user, that should be enough for now


Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_IQSS_dataverse_issues_2681-23issuecomment-2D151245720&d=CwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=Y9aY3P6kFFpmMLaYYO_id08dS3gL1xWMQuI2CZ74PoI&m=wYq843_t08xsRvtXNnSPokFv8NIkyc13b35Tghcjysw&s=xtqpE7bgLiUG3CRq7i0O6p_T97DZL7n3NP-cVrOgx3Q&e=.

@sekmiller
Copy link
Contributor

We aren't dealing this now because there is no way to capture the select and deselect file on the click event without using an ajax event which we removed because it was taking too long to refresh the page. it's the same reason all of the buttons are active and only checked for validity when you click on it.

@pdurbin
Copy link
Member

pdurbin commented Oct 26, 2015

if there's some relatively easy way to get the number of files currently in the scroller to display that number to the user

I looked through the PrimeFaces 4.0 documentation linked from http://www.primefaces.org/documentation and I'm not saying there isn't a way, but I couldn't find it.

The area of the code in question is in filesFragment.xhtml (near the top) and the size indicates the total number of files in the dataset version.

<div id="filesHeaderCount" class="col-xs-6 text-left">
    <!-- Files Count -->
    <h:outputFormat value="#{bundle['file.count']}" styleClass="highlightBold">
        <f:param value="#{DatasetPage.fileMetadatasSearch.size()}"/>
    </h:outputFormat>
</div>

So it's showing "33 files"...

screen shot 2015-10-26 at 3 47 44 pm

... when what we want it to show is "25 of 33 files", similar to these mockups:

screen shot 2015-10-26 at 3 49 40 pm

@sekmiller I'm passing this back to you since I couldn't find an answer quickly. Sorry. I did write a little script at scripts/issues/2681/create-files to create a zip file with an arbitrary number of files. Call it with scripts/issues/2681/create-files 33 to create /tmp/33files.zip for example.

@pdurbin pdurbin assigned sekmiller and unassigned pdurbin Oct 26, 2015
@pdurbin
Copy link
Member

pdurbin commented Oct 28, 2015

Yesterday @raprasad suggested using jQuery to count the number of visible rows. This is if we can't find a way for PrimeFaces to report the number of currently displayed files.

@sekmiller
Copy link
Contributor

The problem here is that there's no event to trigger the update of the counts. PrimeFaces has a new feature ticket to do so:

primefaces/primefaces#65

@sekmiller
Copy link
Contributor

@mheppler Thanks for offering to take a shot at this. Not sure how to do it without some kind of triggering event. See my notes above and the prime faces issue.

@scolapasta scolapasta modified the milestones: 4.2.1 Performance patch, 4.3 Nov 2, 2015
@mercecrosas mercecrosas modified the milestones: 4.3, In Review Nov 30, 2015
@mheppler mheppler removed their assignment Jan 11, 2016
@scolapasta scolapasta modified the milestone: Not Assigned to a Release Jan 28, 2016
@mheppler mheppler added the UX & UI: Design This issue needs input on the design of the UI and from the product owner label Feb 1, 2016
@eaquigley
Copy link
Contributor

Related to #2716 as there needs to be a counter in that workflow as well. We need to figure out a solution.

@mheppler
Copy link
Contributor

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

7 participants