Shouldn't have a copy of Clutter

Bug #331033 reported by Bruce Cowan
4
Affects Status Importance Assigned to Milestone
Aachen
Won't Fix
Wishlist
Mirco Müller

Bug Description

It appears as if this project copies parts of clutter. It shouldn't do this.

Revision history for this message
Neil J. Patel (njpatel) wrote :

We require animations which require timelines. Clutter has a great timeline implementation but it, of course, has not been written to also work with Gtk. By taking the timeline code out of clutter and adapting it for Gtk, we don't need to re-invent the wheel.

Of course this means that we would need to track any fixes that the clutter folks have made to the timeline & alpha code, but hopefully Gtk will be getting animations soon, so we can drop EggTimeline in favour of that. If there is interest in the EggTimeline code, it may also be worth to separate it out into it's own library for other developers to use (as the need for animation in Gtk is quite large).

I'm marking this bug as invalid as you haven't provided an alternative to using EggTimeline.

Changed in aachen:
status: New → Invalid
Revision history for this message
Mirco Müller (macslow) wrote :

The new notification-daemon does use the timeline and alpha classes of clutter. Why should we not use this proven code-base? It helps speed up the development, avoids reinventing the wheel, makes use of tested code and we credit the originating upstream project. What do you think is wrong with this approach?

What is your reasoning here to discourage that?

Changed in aachen:
assignee: nobody → macslow
status: Invalid → Won't Fix
Revision history for this message
Bruce Cowan (bruce89-deactivatedaccount) wrote :

I'll try to remember to engage my brain before filing bugs in the future.

Revision history for this message
Mark Shuttleworth (sabdfl) wrote : Re: [Bug 331033] Re: Shouldn't have a copy of Clutter

More brain does == better bugs, but nevertheless, all bug reports are
appreciated!

Mirco Müller (macslow)
Changed in aachen:
importance: Undecided → Wishlist
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.