Opened 6 years ago

Closed 10 months ago

#328 closed defect (fixed)

no feedback when you Start a journal entry that no activity can open

Reported by: skierpage Owned by: tomeu
Priority: Normal Milestone: Unspecified
Component: Sugar Version: 0.82.x
Severity: Minor Keywords: r?
Cc: smparrish Distribution/OS:
Bug Status: New

Description

This is a copy of http://dev.laptop.org/ticket/8733 , is it OK to duplicate bugs in the two Tracs?

In Sugar 0.82.1 on staging-27 (an 8.2.1 pre-release) on an XO, I inserted a USB flash drive containing a .mp3 file. In the Journal I can hover over the icon and a menu appears with Start; and I can click (>) for more details then click the [>] Start menu item. But these do nothing.

The JournalActivity log contains

WARNING root: No activity can open this object, audio/mpeg.

I think the Journal should show some kind of warning, or better it should disable the Start menu item and replace its hover dropdown with a disabled "Start (no installed activity can open this)".

Attachments (1)

0001-Disable-Start-menu-item-for-entries-that-can-t-be-op.patch (4.8 KB) - added by mukul 4 years ago.

Download all attachments as: .zip

Change History (15)

comment:1 in reply to: ↑ description Changed 6 years ago by tomeu

  • Bug Status changed from Unconfimed to Assigned
  • Milestone changed from Unspecified by Release Team to 0.84
  • Severity changed from Unspecified to Minor
  • Status changed from new to assigned

Replying to skierpage:

This is a copy of http://dev.laptop.org/ticket/8733 , is it OK to duplicate bugs in the two Tracs?

Yes, ideally in laptop.org would be only bugs specific to the OLPC hw/distro and in sugarlabs.org the ones that are independent of the platform. But downstream might also want to trac upstream bugs in their own tracker.

In Sugar 0.82.1 on staging-27 (an 8.2.1 pre-release) on an XO, I inserted a USB flash drive containing a .mp3 file. In the Journal I can hover over the icon and a menu appears with Start; and I can click (>) for more details then click the [>] Start menu item. But these do nothing.

The JournalActivity log contains

WARNING root: No activity can open this object, audio/mpeg.

I think the Journal should show some kind of warning, or better it should disable the Start menu item and replace its hover dropdown with a disabled "Start (no installed activity can open this)".

Makes sense, though we are in string freeze so this may not make 0.84 :/

comment:2 Changed 6 years ago by erikos

  • Milestone changed from 0.84 to 0.86

Let's handle this in 0.86

comment:3 Changed 6 years ago by homunq

  • Keywords sugar-love added

comment:4 Changed 4 years ago by smparrish

  • Cc smparrish added
  • Milestone changed from 0.86 to 0.88.x

comment:5 Changed 4 years ago by smparrish

  • seeta_dev set to Mukul

comment:6 Changed 4 years ago by mukul

  • Keywords r? added; sugar-love removed

comment:7 Changed 4 years ago by FGrose

  • Distribution/OS OLPC deleted

For an uninstantiated Journal entry in Sugar 0.88.1, the 'Start with' menu
item reveals an inactive, side sub-palette with the words 'No activity to
start entry' in gray.

Should we not try to keep the wording consistent for this similar condition?

Adding 'installed' to the terse wording may help the Learner understand, but
it is still quite technical for the general audience. Would it be better to
use more general wording, such as, "No activity is available to start the
entry."

Since it is a side sub-palette, the additional length should not be a
hindrance, and any additional clarity would better match Sugar's goals.

In messages like this, a grammatically complete sentence (also ending with a
period) is also very helpful and instructive for Learners.

In summary, if this reasoning is accepted, the sub-palette message should be
changed in at least 2 places.

Mailing list discussion in these threads:
http://www.mail-archive.com/sugar-devel@lists.sugarlabs.org/msg17357.html
http://www.mail-archive.com/sugar-devel@lists.sugarlabs.org/msg17115.html

comment:8 Changed 4 years ago by alsroot

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

comment:9 Changed 4 years ago by smparrish

  • Resolution fixed deleted
  • Status changed from closed to reopened

I have applied this patch to Dextrose and it does indeed fix the issue with 1 part of the hover menu. However if the user selects 'View Details" from the hover menu there is a start button at the top of the screen. This also needs to be disabled for this to be considered fixed.

Steven

comment:10 Changed 15 months ago by dnarvaez

  • Component changed from journal to sugar

comment:11 Changed 14 months ago by dnarvaez

  • Milestone changed from 0.88.x to Unspecified

comment:12 Changed 14 months ago by dnarvaez

  • Bug Status changed from Assigned to Unconfirmed

comment:13 Changed 13 months ago by dnarvaez

  • Bug Status changed from Unconfirmed to New
  • Priority changed from Unspecified by Maintainer to Normal

comment:14 Changed 10 months ago by Gonzalo Odiard

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

Show feedback in detailview when sugar can't manage a entry - SL #328

The same criteria and message than in the ObjectPalette is used.

Fixes #328

Signed-off-by: Gonzalo Odiard <gonzalo@…>

Changeset: c88672db9d5643a0dc632cbebed5406f3fcab358

Note: See TracTickets for help on using tickets.