search

pydio / pydio-sync

Python version of the Pydio synchronization client
https://pydio.com
GNU General Public License v3.0
82 stars 32 forks source link

Display date/time for last files synced. #89

Open andrewleech opened 9 years ago

andrewleech commented 9 years ago

There has been a number of times I've wanted to double check the pydio ui to see if files have synced correctly, opened the recent files drop down, and still not known for sure whether the files listed were synced right now, or a couple of days ago.

This PR adds a date/time column to that list for each file. It shows the time if the file synced today, else the date for older log entries.

cdujeu commented 9 years ago

Hi Andrew, Nice to see more people contributing on this! Just two things:

andrewleech commented 9 years ago

Ah, yes I had seen the CLA, sorry I'd completely forgotten to fill it out. Done Now :-)

Here's the screenshot, it's certainly a valid request. I was unsure about the look at first, displaying both the time and date on each did not work well during initial testing - it pushed each entry onto two lines. Hence going with time or date depending on age. This screenshot shows both. pydio _with_timestamps

cdujeu commented 9 years ago

Mmm, I think our UX guy will put his veto on that. We tried to keep all that as simple as possible. But I understand the need, what about using a tooltip that displays when your mouse flies over files? Also we could use the relativeDate we already use elsewhere to say "moments ago", "2 days ago", etc.. What do you think? -c

stephane-ruhlmann commented 9 years ago

I think the best solution is to combine Moment.js relative 'time from now' method with a tooltip to see the exact date/time.

andrewleech commented 9 years ago

While I understand your position in keeping things as as natural language as possible, when it comes to my valuable data, and knowing for certain if it's synced correctly as I'm leaving the office to go on-site etc, I want certainty if/when it happened and something as arbitrary as 'moments ago' is too indeterminate. Full disclosure, I'm and Engineer/Scientist, so I accept that I'm more particular than much of the public.

Perhaps a setting to switch date/time display from natural to exact, with natural as the default?

I had originally thought a tooltip would be good too, although if it's going to display a full timestamp immediately on hover it's likely to look pretty ugly itself depending on where it's anchored. It also adds extra steps/gestures to get the time information, which I consider a crucial part of any kind of logging readout.

cdujeu commented 9 years ago

Maybe on the right side of "Last Files Sync", we could simply add a "More Info" button, that would switch a display of the sync time, what do you think? I would in that case put that in a second line for each file, as adding column will not be very handy for long file names. We could also display the folder of each file, as here we only display the base name.