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 Page: File Tab Sort Button #2506

Closed
eaquigley opened this issue Sep 11, 2015 · 5 comments
Closed

Dataset Page: File Tab Sort Button #2506

eaquigley opened this issue Sep 11, 2015 · 5 comments
Labels
Feature: File Upload & Handling Feature: Search/Browse Type: Feature a feature request UX & UI: Design This issue needs input on the design of the UI and from the product owner

Comments

@eaquigley
Copy link
Contributor

Sort button based off of this mock up: https://iqssharvard.mybalsamiq.com/projects/filesprovenancefacetsfilter-dataverse40/Dataset%20-%20Files%20-%20Sort

@eaquigley eaquigley added Type: Feature a feature request Feature: Search/Browse UX & UI: Design This issue needs input on the design of the UI and from the product owner labels Sep 11, 2015
@eaquigley eaquigley added this to the 4.2 milestone Sep 11, 2015
@pdurbin
Copy link
Member

pdurbin commented Sep 11, 2015

The Sort button has been available at https://beta.dataverse.org for a while (example dataset at https://beta.dataverse.org/dataset.xhtml?persistentId=doi:10.5072/FK2/KJQUMO ) but hidden behind a feature flag. @eaquigley and I decided to enable the feature (which I'll do in my next commit) so it can go through QA.

Questions I have:

  • The mockups do not specify whether "Size" should sort biggest to smallest or smallest to biggest. Should we have both?
  • The same is true of "Type" in the mockups. Should we have "Type (A-Z)" and "Type (Z-A)"?

I'm happy to get feedback on the Sort button.

@pdurbin
Copy link
Member

pdurbin commented Sep 11, 2015

In d816183 I refactored the code and enabled the Sort button. Please note that I did not get around to making the selected field to be sorted by bold like in the mockups:

dataset_-files-sort_mybalsamiq-_2015-09-11_11 38 52

I'm a bit concerned with how the Sort button will interact with editing files but since editing is happening in new pages (that I haven't even looked at) maybe this concern is unfounded.

I made it so the Sort button can be easily disabled. On the xhtml side this is #{DatasetPage.sortButtonEnabled} and of course there is an equivalent boolean on the Java side.

There's a bit more cleanup I want to do in filesFragment.xhtml before I had this ticket over to @mheppler for proper positioning of the Sort button.

@pdurbin
Copy link
Member

pdurbin commented Sep 11, 2015

@mheppler in 35c471f I believe I removed the last of the references to weird rendering booleans. You'll still see one for "sortButtonEnabled" but that's expected, per my last comment. Over to you to please position the Sort button in a sane way. Thanks!

@pdurbin
Copy link
Member

pdurbin commented Sep 11, 2015

@kcondon noticed a bug with the Sort button that the list was not being updated, especially when he was switching from Newest to Oldest but locally, I found that if a do a lot of sorting (at least five sorts) eventually the Sort button doesn't respond to my mouse clicks.

I don't have time to fix this for 4.2 unfortunately and @eaquigley said she's fine with deferring this feature (so I disabled the Sort button in 5367b8c) and giving this to @sekmiller for 4.3 as part of the "finishing up file features" project.

@pdurbin pdurbin modified the milestones: 4.3, 4.2 Sep 11, 2015
@pdurbin pdurbin assigned sekmiller and unassigned mheppler Sep 11, 2015
@mercecrosas mercecrosas modified the milestones: 4.3, In Review Nov 30, 2015
@sekmiller sekmiller removed their assignment Jan 28, 2016
@scolapasta scolapasta removed this from the Not Assigned to a Release milestone Jan 28, 2016
@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 Feature: Search/Browse Type: Feature a feature request 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

6 participants