Opened 11 years ago

Closed 11 years ago

Last modified 7 years ago

#1215 closed enhancement (fixed)

add technical details to Journal details view

Reported by: sascha_silbe Owned by: sascha_silbe
Priority: Unspecified by Maintainer Milestone:
Component: Sugar Version: Git as of bugdate
Severity: Unspecified Keywords: r+
Cc: eben, garycmartin Distribution/OS: Unspecified
Bug Status: New

Description

This patch series is an attempt at implementing the technical details (file size, file type, file date) info in the Journal details view as demonstrated in Eben's mockup and thus #517 / #1033.

Open questions (to the designers):
a) what to do if mime_type is not set (currently: displays "Type: ")
b) what to do if filesize is 0 (currently: displays "Size: 0B")

I took the liberty of displaying the MIME type instead of the file extension because it's more appropriate IMO. File extensions are for legacy systems that need to store metadata in the filename and lose some of the information contained in the MIME type.

Attachments (4)

0001-add-sugar.util.format_size.patch (844 bytes) - added by sascha_silbe 11 years ago.
add sugar.util.format_size()
journal-technical-details.png (26.5 KB) - added by sascha_silbe 11 years ago.
Sample screenshot
0001-add-filesize-to-returned-metadata.patch (2.9 KB) - added by sascha_silbe 11 years ago.
sugar-datastore: add 'filesize' property to metadata returned by API calls
0001-add-technical-details-to-details-view-in-Journal.patch (3.4 KB) - added by sascha_silbe 11 years ago.
add technical details to details view in Journal (sugar-datastore change obsolete now)

Download all attachments as: .zip

Change History (10)

comment:1 follow-up: Changed 11 years ago by sascha_silbe

  • Cc eben garycmartin added

Forgot to CC the designers, so asking the questions again:

a) what to do if mime_type is not set (currently: displays "Type: ")
b) what to do if filesize is 0 (currently: displays "Size: 0B")

comment:2 in reply to: ↑ 1 ; follow-up: Changed 11 years ago by garycmartin

Replying to sascha_silbe:

a) what to do if mime_type is not set (currently: displays "Type: ")

"Kind: <mime_type>" (as per Eben's mock-up and your screen shot look good. If there is no associated mime type I'd suggest showing "Kind: Unknown".

b) what to do if filesize is 0 (currently: displays "Size: 0B")

For "Size: 0B" case I would recommend spelling it out "Size: Zero". Also note the space after the digits as in Eben's mock-up helps readability "Size 215 KB".

Quick sanity check :-) The "Date: 17.08.2009" is shown in the correct locale date format, yes? Is it the file modification date or the file creation date (or does that end up being the same due to the way data-store works)?

Changed 11 years ago by sascha_silbe

add sugar.util.format_size()

Changed 11 years ago by sascha_silbe

Sample screenshot

comment:3 in reply to: ↑ 2 Changed 11 years ago by sascha_silbe

Replying to garycmartin:

"Kind: <mime_type>" (as per Eben's mock-up and your screen shot look good. If there is no associated mime type I'd suggest showing "Kind: Unknown".

OK, Unknown it is.

b) what to do if filesize is 0 (currently: displays "Size: 0B")

For "Size: 0B" case I would recommend spelling it out "Size: Zero". Also note the space after the digits as in Eben's mock-up helps readability "Size 215 KB".

Spaces added, but spelled it "Empty" instead for zero.

Quick sanity check :-) The "Date: 17.08.2009" is shown in the correct locale date format, yes?

It is. Haven't tried any other locale, but as US format is the default I'm quite sure it works. :)

Is it the file modification date or the file creation date (or does that end up being the same due to the way data-store works)?

There's only one timestamp in the data store.

Thanks for the quick feedback!

Changed 11 years ago by sascha_silbe

sugar-datastore: add 'filesize' property to metadata returned by API calls

Changed 11 years ago by sascha_silbe

add technical details to details view in Journal (sugar-datastore change obsolete now)

comment:4 follow-up: Changed 11 years ago by tomeu

  • Keywords r+ added; r? removed
  • Owner changed from tomeu to sascha_silbe
  • Status changed from new to assigned

Looks great, other than if file_path: should probably check for is None. Please push, thanks!

comment:5 in reply to: ↑ 4 Changed 11 years ago by tomeu

  • Resolution set to fixed
  • Status changed from assigned to closed

Replying to tomeu:

Looks great, other than if file_path: should probably check for is None. Please push, thanks!

Disregard this, have just pushed it.

comment:6 Changed 7 years ago by dnarvaez

  • Milestone 0.86 deleted

Milestone 0.86 deleted

Note: See TracTickets for help on using tickets.