2017-10-02 18:09 GMT+02:00 Vincent Massol <[email protected]>: > > > On 2 Oct 2017, at 18:08, Vincent Massol <[email protected]> wrote: > > > > > >> On 2 Oct 2017, at 18:05, Vincent Massol <[email protected]> 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 <[email protected]> wrote: > >>> > >>> > >>>> On 2 Oct 2017, at 17:54, Ecaterina Moraru (Valica) <[email protected]> > wrote: > >>>> > >>>> Event Type: BlogPostPublishedEvent > >>>> from > >>>> http://design.xwiki.org/xwiki/bin/view/Proposal/ > NotificationsImprovements9x#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 <[email protected]> > wrote: > >>>> > >>>>> Hi Caty, > >>>>> > >>>>>> On 2 Oct 2017, at 17:21, Ecaterina Moraru (Valica) < > [email protected]> > >>>>> 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 ([email protected]) Research & Development Engineer at XWiki SAS Committer on the XWiki.org project
