26 Apr
2018
26 Apr
'18
9:37 a.m.
On Wed, Apr 25, 2018 at 4:53 PM, Thomas Mortagne <thomas.mortagne(a)xwiki.com>
wrote:
So it seems that 2) is currently leading with
Ecaterina and Vincent.
Guillaume I'm not sure if you prefer 2) or 3) (i.e. what do you think
about delete ?).
Marius, does your proposal means you are more for 1)
but with the
difference that the default color theme would be allowed for edit ?
Yes, but I'm ok with 2)
Any other point of view input ?
On Wed, Apr 25, 2018 at 1:50 PM, Ecaterina Moraru (Valica)
<valicac(a)gmail.com> wrote:
start
On Wed, Apr 25, 2018 at 2:02 PM, Thomas Mortagne
<
thomas.mortagne(a)xwiki.com>
wrote:
> On Wed, Apr 25, 2018 at 12:08 PM, Vincent Massol <vincent(a)massol.net>
> wrote:
> > To give my opinion, I’m hesitating about 2 approaches:
> >
> > Approach 1: “standard"
> > ==================
> >
> > * We should have “Default Color Theme” be a copy from the Iceberg
color
> theme page. I think I’d prefer the copy to be
done at runtime; for
example
> if the currently defined color theme page
doesn’t exist when going to
the
> L&F > Themes admin page, create it by
copying Iceberg. This provides a
self
> healing feature if the color theme page has
been removed/doesn’t
exist/etc.
> >
> > * Predefined color theme pages should be considered “standard” and a
> warning message displayed if a user wants to edit them. BTW would be
nice
> to have a feature to have a customized
message per “type”. For example
for
> color theme pages we would display a message
saying that the user should
> copy the page to customize it instead of editing it.
> >
> > * The Color Theme UI should also be improved a bit to have a
“Customize”
> button on color theme pages that would
perform a copy to let the user
> customize a theme.
> >
> > Approach 2: “demo"
> > ================
> >
> > * Consider all color themes to be demo content and once the user
starts
> modifying them don’t merge them anymore
> > * When we want to provide modified color themes, introduce new theme
> pages
> > * Don’t provide a “Default Color Theme” page. Directly set “Iceberg”
to
be the
default CT.
For me delete is ok too. IMO we should provide just a few themes by
default, and the user should be able to uninstall and install what themes
he wants (ideally he would be able to preview them live).
I don't like the copying part very much. Users like to have a base to
Analysis
=======
Approach 2 is more wiki style and simpler for sure. Users can use the
diff feature
and the rollback feature if they want to go back to the
original versions.
I think I’m leaning more towards 2 ATM.
So you think delete is OK too, right ?
from, but they also have history as a fallback.
Also we rarely make changes to ColorThemes, especially since they are not
very complex since they should contain only variables. Still it all
depends
on how well the Default Theme is tested
initially.
Thanks,
Caty
>
> >
> > Thanks
> > -Vincent
> >
> >> On 25 Apr 2018, at 11:35, Vincent Massol <vincent(a)massol.net> wrote:
> >>
> >> Is this a VOTE or a proposal or a brainstorming? I’m asking since
> nobody has voted yet, not even Thomas (except if we consider that
“prefer”
> means +1 and “OK” means +0 ;)).
> >>
> >> From the answer it seems we might need a new VOTE since several new
> points have been added to the discussion. I’m not able to VOTE right
now.
> >>
> >> Thanks
> >> -Vincent
> >>
> >>> On 23 Apr 2018, at 12:29, Thomas Mortagne <
thomas.mortagne(a)xwiki.com>
> wrote:
> >>>
> >>> Hi xwikiers,
> >>>
> >>> Following new XAR entry type mail here is a question about color
> >>> themes we provide in standard XWiki (Cerulean, Charcoal, etc.).
> >>>
> >>> 1) Standard stuff, if you want a custom color theme create a new one
> >>> (would be nice to be able to copy a standard one and propose it when
> >>> you try to edit a standard one).
> >>>
> >>> 2) Demo content, edit and delete it all you want and upgrade won't
> >>> touch a customized theme to avoid surprises (background color
changed
> >>> a bit in the standard version
which now collide with your logo)
> >>>
> >>> 3) Same as 2 but delete is bad (same as home page)
> >>>
> >>> WDYT ?
> >>>
> >>> I'm think I prefer 1) but I'm OK with 3) if other think it's
more
> >>> example than standard material. Let's say -0 for 2).
> >>>
> >>> Thanks,
> >>> --
> >>> Thomas Mortagne
> >>
> >
>
>
>
> --
> Thomas Mortagne
>
--
Thomas Mortagne