Hello. My personal favorite is http://design.xwiki.org/xwiki/bin/download/Proposal/NotificationsImprovements9xTypes/eventCentered_smallAvatars.png
With this "details" section: http://design.xwiki.org/xwiki/bin/download/Proposal/NotificationsImprovements9xTypes/userCentered_uncropped.png Could we consider having a little extract of the document's content in the notification. For a blog post, I could imagine a little extract, such as: *This release focused on fixing bugs on Notifications, Office Viewer and > Solr Search. The server is...* (extract from http://www.xwiki.org/xwiki/bin/view/Blog/XWiki%2099%20Released ) Thanks, Guillaume 2017-10-27 20:04 GMT+02:00 Clément Aubin <[email protected]>: > Hello ! > > On 10/27/2017 05:31 PM, Ecaterina Moraru (Valica) wrote: > > Hi, > > > > So these are some more iterations for the notifications layout > > http://design.xwiki.org/xwiki/bin/view/Proposal/ > NotificationsImprovements9xTypes > > > > I've played with User centered vs. Event centered and variants for > > displaying the avatars, user names and event names. > > > > Let me know what version you prefer. Also you can fork the prototypes if > > you want to make some small changes. > > I really like the Event-focused "Description with small avatars" options: > > * The notification title does not mentions the full name of the users > involved in this notification (if multiple) ; this reduces the size of > the title. Also, the users involved in the notifications are still > available through their icons. > * The main focus isn't on the users involved in the notification, but on > its content. > > Hope it helps, > Thanks, > > Clément > > > Thanks, > > Caty > > > > On Tue, Oct 24, 2017 at 12:47 PM, Vincent Massol <[email protected]> > wrote: > > > >> > >>> On 24 Oct 2017, at 11:24, Guillaume Delhumeau < > >> [email protected]> wrote: > >>> > >>> Hello > >>> > >>> 2017-10-24 11:12 GMT+02:00 Vincent Massol <[email protected]>: > >>> > >>>> Hi Guillaume, > >>>> > >>>>> On 24 Oct 2017, at 10:58, Guillaume Delhumeau < > >>>> [email protected]> wrote: > >>>>> > >>>>> Any news on this? I'd like to implement this quickly because 9.10 is > >>>> coming > >>>>> :) > >>>> > >>>> This the current status: > >>>> * Caty proposed some improved layout for displaying notifications > >>>> * I commented about the need to display event + type, and about the > need > >>>> to have a link to see the diff. I also mentioned that we need to work > on > >>>> the mail template display > >>>> * Caty replied and updated her proposal to take into account those > >>>> comments. > >>>> * I kind of like Caty’s proposal focused on the user. We want to favor > >>>> contributions and I think it helps. It also makes the UI more > appealing > >>>> visually. > >>>> * You mentioned that you’d prefer a proposal more focused on the event > >>>> rather than the user > >>>> * Thomas agrees with you > >>>> * Edy mentioned some extra points > >>>> > >>> > >>> I have also mentioned that the diff we propose is too technical for > many > >>> users. > >> > >> Yes and we have 4 solutions: > >> > >> * Solution 1: make it simpler to understand for everyone > >> * Solution 2: have a wiki admin UI level option to select whether to > >> include diff or not in the default template + user-level profile UI > option > >> (ie. per user) > >> * Solution 3: have a singe button in the mail, that, when clicked, > >> displays the diff (which is hidden by default) > >> * Solution 4: implement 2 mail templates (one for simple users and one > for > >> advanced users) and let the users choose in their profile which one they > >> want to receive + have a wiki admin UI level option to decide which is > the > >> default when not overridden by the user. > >> > >> Thanks > >> -Vincent > >> > >>> > >>>> So I guess we need to decide about whether we show the avatar as in > >> Caty’s > >>>> proposal or not, right? Is that what you’re asking about Guillaume? > >>>> > >>>> On my side, I don’t see the issue in displaying the user avatar large > vs > >>>> small. The events are still ordered by page events. > >>>> > >>>> Thanks > >>>> -Vincent > >>>> > >>> > >>> Thanks > >>> > >>> > >>>> > >>>>> > >>>>> 2017-10-23 13:19 GMT+02:00 Guillaume Delhumeau < > >>>>> [email protected]>: > >>>>> > >>>>>> Note that you have a mail template for the notifications too. It's > >>>>>> https://github.com/xwiki/xwiki-platform/blob/ > >>>>>> 83b76ed4c26954aa2755bbdce23b32c41725ac06/xwiki-platform- > >>>>>> core/xwiki-platform-notifications/xwiki-platform- > >>>>>> notifications-ui/src/main/resources/XWiki/Notifications/ > >>>>>> MailTemplate.xml#L23 but it's not documented yet. > >>>>>> > >>>>>> This mail template allows you to customize the header and the footer > >> of > >>>>>> the email, but not the actual content. > >>>>>> > >>>>>> To be precise, each *event type* can have its own template or > failback > >>>> to > >>>>>> the default one (https://github.com/xwiki/xwiki-platform/blob/ > >>>>>> acdf68f4c0d20a9b44ea48eec4df808e22c54548/xwiki-platform- > >>>>>> core/xwiki-platform-web/src/main/webapp/templates/ > >>>>>> notification/email/default.html.vm) > >>>>>> > >>>>>> So it's a template *per event type* and it's not linked to any user > >>>>>> choice. But we could add some options in the default template for > >>>> example. > >>>>>> > >>>>>> 2017-10-17 16:23 GMT+02:00 Vincent Massol <[email protected]>: > >>>>>> > >>>>>>> Hi Guillaume, > >>>>>>> > >>>>>>>> On 17 Oct 2017, at 07:16, Guillaume Delhumeau < > >>>>>>> [email protected]> wrote: > >>>>>>>> > >>>>>>>> Some ideas. > >>>>>>>> > >>>>>>>> I have feedback about the fact the diff is way too technical for > >>>> users. > >>>>>>>> > >>>>>>>> Example: "What does it means???" https://img15.hostingpics.net/ > >>>>>>>> pics/670307example1.png > >>>>>>>> > >>>>>>>> So we could hide this in a "technical details" link. > >>>>>>> > >>>>>>> That’s one option but there’s another which is to provide the > ability > >>>> to > >>>>>>> choose which mail template to use in the Admin and provide several > >>>>>>> templates for different needs. > >>>>>>> > >>>>>>> For example for the xwiki.org watchlist we absolutely need to be > >> able > >>>> to > >>>>>>> see the diffs without clicking a “technical details” link for each > >>>>>>> notification. It could be ok if there’s a single “technical > details” > >>>> link > >>>>>>> for ALL notifications that shows all diffs at once. > >>>>>>> > >>>>>>> Thanks > >>>>>>> -Vincent > >>>>>>> > >>>>>>>> > >>>>>>>> The "details" link could be replaced by a "more" link, with a very > >>>>>>>> simplified diff (the complete diff would go the the technical > >> details > >>>>>>> view). > >>>>>>>> > >>>>>>>> In the body of the event, in addition with the title of the > >> document, > >>>> we > >>>>>>>> could add some extracts of the diff. > >>>>>>>> > >>>>>>>> Example taken on gmail: > >>>>>>>> https://img15.hostingpics.net/pics/763456example2.png > >>>>>>>> > >>>>>>>> I hope it helps, > >>>>>>>> > >>>>>>>> Thanks > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>> 2017-10-12 15:38 GMT+02:00 Vincent Massol <[email protected]>: > >>>>>>>> > >>>>>>>>> Looks nice. Does it work in all mail clients that support HTML? > >>>>>>>>> > >>>>>>>>> I'd like an option to show the details for all events as > otherwise > >>>> it's > >>>>>>>>> too painful to have to open all one by one (for the use case when > >> you > >>>>>>> want > >>>>>>>>> to check everything). > >>>>>>>>> > >>>>>>>>> Same as Guillaume: not all events are page-related events. So we > >> need > >>>>>>> to > >>>>>>>>> handle those too. > >>>>>>>>> > >>>>>>>>> Thanks! > >>>>>>>>> -Vincent > >>>>>>>>> > >>>>>>>>>> On 12 Oct 2017, at 15:03, Guillaume Delhumeau < > >>>>>>>>> [email protected]> wrote: > >>>>>>>>>> > >>>>>>>>>> First of all, it looks very nice. > >>>>>>>>>> > >>>>>>>>>> Now, you said the events are displayed like in the > notifications, > >>>> but > >>>>>>> in > >>>>>>>>>> the notifications they are ordered by dates, not by pages. Does > >> your > >>>>>>>>>> proposal imply to change the ordering? What about events that do > >> not > >>>>>>>>>> concern any page (like: "a new wiki has been created" or "a new > >>>> XWiki > >>>>>>>>>> version is available")? We don't have such events for now but we > >>>> plan > >>>>>>> to > >>>>>>>>>> have some soon. We could display them last for example. > >>>>>>>>>> > >>>>>>>>>> Nice CSS expand! > >>>>>>>>>> > >>>>>>>>>> Thanks you, > >>>>>>>>>> > >>>>>>>>>> 2017-10-12 14:47 GMT+02:00 Ecaterina Moraru (Valica) < > >>>>>>> [email protected] > >>>>>>>>>> : > >>>>>>>>>> > >>>>>>>>>>> This is a proposal for the Notifications mail: > >>>>>>>>>>> http://design.xwiki.org/xwiki/bin/view/Proposal/ > >>>>>>>>>>> NotificationsImprovements9xEmail > >>>>>>>>>>> > >>>>>>>>>>> I've continued with the 'user' avatar focused proposal, since > >> I've > >>>>>>>>>>> currently focused on the email template. If we drop the user > >> avatar > >>>>>>>>> (since > >>>>>>>>>>> we group multiple users) then we can have the event/app icon > >>>> instead > >>>>>>> of > >>>>>>>>> the > >>>>>>>>>>> avatar. > >>>>>>>>>>> > >>>>>>>>>>> What I want to have feedback on is the email layout, > >> functionality > >>>>>>> (tree > >>>>>>>>>>> navigation, details expanding) and styling. > >>>>>>>>>>> > >>>>>>>>>>> You can play with the Prototype > >>>>>>>>>>> http://jsfiddle.net/risherry/cj25759w/embedded/#Result > >>>>>>>>>>> > >>>>>>>>>>> If we were to translate to the mail template, there are still > >> some > >>>>>>>>> things > >>>>>>>>>>> to be tested (like translating the CSS selectors into inline, > >>>> decide > >>>>>>>>> what > >>>>>>>>>>> email clients we support, etc.) > >>>>>>>>>>> > >>>>>>>>>>> Let me know, > >>>>>>>>>>> Caty > >>>>>>>>>>> > >>>>>>>>>>> > >>>>>>>>>>> On Tue, Oct 10, 2017 at 5:51 PM, Guillaume Delhumeau < > >>>>>>>>>>> [email protected]> wrote: > >>>>>>>>>>> > >>>>>>>>>>>> 2017-10-03 17:32 GMT+02:00 Ecaterina Moraru (Valica) < > >>>>>>>>> [email protected] > >>>>>>>>>>>> : > >>>>>>>>>>>> > >>>>>>>>>>>>> On Tue, Oct 3, 2017 at 10:50 AM, Guillaume Delhumeau < > >>>>>>>>>>>>> [email protected]> wrote: > >>>>>>>>>>>>> > >>>>>>>>>>>>>> 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. > >>>>>>>>>>>>>> > >>>>>>>>>>>>> > >>>>>>>>>>>>> The focus of this proposal is on user, compared to the old > one, > >>>>>>> when > >>>>>>>>> it > >>>>>>>>>>>> was > >>>>>>>>>>>>> on the app/doc. > >>>>>>>>>>>>> Close to what we have on Facebool. So it didn't made sense > that > >>>> we > >>>>>>> say > >>>>>>>>>>>> that > >>>>>>>>>>>>> a page has been edited by vmassol, and that the details > >> showcase > >>>>>>> that > >>>>>>>>>>>>> multiple user actually did the edit. > >>>>>>>>>>>>> > >>>>>>>>>>>> > >>>>>>>>>>>> In Facebook, you have it actually: > >>>>>>>>>>>> https://pasteboard.co/GOioRAT.png > >>>>>>>>>>>> > >>>>>>>>>>>> On XWiki, we have the following (already implemented): > >>>>>>>>>>>> https://pasteboard.co/GOiqgYf.png > >>>>>>>>>>>> > >>>>>>>>>>>> > >>>>>>>>>>>> Thanks, > >>>>>>>>>>>> > >>>>>>>>>>>> > >>>>>>>>>>>>> > >>>>>>>>>>>>> > >>>>>>>>>>>>>> > >>>>>>>>>>>>>> > >>>>>>>>>>>>>> > >>>>>>>>>>>>>>> > >>>>>>>>>>>>>>> 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 > >>>>>>>>>>>>>> > >>>>>>>>>>>>> > >>>>>>>>>>>> > >>>>>>>>>>>> > >>>>>>>>>>>> > >>>>>>>>>>>> -- > >>>>>>>>>>>> Guillaume Delhumeau ([email protected]) > >>>>>>>>>>>> Research & Development Engineer at XWiki SAS > >>>>>>>>>>>> Committer on the XWiki.org project > >>>>>>>>>>>> > >>>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> -- > >>>>>>>>>> Guillaume Delhumeau ([email protected]) > >>>>>>>>>> Research & Development Engineer at XWiki SAS > >>>>>>>>>> Committer on the XWiki.org project > >>>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>> -- > >>>>>>>> Guillaume Delhumeau ([email protected]) > >>>>>>>> Research & Development Engineer at XWiki SAS > >>>>>>>> Committer on the XWiki.org project > >>>>>>> > >>>>>>> > >>>>>> > >>>>>> > >>>>>> -- > >>>>>> Guillaume Delhumeau ([email protected]) > >>>>>> Research & Development Engineer at XWiki SAS > >>>>>> Committer on the XWiki.org project > >>>>>> > >>>>> > >>>>> > >>>>> > >>>>> -- > >>>>> Guillaume Delhumeau ([email protected]) > >>>>> Research & Development Engineer at XWiki SAS > >>>>> Committer on the XWiki.org project > >>>> > >>>> > >>> > >>> > >>> -- > >>> Guillaume Delhumeau ([email protected]) > >>> Research & Development Engineer at XWiki SAS > >>> Committer on the XWiki.org project > >> > >> > -- Guillaume Delhumeau ([email protected]) Research & Development Engineer at XWiki SAS Committer on the XWiki.org project
