Ticket #2702 (assigned defect)

Opened 2 years ago

Last modified 8 months ago

"Invite to <activity>" shown even for activities that don't support sharing

Reported by: sascha_silbe Owned by: erikos
Priority: Unspecified by Maintainer Milestone: 1.0
Component: sugar Version: Git as of bugdate
Severity: Minor Keywords:
Cc: rgs, erikos, garycmartin, walter, humitos Distribution/OS:
Bug Status: New

Description

On the Neighbourhood and Group Views, we show "Invite to <activity name>" even for activities that don't support sharing (i.e. sugar.activity.Activity.max_participants = 1).
This is confusing to the user. It advertises a capability that the activity doesn't have and leaves the user to wonder how to use it.

Change History

  Changed 2 years ago by sascha_silbe

  • cc rgs, erikos added

We currently key the invite menu off the "active activity" (jarabe.model.shell.ShellModel.get_active_activity(), based on mapped X11 windows) and don't have API to query the collaboration capabilities of a running activity from within Sugar (the "Shell").

Whatever API we choose, it should be compatible with non-Sugar applications (that get started from within Sugar, e.g. via Terminal). CC'ing rgs as he has done some work on making Gnome applications collaborate with Sugar activities.

  Changed 2 years ago by erikos

  • keywords 11.2.0 added

follow-ups: ↓ 4 ↓ 5   Changed 2 years ago by martin.langhoff

Notes from an outsider... it seems to me that this should not be keyed on max_participants. Instead, it should be keyed on whether a currently open activity is being shared at this moment (ie: if the user has selected 'share with... ')

Perhaps this is also easier -- the telepathy/tubes stack knows about currently running shared sessions.

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

  • cc garycmartin, walter added

Replying to martin.langhoff:

Notes from an outsider... it seems to me that this should not be keyed on max_participants. Instead, it should be keyed on whether a currently open activity is being shared at this moment (ie: if the user has selected 'share with... ')

That was my thought exactly. Unfortunately we currently conflate "no sharing" with "private sharing", at least on the UI side. Pulling in Gary and Walter to (re)consider the UI (Christian does not seem to have a Trac account).

in reply to: ↑ 3   Changed 23 months ago by erikos

Replying to martin.langhoff:

Notes from an outsider... it seems to me that this should not be keyed on max_participants. Instead, it should be keyed on whether a currently open activity is being shared at this moment (ie: if the user has selected 'share with... ')

Perhaps this is also easier -- the telepathy/tubes stack knows about currently running shared sessions.

But the moment you invite the first person to your activity it is not shared yet. So this does not work neither.

As Sascha said we have the information in the activity instance with 'max_participants' but can not access it from the shell.

Another option might be to add the information to the activity.info file, this would at least be accessible by the shell.

  Changed 22 months ago by erikos

  • keywords 11.3.0 added; 11.2.0 removed

  Changed 21 months ago by erikos

  • keywords 11.3.0 removed
  • milestone changed from 0.94 to 0.96

Discussed this with Sascha and Gary at the Sugar Camp:

As stated above the solution will be to add the information to the activity.info file, that way we know in the shell if an activity supports sharing. Entry in the activity.info file set to off by default, the activity maintainer who cares about it can set it and there are more activities that do NOT support sharing.

Sadly too late for this cycle.

  Changed 13 months ago by erikos

  • milestone changed from 0.96 to 0.98

  Changed 13 months ago by humitos

  • cc humitos added

  Changed 8 months ago by erikos

  • owner set to erikos
  • status changed from new to assigned
  • milestone changed from 0.98 to 1.0

Moving this out as it is again uff :( late in the cycle and it would take activity changes.

Note: See TracTickets for help on using tickets.