On Tue, Mar 20, 2018 at 12:45 PM, Vincent Massol <vincent(a)massol.net> wrote:
On 20 Mar 2018, at 10:49, Ecaterina Moraru
(Valica) <valicac(a)gmail.com>
wrote:
On Tue, Mar 20, 2018 at 10:18 AM, Vincent Massol <vincent(a)massol.net>
wrote:
Hi Caty and all,
> On 19 Mar 2018, at 14:15, Ecaterina Moraru (Valica) <valicac(a)gmail.com
> wrote:
>>
>> On Sun, Mar 18, 2018 at 1:09 PM, Vincent Massol <vincent(a)massol.net>
> wrote:
>>
>>> Hi Caty and all,
>>>
>>> I’m fine with creating the color-themes repo on xwiki-contrib for
themes
>>> that are not good enough in term of
quality and that we don’t want to
>>> bundle in XS as a consequence.
>>>
>>> In term of naming I’d propose: “application-colorthemes” to be in sync
>>> with our current naming scheme (see
http://contrib.xwiki.org/
>>> xwiki/bin/view/Main/WebHome#HChoosingthename).
>>
>>
>> This repo will contain just Color Themes instances, not the
application.
>> The application and UI is found at
>>
https://github.com/xwiki/xwiki-platform/tree/master/
> xwiki-platform-core/xwiki-platform-flamingo/xwiki-
> platform-flamingo-themes/xwiki-platform-flamingo-theme-ui/
>> also your proposal is kind of conflicting with the old ColorThemes
>> Application, see
>>
http://extensions.xwiki.org/xwiki/bin/view/Extension/
> Color%20Theme%20Application
>>
>>
>>> An alternative would be to introduce a new prefix “colortheme-“ and
use
>>> something like “colortheme-default”
or “colortheme-pack1” or …. This
> second
>>> option is interesting if we want other color theme repos to exist. If
we
>>> want all color themes to go into a
single repo then the 1st naming
> option
>>> seems better.
>>>
>>
>> I would like us to add the color-themes prefix.
>
> I saw what you did but:
> 1) it seems you didn’t add a prefix, you just named the repo
> “color-themes” (that’s its full name)
> 2) you didn’t update the
http://contrib.xwiki.org/
> xwiki/bin/view/Main/WebHome#HChoosingthename page
>
> However, before it can be added to
http://contrib.xwiki.org/
> xwiki/bin/view/Main/WebHome#HChoosingthename could you explain how it’s
> supposed to work because it’s not clear to me.
>
> In my previous reply I mentioned that we had 2 choices:
> A - use a color-themes prefix but then you need a suffix and it means we
> accept other repos also starting with color-themes.
> B - or have a single repo for all color-themes in case we don’t want to
> accept other repos for color themes. In this case we need to mention at
>
http://contrib.xwiki.org/xwiki/bin/view/Main/WebHome#HChoosingthename
> that the name is reserved and that all color themes need to go in
there. In
this case
“color-themes” is not a prefix, it’s just the name of this
special repo that accepts all color themes.
I went for B, putting multiple color-themes in the same repo, but this
doesn't prevent the creation on contrib of individual repos using the
"color-theme" prefix.
well that causes a problem. It means that the "color-themes” repo is
special and more important than the other ones. Why?
That’s why I suggested using a suffix in this case, as in
“color-themes-standard”, “color-themes-default”, “color-themes-pack1”, etc.
So IMO it shouldn’t be a prefix with what you’ve done but it should be
documented on
contrib.xwiki.org.
I prefer having the themes grouped together since
it's a pain to do all
the
steps required for a Contrib repo (mail, repo,
jira project, release,
etc.)
when someone wants to contribute a color-theme.
Also the color themes are
very small and related between them, so grouping them seems good to me,
but
as individual modules, since this allows to see
the individual installs /
popularity, plus allow to define different versioning / dependencies.
> Could you tell me what you’ve chosen and what you prefer? Seems you’re
> going for B but I’m not sure.
>
>> It follows the naming
>> scheme and we do similar things for skins and icon-themes,
>
> I don’t agree for skins. They’re large things and have different release
> cycles and they should each have their own repo IMO.
>
> For icon-themes I’m not sure. What do others think?
>
>> examples:
>> skin-bluebird, skin-leiothrix, icon-theme-material,
> icon-theme-glyphicons,
>> so we will have color-theme-dawn, etc. The repository application also
>> knows the Skin, Color Theme and Icon Theme categories, so there will be
>> some consistency between theme-ing entities.
>
> For icon-theme-fontawesome, I thought that was something we had by
default
in XS?
The stable 4.7.0 version is in XS by default.
So are you saying that we bundle a contrib extension in XS for icon
themes? I didn’t notice this. Could you tell me when this was done? (Also
note that this is NOT setup in
http://jira.xwiki.org/secure/
BrowseProjects.jspa#10000 ).
We don't bundle a contrib icon theme. The default Font Awesome 4.7.0 is at
. More details about the contrib version at
. We haven't upgraded to 5.0 because they added paid variants and reduce
the number of available icons from 750+ to 450+, so kind of a downgrade.
The Platform is using the IconTheme component
where is committed with
the UI.
Everything that is in Contrib related to colors/icons is optional /
experimental. It's the same also for color theme. I'm trying to put them in
contrib in order to not lose the work. Sometimes they are the results of
investigations, or they are problematic and don't match our support
criteria, but we might use them in the future or see if they are
interesting enough to be bundled/integrated.
Thanks Caty
-Vincent
PS: I just want to make sure we follow some practices, hence all my
questions.
Thanks,
Caty
>
> Thanks
> -Vincent
>
>> I'm going to create the repo and commit the themes.
>> Thanks for your feedback,
>> Caty
>>
>>
>>>
>>> Generally my main points are:
>>>
>>> * It doesn’t matter that we bundle lots of themes in XS by default
>>> (provided they’re of good-enough quality ofc)
>>>
>>> * If we want themes to be bundled in XS they need to be moved to
>>> xwiki-platform (ie we should stop bundling contrib extensions as much
as
>>> possible - see previous thread for
arguments. BTW on this topic, I
feel
> we
>>> need to start a new discussion thread to decide what we do for the
>>> currently bundled contrib extensions in XS)
>>>
>>> Thanks
>>> -Vincent
>>>
>>>> On 16 Mar 2018, at 11:51, Ecaterina Moraru (Valica) <
valicac(a)gmail.com
>>
>>> wrote:
>>>>
>>>> So Iceberg was committed in Platform in
>>>>
http://jira.xwiki.org/browse/XWIKI-15108
>>>>
>>>> I would want to know what to do with the other proposals.
>>>>
>>>> Thanks,
>>>> Caty
>>>>
>>>> On Tue, Mar 13, 2018 at 6:24 PM, Ecaterina Moraru (Valica) <
>>>> valicac(a)gmail.com> wrote:
>>>>
>>>>> Ok, so after more investigations:
>>>>> - We have problems with Dawn and Pantera on IE11, see
>>>>>
https://jira.xwiki.org/browse/XWIKI-15045 (we would need someone to
>>>>> investigate this issue and see if it can be fixed). Since we are
still
>>>>> supporting IE11, in the
current version these themes are not bundle
>>>>> material since we still support IE11. They were more experimental
> color
>>>>> themes, since they rely heavily on transparency.
>>>>> - Mandarin and Snowdrop work on IE11, on the other hand they
didn't
>>>>> receive any vote on
https://forum.xwiki.org/t/
>>> refresh-the-default-color-
>>>>> theme-for-xwiki-10-x/2677 . I would not want to bundle themes that
are
>>>>> not interesting / wanted;
>>>>> - Cotton Candy as I said, doesn't look great on XS.
>>>>> - Iceberg was voted and will replace the default, so it will be
> bundled
>>> /
>>>>> committed inside Platform.
>>>>>
>>>>> I still think the Contrib is the place place for these kind of
themes.
>>>>>
>>>>> Thanks,
>>>>> Caty
>>>>>
>>>>> On Mon, Mar 12, 2018 at 7:18 PM, Ecaterina Moraru (Valica) <
>>>>> valicac(a)gmail.com> wrote:
>>>>>
>>>>>> There are several discussion in this thread: if the themes we
want
to
>>>>>> bundle should be in
Platform or Contrib, if we should bundle the
> other
>>>>>> themes that were alternatives to the default Iceberg, and having
a
>>> place to
>>>>>> commit themes inside Contrib.
>>>>>>
>>>>>> For example the Cotton Candy theme does not look good with XS,
but
> is a
>>>>>> theme that could be used by some Flavor. It should be committed
>>> somewhere
>>>>>> on Contrib.
>>>>>>
>>>>>> Then even if we commit themes in Platform, I would not put them
in
>>>>>> xwiki-platform-flamingo-theme-ui, they would need their separate
>>> module,
>>>>>> so xwiki-platform-flamingo-theme-dawn,
> xwiki-platform-flamingo-theme-
>>> snowdrop,
>>>>>> etc. Let's say we mark them as optional modules, so they
could be
>>>>>> uninstalled, but it's a shame they could be installed only
for
10.2+
>>>>>>
>>>>>> Thanks,
>>>>>> Caty
>>>>>>
>>>>>> On Mon, Mar 12, 2018 at 6:56 PM, Vincent Massol <
vincent(a)massol.net>
>>>>>> wrote:
>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>> On 12 Mar 2018, at 17:52, Vincent Massol
<vincent(a)massol.net>
> wrote:
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>>> On 12 Mar 2018, at 17:38, Ecaterina Moraru (Valica)
<
>>>>>>> valicac(a)gmail.com> wrote:
>>>>>>>>>
>>>>>>>>> On Mon, Mar 12, 2018 at 6:09 PM, Vincent Massol <
> vincent(a)massol.net
>>>>
>>>>>>> wrote:
>>>>>>>>>
>>>>>>>>>> Hi Caty,
>>>>>>>>>>
>>>>>>>>>>> On 12 Mar 2018, at 16:50, Ecaterina Moraru
(Valica) <
>>>>>>> valicac(a)gmail.com>
>>>>>>>>>> wrote:
>>>>>>>>>>>
>>>>>>>>>>> Hello devs,
>>>>>>>>>>>
>>>>>>>>>>> I want to publish additional Color Themes
inside the Contrib
>>>>>>>>>> organisation.
>>>>>>>>>>> These themes will be complementary to the
>>>>>>> xwiki-platform-flamingo-themes
>>>>>>>>>>> [1] module, and in the future we could move
optional/deprecated
>>>>>>> themes
>>>>>>>>>> from
>>>>>>>>>>> platform there (for example Kitty, Marina,
etc).
>>>>>>>>>>>
>>>>>>>>>>> I will want to contribute the Dawn
(color-theme-dawn),
Mandarin
>>>>>>>>>>>
(color-theme-mandarin), Pantera (color-theme-pantera) and
> Snowdrop
>>>>>>>>>>> (color-theme-snowdrop) color themes.
>>>>>>>>>>
>>>>>>>>>> What is the rationale for not having those themes
bundled by
>>> default
>>>>>>> in XS
>>>>>>>>>> and committed along with the other color themes
in
> xwiki-platform?
>>> I
>>>>>>> feel
>>>>>>>>>> it would be much simpler for users and as you
said it’s small.
So
>>>>>>> why not
>>>>>>>>>> make it the simplest possible for users and not
have them to
find
>>>>>>> them out
>>>>>>>>>> randomly on e.x.o and have to install the
extension?
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>> Currently there are 4 themes in
xwiki-platform-flamingo-theme-
ui.
>>>>>>> They
>>>>>>>>> don't have individual modules, so there is no way
to specify
their
>>>>>>>>> dependencies.
Some need Open Sans font, others do not. That's
why
>>>>>>> first we
>>>>>>>>> would need to provide individual modules for each
theme in order
> to
>>>>>>>>> correctly define them.
>>>>>>>>> Yes, we could do that in Platform, but why? On
Contrib, I can
> define
>>>>>>> the
>>>>>>>>> Platform dependency to be XWiki 6.2, not 10.2, since
there is
>>> nothing
>>>>>>>>> dependent on 10.2 in them and multiple users might
use them.
>>>>>>>>> Also those 4 themes IMO should be moved outside
Platform, or at
>>> least
>>>>>>> in
>>>>>>>>> their own modules and not being in the UI anymore.
This would
help
>>>>>>> knowing
>>>>>>>>> which theme is used / wanted.
>>>>>>>>>
>>>>>>>>> I understand the new default Iceberg has sense to
have a 10.2
>>>>>>> dependency,
>>>>>>>>> and that's why this is committed in Platform see
>>>>>>>>>
https://github.com/xwiki/xwiki-platform/pull/714
>>>>>>>>> but I don't see why we would block the new themes
to this
version.
>>>>>>>>>
>>>>>>>>> So the answer is modularity, dependencies and
platform version.
> More
>>>>>>>>> details in the related thread [xwiki-devs] Color
Themes
Questions
>>>>>>>>>
http://markmail.org/message/v75q2klsouu72mo7
>>>>>>>>
>>>>>>>> Modularity has a very high cost. Since it means needing
to
release
>>>>>>> modules before we can
bundle them. We’ve done some exceptions so
far
>>> (Tour
>>>>>>> extension, CKEditor, etc) but I’m personally very against
continuing
>>> in
>>>>>>> this direction. Anything that should be bundled by default in
XS
>>> should
>>>>>>> come from the xwiki github org and be released with the same
> version.
>>>>>>>>
>>>>>>>> There’s a reason why we stopped doing this years ago
after trying
> it!
>>>>>>> It’s a major PITA. It means:
>>>>>>>> * creating complex release plans
>>>>>>>> * having to release external modules before we can
release XS
>>>>>>>> * having to test all variations
>>>>>>>> * lots of complexities such as: no single release notes
or
complex
> to
>>>>>>> do release notes to find out and list all external changes in
the
XS
>>>>>>> release notes
>>>>>>>
>>>>>>> So the only valid option for me if you want them in contrib
is to
>>> decide
>>>>>>> that we’ll never want to bundle them in XS. I find that a bit
of a
>>> pity and
>>>>>>> I liked that the were proposing several color themes by
default to
> our
>>>>>>> users.
>>>>>>>
>>>>>>> Thanks
>>>>>>> -Vincent
>>>>>>>
>>>>>>>> Thanks
>>>>>>>> -Vincent
>>>>>>>>
>>>>>>>>> Thanks,
>>>>>>>>> Caty
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> Thanks
>>>>>>>>>> -Vincent
>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> I would need:
>>>>>>>>>>> - a repository on xwiki-contrib called
"color-themes"
>>>>>>>>>>>
https://github.com/xwiki-contrib/color-themes/
>>>>>>>>>>> - a JIRA project called
"COLORTHEMES"
>>>>>>>>>>>
https://jira.xwiki.org/browse/COLORTHEMES/ I
will use
separate
>>>>>>>>>>
Components
>>>>>>>>>>> for each theme
>>>>>>>>>>> - username: evalica
>>>>>>>>>>>
>>>>>>>>>>> A related mail thread is [xwiki-devs] Color
Themes Questions
>>>>>>>>>>>
http://markmail.org/message/v75q2klsouu72mo7
>>>>>>>>>>> I prefer having the themes grouped on
Contrib, but as
individual
>>>>>>> modules,
>>>>>>>>>>> because the themes are related and small
enough; while needing
>>>>>>> individual
>>>>>>>>>>> dependencies, active installs count and
platform version
>>>>>>> independence.
>>>>>>>>>>>
>>>>>>>>>>> Thanks,
>>>>>>>>>>> Caty
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> [1]
>>>>>>>>>>>
https://github.com/xwiki/xwiki-platform/tree/master/
>>>>>>>>>>
xwiki-platform-core/xwiki-platform-flamingo/xwiki-platform-f
>>>>>>> lamingo-themes