After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 200966 - editor needs to set PersistStream dirty flag
editor needs to set PersistStream dirty flag
Status: RESOLVED FIXED
Product: GtkHtml
Classification: Other
Component: html-editor-control
unspecified
Other All
: Normal enhancement
: 1.1.0
Assigned To: Larry Ewing
Evolution QA team
Depends on:
Blocks: 200948 213256
 
 
Reported: 2000-12-07 20:38 UTC by Dan Winship
Modified: 2002-03-27 23:24 UTC
See Also:
GNOME target: ---
GNOME version: Unversioned Enhancement



Description Dan Winship 2000-12-07 20:38:53 UTC
The editor should call bonobo_persist_stream_set_dirty (ps, FALSE) after a
successful load or save. That's easy. The tricky part is setting it TRUE when
the user types/modifies something.
Comment 1 Larry Ewing 2001-05-09 07:04:20 UTC
Okay I'm going to try to get to this soon.  Making it work properly
with undo will be interesting.
Comment 2 Larry Ewing 2001-10-30 22:19:01 UTC
setting milestone to 1.0.x
Comment 3 Dan Winship 2001-11-19 16:29:52 UTC
*Actually* setting the milestone to 1.0.x this time.
Comment 4 Radek Doulik 2001-11-21 19:02:23 UTC
it could be done like creating global undo step counter and save it's
value on beginning and by each save. in undo we need to watch if
current value goes under saved one and in case we drop redo steps
under saved, dirty will be set to TRUE, otherwise dirty could be
obtained as saved != current. (dunno if it's understandable)
Comment 5 Luis Villa 2001-11-27 07:02:31 UTC
Not 1.0.x AFAICS?
Comment 6 Dan Winship 2001-11-27 14:45:00 UTC
Well, it's needed for a bugfix. (The fact that the composer
"save before closing?" dialog doesn't always guess whether
you've changed things or not correctly.)
Comment 7 Luis Villa 2001-11-27 19:47:11 UTC
OK, but is that bug a 1.0.x thing? AFAICS, both dependent bugs are 1.2.
Comment 8 Radek Doulik 2002-02-12 15:47:11 UTC
I've implemented gtk_html_is_dirty method, which is exposed also thru
Editor::Engine CORBA API, so you probably just want to use it for
PersistStream too. Evolution now uses Engine API, so if the only
reason is for evolution composer, then this bug could be probably just
closed, otherwise use this gtk_html_is_dirty method. There is
also gtk_html_drop_undo which resets dirty flag in editor and drops
undo, undo_used and redo queues.
Comment 9 Ettore Perazzoli 2002-03-27 23:24:41 UTC
Closing this as it appears to be implemented.