arXiv / arxiv-submission-ui

User interface of NG submit system.
MIT License
2 stars 6 forks source link

Fixing ordering of files on upload step. #138

Closed bdc34 closed 5 years ago

bdc34 commented 5 years ago

Fixes ordering of files and puts root files at top of list.

Moved group_files to upload controller.

Fixes #83

jimentwood commented 5 years ago

I see the output on issue #83 looks good.

mhl10 commented 5 years ago

Looks good from screenshot in #83. I'll give it a quick try in my environment.

bdc34 commented 5 years ago

Looks good from screenshot in #83. I'll give it a quick try in my environment.

That would be good. I cannot actually upload files and I faked this file list.

mhl10 commented 5 years ago

The screenshot below shows all the files starting with capital letters listed first. #83 was originally advocating for sorting with case folding, like this:

alpha.pdf
Alpha.pdf
beta.png
Gamma.txt
delta
epsilon.gif
some_folder/
     aardvark.png
     cute_kitten.png
     Cuter_Kitten.png
     sad_rhino.jpg
zeta
Zeta_cool

I still think sorting with case folding would be better/more natural than python's default sorting for a file tree. I also think folders ought to be sorted the same way, among the files (not all at the bottom), though this is at odds with @jimentwood 's suggestion in the issue.

Screenshot 2019-06-14 09 47 58

mhl10 commented 5 years ago

Per discussion with @jimentwood I think it's OK to group the folders at the bottom (or top).

bdc34 commented 5 years ago

The screenshot below shows all the files starting with capital letters listed first. #83 was originally advocating for sorting with case folding, like this:

Casefold has been added.