World: r3wp

(Ashley had sent me accout details, by the way.)

How can we now make a change to rebgui.r? Do you have published the 
build-script that creates rebgui.r?

I tried:
REBOL Preprocessor 2.0.0
Copyright 2002 REBOL Technologies

Input file? rebgui-ctx.r
Output file? rebgui.r
***ERROR (widgets/button.r): Cannot load file: button.r
** Press enter to quit...

Using Ladislav's INCLUDE I get: 
>>  include/link %rebgui-ctx.r %rebgui.r

** Access Error: Cannot open /D/Download/rebol/rebgui/widgets/widgets/anim.r
** Where: include-script
** Near: found: load/all found
if header

How about changing drop-list in that it opens the list even when 
the user clicks on the field and not only on the arrow?

How can I use the RebGUI LAYOUT function? I just need to get back 
the face that I can append to a PANE. I get the following error:

>> ctx-rebgui/layout material/gui
** Script Error: color needs a value
** Where: append-widget
** Near: color: any [attribute-color color] image: any

The last one seems to happen if COMMENT is used inside a spec block.

I don't think you can use comments inside vid blocks either.

to get the face created by the rebgui 'display function, I changed 
all the returns to say, "return view-face" in display.r

Do you have published the build-script that creates rebgui.r?

REBOL []

;	combine source scripts
do/args %prerebol.r [%rebgui-ctx.r %tmp.r]
;	remove header
save/header %tmp.r load %tmp.r []
;	remove indentation
do/args %prerebol.r [%tmp.r %rebgui.r]
delete %tmp.r
;	remove newlines and surplus spaces
gui: trim/lines read %rebgui.r
;	compact block delimiters
replace/all gui "[ " "["
replace/all gui " ]" "]"
replace/all gui " [" "["
replace/all gui "] " "]"
;	compact expression delimiters
replace/all gui "( " "("
replace/all gui " )" ")"
;	final write
write %rebgui.r gui

to get the face created by the rebgui 'display function ...

The last line of the display function is:

	view-face

so it should already do this without change.

so, it does .. I must be using an older version.

Anton, unfortunately your patch to the action of the slider gives 
me this error:

* Script Error: max expected value1 argument of type: number pair 
char money date time tuple series
** Where: action
** Near: system/view/caret:
offset-to-caret parent-face

Hang on, I'm just about to publish what I have, which is working. 
 In Tortoise, do I use the Import... menu option or SVN Checkout 
?

Isn't it commit ?

Checkout is to grab a copy of the repository.

So, I think you just select the files you want to commit to the repository 
and commit them.

the SVN book can be downloaded from http://svnbook.red-bean.com/

Sorry, yes, that's ok, I just read some Tortoise help.

I need to "check out", then "commit" changes, and tell others to 
"update".

So just now I committed my changes.

The build script and tour.r should be placed in the repository as 
well.

Yes, once you do an "SVN Checkout" the folders and files are displayed 
with a green tick. Editing a file causes it (and the folder it appears 
in) to be displayed with a red changed icon. Right clicking the root 
folder, RebGUI in my case, and selecting "SVN Commit" prompts for 
which changed scripts (and a comment) you wish to save back. Pretty 
straight forward once you've done it.

Yes, pretty straightforward after I read the help :) I just needed 
to be clear on the definitions of import/export/check out.

Ashley, I didn't make a history entry in rebgui-widgets.r for my 
changes, which I have commented next to the changes - would you prefer 
I did make a history entry ?

The change comment (which appears under Last Change when doing a 
"Browse Source") is the more important thing.

Not sure whether putting %tour.r up is a good thing as it has dependencies 
on images, dat files and / or dictionaries that don't really belong 
in SVN. I sort of wanted the repository to only contain code needed 
to produce %rebgui.r, while the %get-rebgui.r process delivers an 
"end-user" bundle with %tour.r, demos, etc ready to run (i.e. the 
repository is for developers/documentors only).


Likewise the build script has dependencies on prebol.r and prerebol.r 
so I'll need to extract the bits it uses before I put it up. Any 
ideas on what this script should be called? build.r, merge.r, make.r, 
compile.r, other?

build.r is good.

tour.r is needed for quick testing.

Unfortunately I don't see these changes fixing the problem, and it 
seems to introduce odd cursor movements when moving at boundaries 
( vertical).

It's not perfect, but is it not better than what we had before?

Could you restate the problem with the cursor as you saw it ?

note: the cursor confining code in area.r requires the patch to rebgui-widgets.r 
to work properly.

I've got the patch that sets the indent to 0.

Ok...

when the cursor reaches the bottom of the area, it suddenly then 
jumps up about a third of the way of the area on the same line.

ok let me try that.  Just using the scroller ? arrows or drag bar 
?

just the cursor key

down arrow ..

then as you continue to move down, you don't.. the screen moves up 
instead.

just down arrow, ok trying that.

well, combination of both

Ah yes, that's a separate problem.

but the original problem is also there.

curiously scrolling upwards is fine.

Cursor movement causes scrolling unnecessarily. It should only scroll 
when the cursor moves beyond the top or bottom borders.

exactly what I was about to say

oh! I just re-read your original problem...

Were you attempting to fix another problem? ;)