World: r3wp

ok. i read "its not ashleys code", thought its from somewhere else.

Yes, it is in the svn

just talking to Cyphre on icq - he will look into problem - he thinks 
there is overall problem with iterator in rebgui ... he tried to 
fix inside/outside drop-list parameter, so now it works, but in outside 
mode it looses hilite .... he will look into it, once he produces 
newer grid update for me ...

he also asks for eventual code example, if there is any, to being 
able to reproduce the problem ...

unless find [object! block!] type?/word get in face 'pane [      
       ; [che] -- An iterated faces may of course be tabbable, too. 
I don't handle this case for now, though.  

            return none                                                      
               ; [che]

        ]                                                                
               ; [che]

how does this code work?

looks in the pane and allows only object! and block!. on other things 
it returns none. especially on functions, thats why it uses 'get.

needs to be rewritten to be a non recursive function

or perhaps to pass the recursion count so it can exit gracefully...

its called from 'next-field too. That recurses too. looks a bit suspicious 
to me at first read. but then the error-m,essage should sometimes 
show it and not always bail out on 'into-widget IMHO.

the face-tree is recursive, so is the function.

and back-field

WHat looks suspicious is: it goes thru the parent-face. but it does 
not exclude the current face.

any quick fixes you can  suggest ?

** Internal Error: Stack overflow
** Where: into-widget
** Near: unless find [object! block!] type?/word get
** Press enter to quit...

Got it again.

it happened after I returned to the window after closing another 
window down.

Maybe

         unless wrap [siblings: find/tail siblings face]                  
                
->

  either wrap [siblings: head remove find siblings ]  [siblings: find/tail 
  siblings face]
but that is guessing

either wrap [siblings: head remove find siblings face]  [siblings: 
find/tail siblings face]

Maybe I need to make sure I set the focus when I return to the window

needs copy..

either wrap [siblings: head remove find copy siblings face]  [siblings: 
find/tail siblings face

getting closer to reproducing it

resetting focus would be worth a try. or unfocus.

5 steps to reproduce

* Internal Error: Stack overflow
** Where: tabbed?
** Near: all [find tabbed face/type face]
** Press enter to quit...

Sorry, different message now

still stack-overflow. filled by lots of 'into-widget or whatever.

Try this .. download http://www.compkarori.com/emr/synapse.exe

click on the server button.

This pops up a VID window .. enter some text and then "Accept"

Now shift tab ..and boom!!!

need to reboot windows. cu

The problem appears to be that no widget has focus

yet the shift-tab key is processed as though there is ....

Rebol []

do %rebgui.r
user: "Guest"

display "Test" compose/deep [
	field (user) return
	field return
	radio-group data [ 2 1 2 3 ] return

 button "Server" [ inform layout [ field "Type here" button "OK" [ 
 hide-popup ]]]
]
do-events

1. Highlight the first text field
2. Click on the "server" button
3. Highlight the "type here" field to give focus
4. Click on the "OK" button
the rebgui no longer has focus.
Now do tab .. and it crashes with a stack overflow.

posted to Cyphre on icq ...

Yep, reproducible.

The radio-group is redundant to the example.  You can remove it.

Oh well, I'm glad I have reproduced this annoying bug .. I deserve 
some sleep now!  past midnight.

Good work. And good night. :)

maybe some wizard will have fixed it by the time I awake :)

Cyphre will look into it now and post results here ...

;a quick fix seems to be
	process-keystroke: make function! [face event /local f] [
		switch/default event/key [
			#"^-" [
				if all[view*/focal-face viewed? view*/focal-face][

added the "viewed? view*/focal-face"

recurses in next-field with wrap: true, and the face is always the 
focused one from the closed window. but i guess cyphre found that 
too :9

not sure he looked into it, yet :-)

rebooted to early. forgot to mention its here:

        all [                                                            
               
; [che] -- Return older siblings. 
            target: next-field/wrap face

not? how long does graham sleep? :)

Graham: "For me, function is beauty"


As it is for me.  But your statement here implies that this issue 
is superfluous in light of deeper issues, which mischaracterizes 
the problem.   Form and function go hand in hand.  In a GUI (which 
literally implies both form and function) subtle information is transmitted 
in the cross roads of the form and the function.


Links, buttons, rounded buttons mean different things in different 
places.  To marginalize it may be myopic.


Anton: "I think that can't be good for people with vision problems. 
I say stick to the simple old ways."


We are actually going to offer both, we are actively working on this. 
 


.