Some ideas.
I have feedback about the fact the diff is way too technical for users.
Example: "What does it means???"
pics/670307example1.png
So we could hide this in a "technical details" link.
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:
I hope it helps,
Thanks
2017-10-12 15:38 GMT+02:00 Vincent Massol <vincent(a)massol.net>et>:
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
<
guillaume.delhumeau(a)xwiki.com> 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) <valicac(a)gmail.com
:
> 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 <
> guillaume.delhumeau(a)xwiki.com> wrote:
>
>> 2017-10-03 17:32 GMT+02:00 Ecaterina Moraru (Valica) <
valicac(a)gmail.com
>> :
>>
>>> On Tue, Oct 3, 2017 at 10:50 AM, Guillaume Delhumeau <
>>> guillaume.delhumeau(a)xwiki.com> wrote:
>>>
>>>> 2017-10-02 18:09 GMT+02:00 Vincent Massol <vincent(a)massol.net>et>:
>>>>
>>>>>
>>>>>> On 2 Oct 2017, at 18:08, Vincent Massol
<vincent(a)massol.net>
>> wrote:
>>>>>>
>>>>>>
>>>>>>> On 2 Oct 2017, at 18:05, Vincent Massol
<vincent(a)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.
>>>>
>>>
>>> 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
<vincent(a)massol.net>
> wrote:
>>>>>>
>>>>>>
>>>>>>> On 2 Oct 2017, at 17:54, Ecaterina Moraru (Valica) <
>> valicac(a)gmail.com>
>>> 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 <
> vincent(a)massol.net>
>>> wrote:
>>>>>>>
>>>>>>>> Hi Caty,
>>>>>>>>
>>>>>>>>> On 2 Oct 2017, at 17:21, Ecaterina Moraru (Valica)
<
>>> valicac(a)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.delhumeau(a)xwiki.com)
>> Research & Development Engineer at XWiki SAS
>> Committer on the
XWiki.org project
>>
>
--
Guillaume Delhumeau (guillaume.delhumeau(a)xwiki.com)
Research & Development Engineer at XWiki SAS
Committer on the
XWiki.org project
--
Guillaume Delhumeau (guillaume.delhumeau(a)xwiki.com)
Research & Development Engineer at XWiki SAS
Committer on the
XWiki.org project
--
Guillaume Delhumeau (guillaume.delhumeau(a)xwiki.com)
Research & Development Engineer at XWiki SAS
Committer on the