[Xbae-discuss] [ xbae-Bugs-1124116 ] unmap in CommitEdit and CancelEdit

[SourceForge.net <no...@so...>]

Bugs item #1124116, was opened at 2005-02-16 19:37
Message generated for change (Comment added) made by evgeny
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=401980&aid=1124116&group_id=31337

Category: Bugs
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Evgeny Stambulchik (evgeny)
Assigned to: Nobody/Anonymous (nobody)
Summary: unmap in CommitEdit and CancelEdit

Initial Comment:
XbaeMatrixCommitEdit() and XbaeMatrixCancelEdit()
ignore unmap = True. Used to work fine in 4.7.


----------------------------------------------------------------------

>Comment By: Evgeny Stambulchik (evgeny)
Date: 2005-02-17 18:24

Message:
Logged In: YES 
user_id=27019

OK, so it enters the cell back via TraverseInTimeOut().
Zero-timeout, heh? No wonder it produces race conditions.
Notice this callback is added to ClipChild. On the other
hand, Clip has its own FocusIn() action/translation. To make
things even more funny, the primitive's translations field
is set to NULL, instead of XtInheritTranslations (why?). I
was able to solve the problem by setting
XtInheritTranslations (I recalled I did so some time ago
with my Xbae-4.7 copy; so the statement "work fine in 4.7"
is probably incorrect). And defaultTranslations could
probably include entries for pageup/down scrolling.

Overall, I got a strong feeling the traversal stuff may need
a serious cleanup.

----------------------------------------------------------------------

Comment By: Evgeny Stambulchik (evgeny)
Date: 2005-02-17 12:23

Message:
Logged In: YES 
user_id=27019

> example/cancel that was submitted with the fix for that
bug still works here.

Here, it works on one computer and doesn't on another one
(they're completely different archs/OS/...; but Xbae and
Motif are the same on both). The one where "cancel" doesn't
work exhibits all kind of funky behaviour: clicking on a
cell copies contents of another one to it etc - typical race
condition symptoms.

BTW, there is no make rule for this example.

----------------------------------------------------------------------

Comment By: Tobias Oed (tobiasoed)
Date: 2005-02-17 06:10

Message:
Logged In: YES 
user_id=101470

> Actually, it also worked a couple of days ago: #1121687.

example/cancel that was submitted with the fix for that 
bug still works here. Do you have a specific scenario 
under which it fails for you (I can actually imagine plenty. 
The whole traversal buisness still needs more work, 
maybe a whole new widget to decouple CellClip 
translation from textChild translations).
Tobias.




----------------------------------------------------------------------

Comment By: Tobias Oed (tobiasoed)
Date: 2005-02-17 06:03

Message:
Logged In: YES 
user_id=101470

> Actually, it also worked a couple of days ago: #1121687.

example/cancel that was submitted with the fix for that 
bug still works here. Do you have a specific scenario 
under which it fails for you (I can actually imagine plenty. 
The whole traversal buisness still needs more work, 
maybe a whole new widget to decouple CellClip 
translation from textChild translations).
Tobias.




----------------------------------------------------------------------

Comment By: Evgeny Stambulchik (evgeny)
Date: 2005-02-17 00:28

Message:
Logged In: YES 
user_id=27019

Actually, it also worked a couple of days ago: #1121687.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=401980&aid=1124116&group_id=31337