Opened 9 years ago

Closed 9 years ago

#4718 closed defect (fixed)

Missing options in Journal batch copy palette

Reported by: tch Owned by: tch
Priority: Unspecified by Maintainer Milestone: Unspecified
Component: Sugar Version: Unspecified
Severity: Unspecified Keywords: journal batch copy
Cc: godiard, waltertch Distribution/OS: Unspecified
Bug Status: Unconfirmed

Description

The palette for batch copy-to, in the Journal, doesn't show options for external volumes.

To reproduce:

  1. Boot the laptop with the memory stick plugged in.
  2. Once in Sugar, go to the Journal.
  3. Select any entry to activate the batch operations toolbar.
  4. Click on the batch copy-to icon.

Outcome:
You should see no options for the memory stick.

Some clues:
if go to volumes or documents view and go back, then the option appears.

Change History (2)

comment:1 Changed 9 years ago by tch

On sugar-build, it can be reproduced by first entering to the journal and then plugging the memory stick.

I am sending a pull request now.

comment:2 Changed 9 years ago by Martin Abente Lahaye

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

Fix missing options in batch copy palette

Its too early for using CopyMenuBuilder, wait until
the palette its propely initialized.

I do not fully understand why this is happening.
My only guess is that there could be some async
stuff happening between the constructor and callback.

Fixes #4718

Signed-off-by: Martin Abente Lahaye <tch@…>

Changeset: 87d3199c0ec6ed53cb1c5b2b0e069a442c3ab648

Note: See TracTickets for help on using tickets.