- Notifications
You must be signed in to change notification settings - Fork5.5k
Add sort by last modified date and name to file tree#943
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
ellisonbg commentedJan 10, 2016
We should use moment.js to format the last modified datetime... |
captainsafia commentedJan 10, 2016
I'm currently using moment. Just creating a light wrapper function with the format string. |
ellisonbg commentedJan 10, 2016
Great! On Sat, Jan 9, 2016 at 10:58 PM, Safia Abdallanotifications@github.com
Brian E. Granger |
dsblank commentedJan 10, 2016
This is brilliant! This will help so much in managing courses based on the notebook. It will help in finding recent lecture notes, for finding assignments submitted early/late, and myriad other tasks. Thank you! |
captainsafia commentedJan 10, 2016
OK. So sorting on the front-end now works. I'm figuring out how to make the last modified button in the header have a more meaningful text. I'm thinking of having it say "Sort By Date Modified" and then an arrow icon that indicates in what order the list will be sorted on if you click the button. Thoughts on the UX for this? |
captainsafia commentedJan 12, 2016
OK. Taking the WIP flag off of this. I set up the UX to work as follows.
EDIT: Updated gif screencast. |
ellisonbg commentedJan 12, 2016
Safia, I think that being able to sort by last modified is very nice. Love However, I think the last modified dates should be written in plain english
That was what I meant about using moment.js for that. On Mon, Jan 11, 2016 at 4:34 PM, Safia Abdallanotifications@github.com
Brian E. Granger |
blink1073 commentedJan 12, 2016
Share and share alike!https://github.com/jupyter/jupyter-js-filebrowser/blob/master/src/listing.ts#L933 |
ellisonbg commentedJan 12, 2016
Thanks! On Mon, Jan 11, 2016 at 4:39 PM, Steven Silvesternotifications@github.com
Brian E. Granger |
captainsafia commentedJan 12, 2016
@ellisonbg: OOOOOOOOOOOOOOOOOOOOHHHHHHHHHHHHHHHHHHH! codes furiously |
ellisonbg commentedJan 12, 2016
LOL :-) On Mon, Jan 11, 2016 at 4:47 PM, Safia Abdallanotifications@github.com
Brian E. Granger |
captainsafia commentedJan 12, 2016
OK. Updated to have human-friendly dates now. Thanks@blink1073 for that handy-dandy function. |
ellisonbg commentedJan 12, 2016
Very nice, I think that will be a huge help to folks. Question - after On Mon, Jan 11, 2016 at 5:03 PM, Safia Abdallanotifications@github.com
Brian E. Granger |
captainsafia commentedJan 12, 2016
I think it is possible. I can copy and modifythe callback in the Let me see if I can get that to work. |
dsblank commentedJan 12, 2016
Although the fromNow time span is useful in knowing approximately how old something is, for assignments and other date-oriented tasks, the actual date/time is more useful. How many students turned their assignments in in the last few moments before the due date? Can we see both date and span? Or make it toggle between them? |
ellisonbg commentedJan 12, 2016
What about adding the actual date/time as a tooltip? On Mon, Jan 11, 2016 at 6:20 PM, Doug Blanknotifications@github.com
Brian E. Granger |
dsblank commentedJan 12, 2016
Yes, that would be better. Although, you might consider showing the date/time in the list and having the more casual, human fromNow() text be the tooltip. That would be more standard (I couldn't find any example of file date/time listings that show anything other than date/time.... Google Docs, my file manager, ls, etc.) |
takluyver commentedJan 12, 2016
The Github web interface shows e.g. '3 days ago': https://github.com/jupyter/notebook I can see arguments either way. For a quick 'about how recent is this', I prefer the 'ago' style, but sometimes it's useful to have the detailed information. |
dsblank commentedJan 12, 2016
I guess the github method is very relevant to github, because files are always seen in terms of their last changed status to the whole project. But I don't think that is typical for a list of files of unknown relationships to each other, or to the user's goals/intentions. Tooltip or not, this is a great usability gain! |
Carreau commentedJan 12, 2016
+1 How does that looks with started kernels, selected files ? |
willingc commentedJan 15, 2016
@Carreau@blink1073 I will take a look later as well. |
captainsafia commentedJan 16, 2016
Can this be merged? |
Add sort by last modified date and name to file tree
ellisonbg commentedJan 16, 2016
Great work! On Sat, Jan 16, 2016 at 10:30 AM, Matthias Bussonnier <
Brian E. Granger |
gnestor commentedOct 24, 2016
What do you guys think about including in this in a minor release vs. major? It was implemented in the beginning of the year and there's a good chance it won't make it into a release (5.0) until next year... |
rgbkrk commentedOct 24, 2016
I'm certainly a fan of targeting this to a minor release. |
michaelaye commentedDec 14, 2016
I am eagerly awaiting this feature! |
gnestor commentedDec 14, 2016
@Carreau@ellisonbg What do you say about including this in 4.3.1 or 4.4? |
takluyver commentedDec 14, 2016
I'd say no to 4.3.1, because it's not a bugfix. I'd like to do a feature release soon, whether we call that 4.4 or 5.0. |
gnestor commentedDec 14, 2016
Ok, I'm going to try to push this through with 4.4 because there are lots of users waiting for it and it's been almost a year since it was merged! |
Carreau commentedDec 14, 2016
I would be fine pushing things from 5.0 to 5.1, there are a lot of new feature in master and I think trying to backport will be too painful. We should check that the shortcut editor still work (or fix it). It uses preact, but apparently we can move to react if needed now that the discussion has ended. |
takluyver commentedDec 15, 2016
Yeah, there are a range of new features in master that people would like to get out (I'm keen to see my clipboard work released, for instance). If 4.x is open to new features, then people will want to backport many things. I suspect it's easier to aim for a 5.0 release from master. |
gnestor commentedDec 15, 2016
Ok, I agree with you guys. However, a 5.0 release will require a revert back to no Webpack (#1554), correct? I know@blink1073 did some exploration (#1793) and determined that reverting is our the option, so I think that leaves that task up to him or me. Additionally, if we intend to release 5.0 soon, we have 150 issues to triage! 😮 So I suppose that we should work on the bug fixes and critical ones and mark the rest as "5.x". |
esvhd commentedApr 5, 2017
Hi guys, This is a great feature, thank you very much for the effort. After upgrading to This seems to be very confusing, reading the above, it feels like the Is there a way to change this so that the sort order does not reset when switching around tabs/apps? Thanks. Will |
michaelaye commentedApr 5, 2017
esvhd commentedApr 5, 2017
@michaelaye you are right. I think they might be the same. Any background event seems to be trigging a reset of sort order. Maybe changing the default is an easy workaround. |
deeTEEcee commentedOct 1, 2020 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
im a bit late here but what about adding "created at"? |




Closes#930.
Working on adding sorting by last modified date.
cc:@minrk