Ticket #3076 (closed enhancement: fixed)

Opened 21 months ago

Last modified 3 months ago

Journal: add support for copying to Documents folder (detail view, copy-to palette)

Reported by: greenfeld Owned by: erikos
Priority: High Milestone: 0.98
Component: sugar Version: 0.93.x
Severity: Major Keywords: r+, olpc-test-passed
Cc: humitos, erikos Distribution/OS: OLPC
Bug Status: Assigned

Description

The Journal item detail view can copy items to a USB stick (if inserted) and the Clipboard. But it cannot copy the currently active item to the Documents folder if said folder is available from the main Journal list view.

Seen in 11.3.0 os3.

Attachments

0001-Copying-to-Documents-folder-SL-3076.patch Download (2.2 KB) - added by humitos 7 months ago.
This patch is not finished. I don't know how to get the "selected_volume" yet
0001-Copying-to-Documents-folder-SL-3076.2.patch Download (4.1 KB) - added by humitos 7 months ago.
v2 - adds the same behaviour on toolbox's icon
0001-Copying-to-Documents-folder-.v3-SL-3076.patch Download (3.9 KB) - added by godiard 6 months ago.
Add new version (v3) of patch, with the pending issue solved. Now only show the option when needed.

Change History

Changed 21 months ago by erikos

  • keywords 11.3.0 removed
  • summary changed from Journal item detail view cannot copy to Documents folder to Journal: add support for copying to Documents folder (detail view, copy-to palette)
  • type changed from defect to enhancement
  • severity changed from Unspecified to Major
  • status_field changed from Unconfirmed to Assigned

Yes, this is known. Would be a bigger change to add support for that. Moving out to 0.96.

Changed 21 months ago by erikos

  • milestone changed from 0.94 to 0.96

Changed 13 months ago by erikos

  • milestone changed from 0.96 to 0.98

Changed 9 months ago by erikos

  • owner set to erikos
  • status changed from new to assigned

Changed 8 months ago by erikos

  • cc humitos added

Changed 7 months ago by humitos

This patch is not finished. I don't know how to get the "selected_volume" yet

Changed 7 months ago by erikos

  • keywords r? added

Changed 7 months ago by humitos

v2 - adds the same behaviour on toolbox's icon

Changed 7 months ago by humitos

I implemented this behaviour but the value of "selected_folder" variable is missing. I don't know how to (or from where) get it.

This patch adds the same code in two different files and that is not so good. Actually, there are repeated old code on those files that this patch touches.

Changed 6 months ago by godiard

  • priority changed from Unspecified by Maintainer to High

Changed 6 months ago by godiard

  • cc erikos added

Changed 6 months ago by godiard

Add new version (v3) of patch, with the pending issue solved. Now only show the option when needed.

Changed 6 months ago by godiard

Simon, new patch version, I think ready to include.

Changed 6 months ago by erikos

  • keywords r+, olpc-test-pending added; r? removed
  • status changed from assigned to closed
  • resolution set to fixed

Pushed as b01cd3e080a0df6d046057c67b6fb7ff7b0daa63 Changed the icon to be black and white after discussing with Gary.

erikos: garycmartin: the document icon in the Palette, should it be owner-coloured or bw? 
garycmartin: erikos: re document icon in the Palette, owner-coloured or bw? Hmmmm, clip board and USB sticks are showing as B&W at the moment, so for now stick with B&W for the Documents folder, though I'd say really USB sticks and Documents folder should be in user colour (to matches how they are show in the journal volumes toolbar/frame).
erikos: garycmartin: yeah, I had similar thinking 
erikos: garycmartin: but clipboard, is my clipboard as well right?
erikos: garycmartin: why should that be bw?
garycmartin: erikos: The clipboard icon really doesn't work well in user colours, I tried once before, it's just a single fill colour with no outline.
erikos: garycmartin: ok, I feared such an answer
erikos: garycmartin: so, I would vote for bw for the document folder as well
erikos: garycmartin: to have a consistent Palette there
garycmartin: erikos: needs a new icon for that, and then raises the question if it should also be coloured in Activity toolbars...
garycmartin: erikos: +1

Thanks everyone!

Changed 3 months ago by greenfeld

  • keywords olpc-test-passed added; olpc-test-pending removed

The copy-to-journal option exists in OLPC 13.1.0 os27.

Changed 3 months ago by greenfeld

The copy-to-documents folder option rather.

Note: See TracTickets for help on using tickets.