On Sun, Mar 18, 2018 at 1:09 PM, Vincent Massol <vinc...@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. It follows the naming scheme and we do similar things for skins and icon-themes, 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. 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) <vali...@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) < > > vali...@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) < > >> vali...@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 <vinc...@massol.net> > >>> wrote: > >>> > >>>> > >>>> > >>>>> On 12 Mar 2018, at 17:52, Vincent Massol <vinc...@massol.net> wrote: > >>>>> > >>>>> > >>>>> > >>>>>> On 12 Mar 2018, at 17:38, Ecaterina Moraru (Valica) < > >>>> vali...@gmail.com> wrote: > >>>>>> > >>>>>> On Mon, Mar 12, 2018 at 6:09 PM, Vincent Massol <vinc...@massol.net > > > >>>> wrote: > >>>>>> > >>>>>>> Hi Caty, > >>>>>>> > >>>>>>>> On 12 Mar 2018, at 16:50, Ecaterina Moraru (Valica) < > >>>> vali...@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 > >>>> > >>>> > >>> > >> > >