[Sugar-devel] Share button in the activity toolbar (from the toolbar redesign thread)

[Simon Schampijer]

On 08/06/2009 04:59 PM, Simon Schampijer wrote:
> On 08/06/2009 03:37 PM, Eben Eliason wrote:
>> On Thu, Aug 6, 2009 at 6:56 AM, Simon Schampijer<simon at schampijer.de>   wrote:
>>> On 07/31/2009 05:14 PM, Eben Eliason wrote:
>>>> On Fri, Jul 31, 2009 at 8:41 AM, Aleksey Lim<alsroot at member.fsf.org>
>>>>    wrote:
>>>>> On Fri, Jul 31, 2009 at 02:05:48PM +0200, Tomeu Vizoso wrote:
>>>>>> On Fri, Jul 31, 2009 at 13:47, Simon Schampijer<simon at schampijer.de>
>>>>>>    wrote:
>>>>>>> Another question that came up today:
>>>>>>>
>>>>>>> Should the activity toolbar contain the "share" option by default?
>>>>>>> Disabled by default?
>>>>>> Maybe we could use the max_participants property to know when to
>>>>>> disable the button?
>>>> This makes sense to me. I think it's useful to keep it in view to
>>>> indicate that the activity is private. A non-collaborative activity
>>>> still has a sharing scope.
>>>>
>>>>> So, the question is - what is default value for max_participants :)
>>>>>
>>>>> In current code it equals to 0 but old sharing component compares it
>>>>> with 1 thus share combo is visible by default.
>>>> An activity that supports 0 participants (or fewer!) sounds pretty
>>>> useless. Clearly 1 is the logical default for this property.
>>>>
>>>>> btw all activities, I've seen before that don't want sharing, hide
>>>>> share combo by share.props.visible not by max_participants, I guess
>>>>> it signalizes about not consistency of current implementation where
>>>>> user can hide/show share button by two different methods.
>>>> Yup, we should push towards consistency here. I'd bet activities
>>>> wouldn't bother to hide it if Sugar made it insensitive properly.
>>>>
>>>> Eben
>>> So, sounds like max-participants should be set to '1' by default. And we use
>> Definitely.
>>
>>> this value to display the sharing option or not.
>>>
>>> Or did you mean to display it and make it sensitive/insensitive, Eben?
>> I think it should always be shown, but made insensitive accordingly
>> based on max_participants.
>>
>> Eben
>
> Sounds good to me.
>
> Thanks for following up,
>      Simon

There had been some discussions on the max_participants property in the 
last days. What we agree on is that the share_button should be made 
insensitive when the activity does not provide that functionality. The 
API at the moment let's you set max_participants to 1, or just use 
sensitive property of the button and set it to false.

The max_participants property makes sense, when we have more things that 
need to change in the UI accordingly, not only the share button. For 
example, the invite option in the neighborhood view. Though the shell 
can not access that property neither. An option would be to move this 
property to the activity.info.
And to give the max_properties a real meaning, we would have to know how 
many participants are currently in that activity. To grey it out in the 
mesh view, when not more people can join or similar. This request would 
need to be made to the PS, not sure if there exist such an option. All 
in all I wonder if this would be a needed functionality for 0.86, or if 
we should better invest in making collaboration more stable.

Comments appreciated,
    Simon