+1 for the extension point
Thanks,
Caty
On Fri, Dec 5, 2014 at 5:08 PM, Marius Dumitru Florea <
mariusdumitru.florea(a)xwiki.com> wrote:
On Wed, Dec 3, 2014 at 7:37 PM, vincent(a)massol.net
<vincent(a)massol.net>
wrote:
On 3 Dec 2014 at 18:34:23, vincent(a)massol.net (vincent(a)massol.net
(mailto:vincent@massol.net)) wrote:
>
>
>
>
>
> On 3 Dec 2014 at 18:05:47, Victor Rachieru (victor.rachieru(a)xwiki.com
(mailto:victor.rachieru@xwiki.com)) wrote:
>
> > Hei devs,
> >
> > By default, the ratings app uses JS to inject an HTML element right
after
> > the #hierarchy element. This is a bit
awkward since that element
could be
> > (re)moved and the ratings element would
not be present or follow the
> > position of the hierarchy (breadcrumb).
> >
> > As a solution I propose implementing an extension point, namely
> > org.xwiki.platform.template.title.before which would go right before
the
> > document title.
>
> Where do you wish to display the rating of a page?
>
> Wouldn’t it be better to display it inside the page content?
>
> By “Before the title” WDYM exactly? I don’t think you really mean
before the
title as otherwise we will have:
>
> *** My title here
>
> Maybe you mean above the title? In which case the extension point would
be a
placeholder taking the full horizontal content space (i.e without the
panels), similar to
http://platform.xwiki.org/xwiki/bin/view/ExtensionPoint/AfterHeaderUIX
>
> Also it looks as if on Colibri the ratings are located to the right of
the
title. So if we want to keep that location “above title" wouldn’t work
either.
>
> “After title” could work in both cases IMO:
> * On colibri, it could right aligned
> * On Flamingo, it could be displayed just on the right of the title
>
> Another possibility is on the same line as the “Last modified by” line
(shortcuts.vm), after the “Last modified by…” text.
>
> Other possibility include:
> * Don’t display it by default and let users include it in their content
using
a macro. They can put it in some view sheet for example.
> * Provide a Panel for displaying it.
>
> I’m not convinced that you’d want to display ratings on all pages by
default.
I don’t know the Rating app, is there a way to deal with this (not
displaying ratings on some pages)? For example we may not want to have
ratings on home pages. I feel that a typical use case would be to display
ratings only for pages containing a specific XObject. BTW I hope you have
that use case that rating can be associated with some id because you may
want to have 2 different apps in your wiki and provide ratings for both so
the ratings value shouldn’t be stored (this means that the {{ratings
id=“…”/}} should accept an “id” parameter).
I meant:
“[…] so the ratings value shouldn’t be stored in a single xobject. There
should be
one xobject per different id.”
Thanks
-Vincent
>
> So personally I wouldn’t display it by default but make it easy for the
admin/organizer to decide where to display it:
> UC1- only in some livetable column (easy)
> UC2- only on some spaces (and possibly excluding some pages like the
home
page)
> UC3- only on a given wiki (and possibly
excluding some pages like the
home page or some spaces like the “xwiki” space)
> UC4- only inside pages having a specific
xobject through the sheet
system (easy)
>
> For UC2 and 3 a Panel would work. A custom skin could work too but it’s
too
much work probably and an easier solution would be nice.
>
> Do you have the list of all use cases? I don’t see them on
http://design.xwiki.org/xwiki/bin/view/Proposal/Extensionrepositoryratings
and that would be useful to have. We need to see the full list of use cases
before we can find solutions, otherwise we risk creating solutions that
will not work for all cases...
>
> WDYT?
I think Victor should focus on what has been included in the roadmap
for 6.4, precisely: "adding the Ratings to the Repository app". Being
able to rate wiki pages is not a priority at the moment IMO so I
wouldn't spend time arguing where the ratings should be displayed for
wiki pages. There can be many options and even more opinions on such a
topic. Moreover this thread is not only about the Ratings app. The
question Victor raised is: do we think an extension point before the
document title is useful? My answer is yes (I've heard complaints that
we have only a few extension points). If we all agree then the Ratings
app can use it for a start to replace the JavaScript injection hack
until "rating plain wiki pages" becomes a priority (or someone is
willing to work on it).
I tried to move the ratings container before the document title and it
looks decent on both Colibri and Flamingo. Note that "before the
title" is the extension point. The ratings container has some CSS that
makes it float to the right of the title where it looks good in the
end.
If adding this extension point helps Victor move forward to the target
of rating extension in the Repository app then I'm +1.
Thanks,
Marius
>
>> Thanks
>> -Vincent
>
>> > Due to the fact that in
flamingo the "more actions" and "edit"
buttons are
> > now in the same place that the ratings
are displayed brings us to a
point
> > where we will have to integrate the
ratings in this interface as well.
> >
> > As a future improvement we could add more extension points and choose
where
> > to display the ratings depending on the
skin.
> > For example: before title for colibri and after the content menu
(where
the
>
"edit" and "more actions" buttons are) for flamingo.
>
> This extension point would replace the JS inject thus making the code
> cleaner.
>
> Thanks,
> Victor
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs