Re: Notice of browser and toolkit theme renamings

2013-02-26 Thread Marco Bonardo

On 26/02/2013 12:58, Mounir Lamouri wrote:

On 23/02/13 07:25, Jared Wein wrote:



I wanted to let you all know about bug 842913, of which I just landed the patch 
for on mozilla-inbound.




Is themes/linux/ agnostic of the library being used (aka GTK vs QT)?

--
Mounir


See bug 844597 (https://bugzilla.mozilla.org/show_bug.cgi?id=844597).
___
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform


Re: Notice of browser and toolkit theme renamings

2013-02-26 Thread Mounir Lamouri
On 23/02/13 07:25, Jared Wein wrote:
> Hi,
> 
> I wanted to let you all know about bug 842913, of which I just landed the 
> patch for on mozilla-inbound.
> 
> The goal of the bug is to make the process of working with the /browser and 
> /toolkit themes easier for new contributors. The previous names of 
> {winstripe,pinstripe,gnomestripe} were never exposed in the product and were 
> unclear to new users as to which platforms/environments they targeted.
> 
> In toolkit:
> /toolkit/themes/winstripe/ becomes /toolkit/themes/windows/
> /toolkit/themes/pinstripe/ -> /toolkit/themes/osx/
> /toolkit/themes/gnomestripe/ -> /toolkit/themes/linux/
> /toolkit/themes/pmstripe/ -> /toolkit/themes/os2/
> /toolkit/themes/faststripe/ -> (unchanged)
> 
> In browser:
> /browser/themes/winstripe/ becomes /browser/themes/windows/
> /browser/themes/pinstripe/ -> /browser/themes/osx/
> /browser/themes/gnomestripe/ -> /browser/themes/linux/

Is themes/linux/ agnostic of the library being used (aka GTK vs QT)?

--
Mounir
___
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform


Notice of browser and toolkit theme renamings

2013-02-22 Thread Jared Wein
Hi,

I wanted to let you all know about bug 842913, of which I just landed the patch 
for on mozilla-inbound.

The goal of the bug is to make the process of working with the /browser and 
/toolkit themes easier for new contributors. The previous names of 
{winstripe,pinstripe,gnomestripe} were never exposed in the product and were 
unclear to new users as to which platforms/environments they targeted.

In toolkit:
/toolkit/themes/winstripe/ becomes /toolkit/themes/windows/
/toolkit/themes/pinstripe/ -> /toolkit/themes/osx/
/toolkit/themes/gnomestripe/ -> /toolkit/themes/linux/
/toolkit/themes/pmstripe/ -> /toolkit/themes/os2/
/toolkit/themes/faststripe/ -> (unchanged)

In browser:
/browser/themes/winstripe/ becomes /browser/themes/windows/
/browser/themes/pinstripe/ -> /browser/themes/osx/
/browser/themes/gnomestripe/ -> /browser/themes/linux/

Most of the work to complete the patches is based on |hg rename|, but some 
files that reference paths needed to be updated manually. This may affect 
SeaMonkey, Thunderbird, and any other toolkit-based applications.

This work builds on top of bug 838244 (shared theme resources for /browser), 
and upcoming work in bug 844412 (shared theme resources for /toolkit). These 
three bugs should help to make our theme implementations easier and more 
intuitive for both experienced and inexperienced contributors.

One issue that is still remaining is the non-obvious way that the linux-toolkit 
theme is generated using the windows-toolkit theme as a base. The work in bug 
844412 should help to reduce some of this confusion. Overall, there was a 
consensus that the net-wins of this rename are worth it even though it doesn't 
solve the theme-derivation situation.

Cheers,
Jared
___
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform