24 Oct
2017
24 Oct
'17
11:25 a.m.
2017-10-24 11:16 GMT+02:00 Ecaterina Moraru (Valica) <valicac(a)gmail.com>om>:
I don't have any new proposal right now. I could
do another iteration
starting from tomorrow.
If you need help, please ask me.
I'll switch to an other topic until I can implement your ideas.
Thanks,
Thanks,
Caty
On Tue, Oct 24, 2017 at 12:12 PM, Vincent Massol <vincent(a)massol.net>
wrote:
able
your
I've
functionality
vincent(a)massol.net>
example I
done?
showcase
3
if
--
Guillaume Delhumeau (guillaume.delhumeau(a)xwiki.com)
Research & Development Engineer at XWiki SAS
Committer on the XWiki.org project
Hi Guillaume,
Caty’s
On 24 Oct 2017, at 10:58, Guillaume Delhumeau
<
guillaume.delhumeau(a)xwiki.com> 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
So I guess we need to decide about whether we show the avatar as in 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
>
> 2017-10-23 13:19 GMT+02:00 Guillaume Delhumeau <
> guillaume.delhumeau(a)xwiki.com>gt;:
>
>> 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 <vincent(a)massol.net>et>:
>
>> Hi Guillaume,
>>
>>> On 17 Oct 2017, at 07:16, Guillaume Delhumeau <
>> guillaume.delhumeau(a)xwiki.com> 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 to
details
>> 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 >>> view).
>>>>
>>>> In the body of the event, in addition with the title of the
document,
we
you
>>> 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 <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
>>> 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
>>>>>> 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 >>>>>>> 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, >>> (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
:
>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>> On 2 Oct 2017, at 18:08, Vincent Massol
<vincent(a)massol.net
>>>>>>>> wrote:
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>>> On 2 Oct 2017, at 18:05, Vincent
Massol <
>>>>>>> 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
>>>>>> 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 >>>>>>>>>>>
>>>>>>>>>>> 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 >>> 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
>>>>>>> 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 >>>>>>>> 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 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