On Mon, Oct 2, 2017 at 7:08 PM, 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? > This depends on the implementation. My point was that we should use jodatime and present some nicer times, just like we have in AS. The full time is present in the title. > > 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 > >>>> > >>>> > >> > > > >
