[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-07-24 Thread Sebastien Bacher
The issue is deprecated in the karmic version ** Changed in: gdm (Ubuntu) Status: Confirmed = Fix Released -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme https://bugs.launchpad.net/bugs/331445 You received this bug notification

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-03-13 Thread Rick Spencer
** Changed in: gdm (Ubuntu) Importance: Undecided = Medium -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme https://bugs.launchpad.net/bugs/331445 You received this bug notification because you are a member of Ubuntu Bugs, which is

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-03-08 Thread lunch
I noticed that the Mist gtkrc is applied correctly if I remove the symbolic colours and that Kenneth's gtkrc isn't if I add symbolic colours to it. If bug #337106 was fixed, this problem would probably be solved, too. -- gdm does not respect all the given color variables in a gtkrc which is

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-03-03 Thread lunch
This fixes Kenneth's theme. Howerver I still get the error when adding the Mist gtkrc to the HumanCircle GDM theme. The radio buttons in the session menu are Murrine-style, not Mist like they should, and have Human colours. ** Attachment added: gdm-humancircle-mist.png

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-03-03 Thread Martin Pitt
** Changed in: gdm (Ubuntu) Assignee: Martin Pitt (pitti) = Cody Russell (bratsche) -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme https://bugs.launchpad.net/bugs/331445 You received this bug notification because you are a member of

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-03-03 Thread Cody Russell
Where did you put the Mist gtkrc? Try putting it in /usr/share/gdm/themes/HumanCircle/gtk-2.0 -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme https://bugs.launchpad.net/bugs/331445 You received this bug notification because you are a member

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-03-03 Thread lunch
I've put it exactly there. The other widgets are drawn by the Mist engine. -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme https://bugs.launchpad.net/bugs/331445 You received this bug notification because you are a member of Ubuntu Bugs,

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-03-03 Thread Cody Russell
Where would it be pulling in Murrine engine from? Is there another theme that is loading that first? If that's the case then there's nothing we can do about it from the gdm code because GTK can only load a single engine module at a time. -- gdm does not respect all the given color variables in

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-03-03 Thread lunch
Well, it's also using colours from the Human GTK theme for symbolic colours: https://launchpad.net/bugs/337106 How does it get them? Human uses the murrine engine - if I replace the Human theme in /usr/share/themes/Human with Human-Clearlooks, the session menu has Clearlooks radio buttons. --

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-03-03 Thread Kenneth Wimer
It is possible to include references to two different gtk engines within one gtkrc. It is one of those things you don't want to do (for several reasons) but it is possible. -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-03-03 Thread Kenneth Wimer
The current GDM should be including colors from the gtkrc within the theme itself (ie the dark colors, which are included in /usr/share/gdm/themes/Human/gtk-2.0/gtkrc). -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-03-03 Thread lunch
Kenneth: Please replace /usr/share/gdm/themes/Human/gtk-2.0/gtkrc with /usr/share/themes/Mist/gtk-2.0/gtkrc for testing. I get colours and radio buttons from /usr/share/themes/Human/gtk-2.0/gtkrc then, but the other widgets are drawn by the Mist engine. The Mist gtkrc only references the Mist

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-27 Thread Martin Pitt
** Changed in: gdm (Ubuntu) Status: New = Fix Committed -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme https://bugs.launchpad.net/bugs/331445 You received this bug notification because you are a member of Ubuntu Bugs, which is

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-27 Thread Launchpad Bug Tracker
This bug was fixed in the package gdm - 2.20.8-0ubuntu10 --- gdm (2.20.8-0ubuntu10) jaunty; urgency=low * 72_read_theme_gtkrc.patch: Take three, now confirmed to fully fix the issues. (LP: #331445) -- Martin Pitt martin.p...@ubuntu.com Fri, 27 Feb 2009 10:30:47 +0100 **

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-26 Thread Martin Pitt
Cody, does this patch replace the one in comment 14, or does it need to be done in addition? ** Changed in: gdm (Ubuntu) Status: In Progress = Incomplete -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-26 Thread Cody Russell
Sorry for the confusion. It should just replace the previous patch. ** Changed in: gdm (Ubuntu) Status: Incomplete = New -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme https://bugs.launchpad.net/bugs/331445 You received this bug

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-25 Thread Launchpad Bug Tracker
This bug was fixed in the package gdm - 2.20.8-0ubuntu9 --- gdm (2.20.8-0ubuntu9) jaunty; urgency=low * 72_read_theme_gtkrc.patch: Check to see if you have a gtkrc property first, and if not it falls back to using {themedir}/gtk-2.0/gtkrc. Patch by Cody Russell. (LP:

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-25 Thread Kenneth Wimer
Erm, still no luck. I still get exactly the same behavior as before :( -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme https://bugs.launchpad.net/bugs/331445 You received this bug notification because you are a member of Ubuntu Bugs, which

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-25 Thread Cody Russell
Ken sat down with me at the office and showed me the exact behavior, so together we managed to fix this for good. :) For some reason gdm_session_init() is blowing away some of our theming information. Investigating why looks like it will take significant time, and Ken says we're planning to move

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-25 Thread Martin Pitt
Thanks. Please reopen bugs if they aren't fixed, so that they don't fall off the radar. I'll sponsor this after the alpha-5 release today, assinging back to me. ** Changed in: gdm (Ubuntu) Assignee: Cody Russell (bratsche) = Martin Pitt (pitti) Status: Fix Released = In Progress --

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-24 Thread Martin Pitt
** Changed in: gdm (Ubuntu) Status: Fix Released = In Progress -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme https://bugs.launchpad.net/bugs/331445 You received this bug notification because you are a member of Ubuntu Bugs, which is

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-24 Thread Cody Russell
Sorry about the last patch. I normally have a 'write-file-functions 'delete-trailing-whitespace hook in my .emacs so it fixed every line that had extra whitespace. :) ** Attachment added: Cleaner patch http://launchpadlibrarian.net/23046979/gdm-greeter-gtkrc-2.diff -- gdm does not respect

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-24 Thread Cody Russell
Tested this today and I think it works. Can you verify? ** Attachment added: Fixed patch http://launchpadlibrarian.net/23049660/gdm-greeter-gtkrc-3.diff -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-24 Thread Martin Pitt
I'll sponsor this, looks straightforward (I haven't tested it myself, though). This should not be an issue for gdm 2.24, thus no need to forward this upstream. -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-24 Thread Martin Pitt
** Changed in: gdm (Ubuntu) Status: Triaged = Fix Committed -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme https://bugs.launchpad.net/bugs/331445 You received this bug notification because you are a member of Ubuntu Bugs, which is

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-24 Thread Launchpad Bug Tracker
This bug was fixed in the package gdm - 2.20.8-0ubuntu8 --- gdm (2.20.8-0ubuntu8) jaunty; urgency=low * Add 72_read_theme_gtkrc.patch: Respect color variables in a theme's gtkrc. Thanks to Cody Russell for the patch! (LP: #331445) * 35_gdm.conf.patch: Change default

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-24 Thread Cody Russell
This checks to see if you have a gtkrc property first, and if not it falls back to using {themedir}/gtk-2.0/gtkrc ** Attachment added: New patch http://launchpadlibrarian.net/23052913/gdm-greeter-gtkrc-4.diff -- gdm does not respect all the given color variables in a gtkrc which is included

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-24 Thread Cody Russell
I happened to notice what appears to be a memory leak in nearby code. This patch should fix it. ** Attachment added: Fix for memory leak http://launchpadlibrarian.net/23053388/memory-leak.diff -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-24 Thread Cody Russell
kwwii, can you test this one? :) ** Attachment added: gtkrc-or-default.diff http://launchpadlibrarian.net/23059496/gtkrc-or-default.diff -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme https://bugs.launchpad.net/bugs/331445 You received

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-23 Thread Cody Russell
What's the easiest way for me to reproduce this bug? I've never really messed around with themes in gdm before so it's not clear what I should do to include the gtkrc. -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-23 Thread Kenneth Wimer
If you install the latest gdm theme (ubuntu-gdm-theme in bzr) you'll notice that although there is a gtk-2.0 dir with a valid gtkrc it is not respected correctly (the select session pop-up from the options menu does not seem to respect any theme colors). A hint here, the new gtkrc colors have a

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-23 Thread Martin Pitt
Cody, Ken vanDine, can either of you work on this bug? -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme https://bugs.launchpad.net/bugs/331445 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-23 Thread Cody Russell
I have another bug I need to try to work on first, and after that I can work on this one unless Ken is working on it. Earlier doing a really quick grep through the sources I found something that is possibly relevant, but I didn't spend the time yet to be sure. From greeter_parser.c there is a

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-23 Thread Kenneth Wimer
RIght, the current behaviour is that when there is a gtk-2.0 dir with a valid gtkrc in the theme dir gdm will use that gtkrc BUT it apparently does not respect everything as it should as there are problems (only with radio box's it seems) and specifying a gtkrc from the login-window GUI fixes the

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-23 Thread Martin Pitt
Tentatively assigning to Cody then; please unassign if you have trouble with this. Thanks! ** Changed in: gdm (Ubuntu) Assignee: (unassigned) = Cody Russell (bratsche) Status: New = Triaged -- gdm does not respect all the given color variables in a gtkrc which is included with the

[Bug 331445] Re: gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme

2009-02-23 Thread Cody Russell
** Attachment added: gdm-greeter-gtkrc.diff http://launchpadlibrarian.net/23022168/gdm-greeter-gtkrc.diff -- gdm does not respect all the given color variables in a gtkrc which is included with the gdm theme https://bugs.launchpad.net/bugs/331445 You received this bug notification because