Closed Bug 882036 Opened 6 years ago Closed 4 years ago

Port GTK2 to GTK3 - SeaMonkey Part.

Categories

(SeaMonkey :: Build Config, defect)

x86
Linux
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.39

People

(Reporter: philip.chee, Assigned: ewong)

References

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #627699 +++

GTK+ 3.0 and GNOME 3 are approaching and we should get SeaMonkey ready for them.

See Also:
Bug 877626 - Port GTK2 to GTK3 - build config
Bug 879760 - Port GTK2 to GTK3 - browser dir
See Also: → 879760, 877626
Summary: Port GTK2 to GTK3 → Port GTK2 to GTK3 - SeaMonkey Part.
Yes, I guess we may file such bug for Thunderbird too and I can provide such patches. But the recent core state is still a bit raw, there are bug in menu/widget rendering which has to be fixed first.
Assignee: nobody → stransky
See Also: → 884634
Once this is implemented, will SM be backwards compatible with GTK2 systems?  I use Debian and their release cycle runs every two years.  Thanks.
What do you mean with "backwards compatible with GTK2 systems"? Do you mean profiles, extensions, or?
I guess I used the wrong term, sorry.  I just took a look at Synaptic and it shows both GTK2 and GTK3 installed.
Depends on: 943407
No longer depends on: 943407
Depends on: 1034064
No longer depends on: 1034064
See Also: → 1043146
GTK3 is now built by default and there are some issues. At least the following two changesets need to be ported to SM:

http://hg.mozilla.org/mozilla-central/rev/0cb01583448b
http://hg.mozilla.org/mozilla-central/rev/8b27e157b268
Depends on: 1183351
Assignee: stransky → ewong
Status: NEW → ASSIGNED
Attachment #8638240 - Flags: review?(kairo)
Depends on: 1186782
(In reply to Edmund Wong (:ewong) from comment #6)
> Created attachment 8638240 [details] [diff] [review]
> proposed patch(v1)

There is a combined TB, SM and MailNews patch over in bug 1186782
(In reply to Ian Neal from comment #7)
> (In reply to Edmund Wong (:ewong) from comment #6)
> > Created attachment 8638240 [details] [diff] [review]
> > proposed patch(v1)
> 
> There is a combined TB, SM and MailNews patch over in bug 1186782

So I guess this is a WF?
Comment on attachment 8638240 [details] [diff] [review]
proposed patch(v1)

Review of attachment 8638240 [details] [diff] [review]:
-----------------------------------------------------------------

r=me, sorry it took a few days.
Attachment #8638240 - Flags: review?(kairo) → review+
Fixed by
http://hg.mozilla.org/comm-central/rev/ae95dbce7c48
http://hg.mozilla.org/comm-central/rev/28bda87ffa1f
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.39
You need to log in before you can comment on or make changes to this bug.