Bug 29531 - JSVGCanvas API has minor inconsistency on property changes
Summary: JSVGCanvas API has minor inconsistency on property changes
Status: NEW
Alias: None
Product: Batik - Now in Jira
Classification: Unclassified
Component: SVGGraphics2D (show other bugs)
Version: 1.5
Hardware: All other
: P3 enhancement
Target Milestone: ---
Assignee: Batik Developer's Mailing list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-06-11 21:32 UTC by Amanda Ge
Modified: 2011-02-11 08:46 UTC (History)
0 users


Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Amanda Ge 2004-06-11 21:32:29 UTC 
JSVGCanvas overrides JComponent's add/remove PropertyChangeListners, but not 
firePropertyChange(). So users of JSVGCanvas might accidentally use JSVGCanvas' 
version to add change listeners, and JComponent's version of firePropertyChange
() doesn't notify. It's nicer to fully wrap a superclass's "feature" to reduce 
confusion.
Comment 1 Thomas Deweese 2005-03-22 12:16:37 UTC 
Reassigning all open bugs to the development list.
Sorry for the mass mailing.
Comment 2 Adrien CLERC 2011-02-11 06:28:50 UTC 
Please use ALL JComponent add/remove PropertyChangeListners. Using the actual methods, all basic Swing property changes are hidden, such as background. This is quite annoying.
Comment 3 Helder Magalhães 2011-02-11 08:46:09 UTC 
(In reply to comment #2)
> Please use ALL JComponent add/remove PropertyChangeListners. Using the actual
> methods, all basic Swing property changes are hidden, such as background. This
> is quite annoying.

Does this still make sense in trunk version? Can you confirm at least in the latest release (1.7)? This issue is reported against version 1.5...

Also, patches are always welcome!