AltME: R3-GUI

Messages

GrahamC
@DideC .. I've mentioned this typo on the 22nd May, but now we have the sources on github it seems the appropriate way is to fork, fix and make a pull request ...
Cyphre
Marco, since R3 GUI uses own resizing algorithm you should read the documentation first to get the idea. Things like face/size: XxY won't work anymore for you. (unless you "force" to turn the resizing off for certain face(s) or implement own "custom" layout style that is not based on PANEL/GROUP containers)
To lcarify my answer: R3GUI can ofcourse use absolute positioning or any other resizing algorithm possible. The resizing is not "hardcoded" in the R3GUI core code. Anyone can implement own Layout style that treats the contained faces in any way.
Cyphre
Graham: true, the type fix probably slipped during some code merges so yes, we are on GitHub nows corrections are welcome and won't be forgotten ;)
type=typo
GrahamC
@Cyphre I submitted a pull request 2 days ago https://github.com/saphirion/r3-gui/pulls in case you didn't see it.

DideC
I have just submited my first Github "pull request" ever.
PeterWood
Great ... hopefully the first of many :-)
Arnold
Congratulations! Wouldn't that have to be a Push request? (No just kidding!) ;-)
GrahamC
@Cyphre - did you already fix this bug I mentioned here on the 26-May with the leading === ?  That was the reason for the pull request because it looked as though you hadn't ?
Cyphre
GrahamC: see the pull-request chat for answer
(According to the MDP docs the leading "===" is not valid syntax....the first line should be just plain text title.)
Cyphre
Didec: thanks, your request has been merged. Since we are syncing between SVN<->Git now it looks the commits are not fully-credited to the contributor in the log file. I need to discuss this with Andreas.
We'll also put small list of suggestions how to do the pull-requests efficiently into the main README file. This way contributors can easily follow the 'pattern'.
GrahamC
So, this example of yours in all-styles.r3
doc {
            ===DOC style
            Paragraph text.
            
            <b>Bold text</b><i>Italic text</i>
            <em>Emphasized text</em>
        }
is incorrect ??
and this example from Carl http://www.rebol.com/r3/demo.r
instructions: {
    ===GUI Prototype
    This is a <b>development test version</b> of the GUI. It has not
    been finalized because we have a bit more to do.
    
    Note that the skin you are seeing is the development them,
    <i>not the final skin</i>.
is also incorrect ?
Cyphre
yes, since we decided to use the MDP format for the DOC style
And according to the MDP docs http://www.robertmuench.ch/development/projects/mdp/documentation/#toc2 the title line is not optional.
The all-styles.r3 is most probably outdated (and that's why is not on the GitHub as well)
...and examples from Carl are usualy outdated as well when using with current R3GUI
Cyphre
BTW it looks even if we still use the Carl's MakeDoc syntax only your fix would break it anyway. (The MakeDoc syntax also defines the first plain text title line, though probably it is optional)

Last message posted 270 weeks ago.