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.
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?
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
>>>>>
>>>>>
>>>>
>>>
>
>