> On 3 Oct 2017, at 09:54, Guillaume Delhumeau <guillaume.delhum...@xwiki.com> 
> wrote:
> 
> So your proposal focuses a lot more on the users instead of the apps (like
> facebook for example). It's OK for me, but I think the type icon (create,
> edit, comment, etc...) is too small compared to the user face.
> 
> Actually when I see these screenshot, my first impression is that the user
> has been modified :) Would it be nice to have the type icon bigger than the
> user face? To me, what happened is more important than who made it, but
> maybe it's only my feeling.

On this aspect, I think that the social aspect is important to make people 
contribute so I kind of like that we’re showing avatars. They do seem a bit on 
the largish side in the screenshot.

In any case, I think we need 2 icons: one for the app type and one for the 
event type so that should possibly reduce the size of the visible avatar.

Thanks
-Vincent

> 
> Nice work anyway, as usual :)
> 
> Guillaume
> 
> 2017-10-03 9:50 GMT+02:00 Guillaume Delhumeau <guillaume.delhum...@xwiki.com
>> :
> 
>> 
>> 
>> 2017-10-02 18:09 GMT+02:00 Vincent Massol <vinc...@massol.net>:
>> 
>>> 
>>>> On 2 Oct 2017, at 18:08, Vincent Massol <vinc...@massol.net> wrote:
>>>> 
>>>> 
>>>>> On 2 Oct 2017, at 18:05, Vincent Massol <vinc...@massol.net> wrote:
>>>>> 
>>>>> Another feedback/questions:
>>>>> 
>>>>> * Is the avatar icon clickable and leading to the user profile?
>>>>> * We need a way to link directly to the diff, at least for page events
>>> so that the user can see what was modified (as we have currently in the AS)
>>>> 
>>>> * Maybe “2 hours ago” is not precise enough. In your example I see I
>>> created and modified the Roadmap page but they both say “2 hours ago”.
>>> Shouldn’t I be able to see how far away both events were done?
>>> 
>>> My understanding is that events are grouped by app+type but not by entity
>>> anymore (for page events) and thus you can have 3 events displayed for the
>>> same page, f.ex: Creation, Modification, Deletion.
>>> 
>> 
>> And you could also have different users grouped in the same composite
>> event. So when you display the details, it would be nice to show which user
>> made each event, and I see you have removed this info.
>> 
>> 
>> 
>>> 
>>> Thanks
>>> -Vincent
>>> 
>>>> 
>>>> Thanks
>>>> -Vincent
>>>> 
>>>>> 
>>>>> Thanks
>>>>> -Vincent
>>>>> 
>>>>>> On 2 Oct 2017, at 18:01, Vincent Massol <vinc...@massol.net> wrote:
>>>>>> 
>>>>>> 
>>>>>>> On 2 Oct 2017, at 17:54, Ecaterina Moraru (Valica) <
>>> vali...@gmail.com> wrote:
>>>>>>> 
>>>>>>> Event Type: BlogPostPublishedEvent
>>>>>>> from
>>>>>>> http://design.xwiki.org/xwiki/bin/view/Proposal/Notification
>>> sImprovements9x#HEventTypes
>>>>>> 
>>>>>> ok thanks, had missed this since I was reading from top to bottom and
>>> stopped at the overview :)
>>>>>> 
>>>>>> However I don’t think it scales since it means one unique icon per
>>> combination of app type + event type.
>>>>>> 
>>>>>> What would scale better is two icons: one for the app type and one
>>> for the event type.
>>>>>> 
>>>>>> For example imagine that the Blog app had the following events:
>>>>>> * When a blog post is published
>>>>>> * When a blog post is created
>>>>>> * When a blog post is removed
>>>>>> 
>>>>>> You wouldn’t be able to use the RSS icon to represent the 3 events.
>>>>>> 
>>>>>> BTW the RSS icon isn’t necessarily representative of the Blog app. A
>>> lot of apps can have a RSS feed.
>>>>>> 
>>>>>> That’s the main remark I have: it’s going to be harder and harder to
>>> find unique icons as we had more events to apps, especially if there’s only
>>> 1 icon that is supposed to combine both app type + event type.
>>>>>> 
>>>>>> WDYT?
>>>>>> 
>>>>>> Thanks
>>>>>> -Vincent
>>>>>> 
>>>>>>> Thanks,
>>>>>>> Caty
>>>>>>> 
>>>>>>> On Mon, Oct 2, 2017 at 6:52 PM, Vincent Massol <vinc...@massol.net>
>>> wrote:
>>>>>>> 
>>>>>>>> Hi Caty,
>>>>>>>> 
>>>>>>>>> On 2 Oct 2017, at 17:21, Ecaterina Moraru (Valica) <
>>> vali...@gmail.com>
>>>>>>>> wrote:
>>>>>>>>> 
>>>>>>>>> Hi,
>>>>>>>>> 
>>>>>>>>> I've created some improvements suggestions for our Notifications
>>> UI, see
>>>>>>>>> http://design.xwiki.org/xwiki/bin/view/Proposal/
>>>>>>>> NotificationsImprovements9x
>>>>>>>>> 
>>>>>>>>> Let me know what you think.
>>>>>>>> 
>>>>>>>> Re the overview, I don’t see where you mention the app?
>>>>>>>> 
>>>>>>>> For example, could you show how a new Blog post would be displayed
>>> and you
>>>>>>>> differentiate that for example from a Page creation or modification?
>>>>>>>> 
>>>>>>>> Thanks
>>>>>>>> -Vincent
>>>>>>>> 
>>>>>>>>> 
>>>>>>>>> Thanks,
>>>>>>>>> Caty
>>>>>>>> 
>>>>>>>> 
>>>>>> 
>>>>> 
>>>> 
>>> 
>>> 
>> 
>> 
>> --
>> Guillaume Delhumeau (guillaume.delhum...@xwiki.com)
>> Research & Development Engineer at XWiki SAS
>> Committer on the XWiki.org project
>> 
> 
> 
> 
> -- 
> Guillaume Delhumeau (guillaume.delhum...@xwiki.com)
> Research & Development Engineer at XWiki SAS
> Committer on the XWiki.org project

Reply via email to