From 1bb48ec9ea527b3bef19cf89f82a7429cbca09d7 Mon Sep 17 00:00:00 2001 From: Keith Lazuka Date: Wed, 9 Sep 2009 10:44:41 -0400 Subject: [PATCH 01/10] Changed Browser's "Up:" links to a traditional breadcrumbs list. Tweaked a few colors in the Factor UI. --- basis/help/help.factor | 16 +++++++++------- basis/help/markup/markup.factor | 8 +++++++- basis/help/stylesheet/stylesheet.factor | 5 +++-- basis/io/styles/styles-docs.factor | 7 +++++++ basis/io/styles/styles.factor | 3 +++ 5 files changed, 29 insertions(+), 10 deletions(-) diff --git a/basis/help/help.factor b/basis/help/help.factor index 6e09e298f4..c7e7f225dc 100644 --- a/basis/help/help.factor +++ b/basis/help/help.factor @@ -101,19 +101,21 @@ M: word set-article-parent swap "help-parent" set-word-prop ; : $navigation-table ( topic -- ) [ - [ help-path [ \ $links "Up:" $navigation-row ] unless-empty ] [ prev-article [ 1array \ $long-link "Prev:" $navigation-row ] when* ] [ next-article [ 1array \ $long-link "Next:" $navigation-row ] when* ] - tri + bi ] { } make [ $table ] unless-empty ; +: ($navigation) ( topic -- ) + help-path-style get [ + [ help-path [ reverse $breadcrumbs ] unless-empty ] + [ $navigation-table ] bi + ] with-style ; + : $title ( topic -- ) title-style get [ - title-style get [ - [ ($title) ] - [ help-path-style get [ $navigation-table ] with-style ] bi - ] with-nesting - ] with-style nl ; + [ ($title) ] [ ($navigation) ] bi + ] with-nested-style nl ; : print-topic ( topic -- ) >link diff --git a/basis/help/markup/markup.factor b/basis/help/markup/markup.factor index 6e75adc8aa..c64f315d6d 100644 --- a/basis/help/markup/markup.factor +++ b/basis/help/markup/markup.factor @@ -205,8 +205,11 @@ ALIAS: $slot $snippet "Vocabulary" $heading nl dup ($vocab-link) ] when* ; +: (textual-list) ( seq quot sep -- ) + '[ _ print-element ] swap interleave ; inline + : textual-list ( seq quot -- ) - [ ", " print-element ] swap interleave ; inline + ", " (textual-list) ; inline : $links ( topics -- ) [ [ ($link) ] textual-list ] ($span) ; @@ -214,6 +217,9 @@ ALIAS: $slot $snippet : $vocab-links ( vocabs -- ) [ vocab ] map $links ; +: $breadcrumbs ( topics -- ) + [ [ ($link) ] " > " (textual-list) ] ($span) ; + : $see-also ( topics -- ) "See also" $heading $links ; diff --git a/basis/help/stylesheet/stylesheet.factor b/basis/help/stylesheet/stylesheet.factor index 6c0b18e8e9..7f7975a652 100644 --- a/basis/help/stylesheet/stylesheet.factor +++ b/basis/help/stylesheet/stylesheet.factor @@ -30,10 +30,10 @@ H{ { font-style bold } } strong-style set-global SYMBOL: title-style H{ { font-name "sans-serif" } - { font-size 18 } + { font-size 20 } { font-style bold } { wrap-margin 500 } - { foreground COLOR: FactorDarkSlateBlue } + { foreground COLOR: gray20 } { page-color COLOR: FactorLightTan } { border-width 5 } } title-style set-global @@ -46,6 +46,7 @@ H{ { font-name "sans-serif" } { font-size 16 } { font-style bold } + { foreground COLOR: FactorDarkSlateBlue } } heading-style set-global SYMBOL: subsection-style diff --git a/basis/io/styles/styles-docs.factor b/basis/io/styles/styles-docs.factor index 8fcf12aae9..a952c05dbf 100755 --- a/basis/io/styles/styles-docs.factor +++ b/basis/io/styles/styles-docs.factor @@ -90,6 +90,13 @@ HELP: with-style { $notes "Details are in the documentation for " { $link make-span-stream } "." } $io-error ; +HELP: with-nested-style +{ $values { "style" assoc } { "quot" quotation } } +{ $description "Calls the quotation in a new dynamic scope where calls to " { $link write } ", " { $link format } " and other stream output words automatically inherit style settings from " { $snippet "style" } "." } +$nl +"This word is intended to be used when you have a single style assoc that contains both character and paragraph styles." +$io-error ; + ARTICLE: "formatted-stream-protocol" "Formatted stream protocol" "The " { $vocab-link "io.styles" } " vocabulary defines a protocol for output streams that support rich text." { $subsection stream-format } diff --git a/basis/io/styles/styles.factor b/basis/io/styles/styles.factor index 2d25016919..108d4c9eb0 100644 --- a/basis/io/styles/styles.factor +++ b/basis/io/styles/styles.factor @@ -42,6 +42,9 @@ make-cell-stream stream-write-table ; [ output-stream get make-block-stream ] dip with-output-stream ; inline +: with-nested-style ( style quot -- ) + over [ with-nesting ] with-style ; inline + TUPLE: filter-writer stream ; CONSULT: output-stream-protocol filter-writer stream>> ; From 28b8703b5bb61b97a84ac3299f6b79612bd75e3e Mon Sep 17 00:00:00 2001 From: Keith Lazuka Date: Wed, 9 Sep 2009 13:33:40 -0400 Subject: [PATCH 02/10] help.html: link color now matches help.stylesheet link color --- basis/help/html/stylesheet.css | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/basis/help/html/stylesheet.css b/basis/help/html/stylesheet.css index ff657d634e..c56a19bc9a 100644 --- a/basis/help/html/stylesheet.css +++ b/basis/help/html/stylesheet.css @@ -1,4 +1,4 @@ -a:link { text-decoration: none; color: #00004c; } -a:visited { text-decoration: none; color: #00004c; } -a:active { text-decoration: none; color: #00004c; } -a:hover { text-decoration: underline; color: #00004c; } +a:link { text-decoration: none; color: #104e8b; } +a:visited { text-decoration: none; color: #104e8b; } +a:active { text-decoration: none; color: #104e8b; } +a:hover { text-decoration: underline; color: #104e8b; } From 5421f4fcb488b885393651c8f00dcc2c91ce2760 Mon Sep 17 00:00:00 2001 From: Keith Lazuka Date: Wed, 9 Sep 2009 15:27:49 -0400 Subject: [PATCH 03/10] prettyprint.stylesheet: USING/USE/IN are now dimmed out. Added documentation. --- .../stylesheet/stylesheet-docs.factor | 44 +++++++++++++++++++ .../prettyprint/stylesheet/stylesheet.factor | 28 +++++++++--- basis/prettyprint/stylesheet/summary.txt | 1 + 3 files changed, 67 insertions(+), 6 deletions(-) create mode 100644 basis/prettyprint/stylesheet/stylesheet-docs.factor create mode 100644 basis/prettyprint/stylesheet/summary.txt diff --git a/basis/prettyprint/stylesheet/stylesheet-docs.factor b/basis/prettyprint/stylesheet/stylesheet-docs.factor new file mode 100644 index 0000000000..4f7a7f2141 --- /dev/null +++ b/basis/prettyprint/stylesheet/stylesheet-docs.factor @@ -0,0 +1,44 @@ +! Copyright (C) 2009 Your name. +! See http://factorcode.org/license.txt for BSD license. +USING: help.markup help.syntax kernel words ; +IN: prettyprint.stylesheet + +HELP: effect-style +{ $values + { "effect" "an effect" } + { "style" "a style assoc" } +} +{ $description "The styling hook for stack effects" } ; + +HELP: string-style +{ $values + { "str" "a string" } + { "style" "a style assoc" } +} +{ $description "The styling hook for string literals" } ; + +HELP: vocab-style +{ $values + { "vocab" "a vocabulary specifier" } + { "style" "a style assoc" } +} +{ $description "The styling hook for vocab names" } ; + +HELP: word-style +{ $values + { "word" "a word" } + { "style" "a style assoc" } +} +{ $description "The styling hook for word names" } ; + +ARTICLE: "prettyprint.stylesheet" "Prettyprinter Formatted Output" +{ $vocab-link "prettyprint.stylesheet" } +$nl +"Control the way that the prettyprinter formats output based on object type. These hooks form a basic \"syntax\" highlighting system." +{ $subsection word-style } +{ $subsection string-style } +{ $subsection vocab-style } +{ $subsection effect-style } +; + +ABOUT: "prettyprint.stylesheet" diff --git a/basis/prettyprint/stylesheet/stylesheet.factor b/basis/prettyprint/stylesheet/stylesheet.factor index 2be959cc9b..eaed7cba0c 100644 --- a/basis/prettyprint/stylesheet/stylesheet.factor +++ b/basis/prettyprint/stylesheet/stylesheet.factor @@ -1,16 +1,32 @@ ! Copyright (C) 2009 Your name. ! See http://factorcode.org/license.txt for BSD license. -USING: colors.constants hashtables io.styles kernel namespaces -words words.symbol ; +USING: colors.constants combinators combinators.short-circuit +hashtables io.styles kernel namespaces sequences words +words.symbol ; IN: prettyprint.stylesheet + + : word-style ( word -- style ) dup "word-style" word-prop >hashtable [ [ [ presented set ] [ - [ parsing-word? ] [ delimiter? ] [ symbol? ] tri - or or [ COLOR: DarkSlateGray ] [ COLOR: black ] if - foreground set + { + { [ dup parsing-word? ] [ parsing-word-color ] } + { [ dup delimiter? ] [ drop COLOR: DarkSlateGray ] } + { [ dup symbol? ] [ drop COLOR: DarkSlateGray ] } + [ drop COLOR: black ] + } cond foreground set ] bi ] bind ] keep ; @@ -24,7 +40,7 @@ IN: prettyprint.stylesheet : vocab-style ( vocab -- style ) [ presented set - COLOR: cornsilk4 foreground set + dim-color foreground set ] H{ } make-assoc ; : effect-style ( effect -- style ) diff --git a/basis/prettyprint/stylesheet/summary.txt b/basis/prettyprint/stylesheet/summary.txt new file mode 100644 index 0000000000..39a50c8d9f --- /dev/null +++ b/basis/prettyprint/stylesheet/summary.txt @@ -0,0 +1 @@ +prettyprinter syntax highlighting and formatted output From a96e828d7cde5050b6a3b287ff906d655c24d1b8 Mon Sep 17 00:00:00 2001 From: Keith Lazuka Date: Wed, 9 Sep 2009 15:50:25 -0400 Subject: [PATCH 04/10] io.styles: renamed the 'border-width' style to 'inset'. inset takes a pair of 2 numbers (horizontal and vertical insets) --- basis/help/stylesheet/stylesheet.factor | 8 ++++---- basis/help/tips/tips.factor | 2 +- basis/html/streams/streams.factor | 5 +++-- basis/io/styles/styles-docs.factor | 8 ++++---- basis/io/styles/styles.factor | 2 +- basis/ui/gadgets/panes/panes.factor | 6 +++--- 6 files changed, 16 insertions(+), 15 deletions(-) diff --git a/basis/help/stylesheet/stylesheet.factor b/basis/help/stylesheet/stylesheet.factor index 7f7975a652..c2e8e98476 100644 --- a/basis/help/stylesheet/stylesheet.factor +++ b/basis/help/stylesheet/stylesheet.factor @@ -35,7 +35,7 @@ H{ { wrap-margin 500 } { foreground COLOR: gray20 } { page-color COLOR: FactorLightTan } - { border-width 5 } + { inset { 5 5 } } } title-style set-global SYMBOL: help-path-style @@ -72,7 +72,7 @@ H{ SYMBOL: code-style H{ { page-color COLOR: FactorLightTan } - { border-width 5 } + { inset { 5 5 } } { wrap-margin f } } code-style set-global @@ -89,7 +89,7 @@ SYMBOL: warning-style H{ { page-color COLOR: gray90 } { border-color COLOR: red } - { border-width 5 } + { inset { 5 5 } } { wrap-margin 500 } } warning-style set-global @@ -97,7 +97,7 @@ SYMBOL: deprecated-style H{ { page-color COLOR: gray90 } { border-color COLOR: red } - { border-width 5 } + { inset { 5 5 } } { wrap-margin 500 } } deprecated-style set-global diff --git a/basis/help/tips/tips.factor b/basis/help/tips/tips.factor index 4685b6c517..8569be0b8f 100644 --- a/basis/help/tips/tips.factor +++ b/basis/help/tips/tips.factor @@ -30,7 +30,7 @@ SYMBOL: tip-of-the-day-style H{ { page-color COLOR: lavender } - { border-width 5 } + { inset { 5 5 } } { wrap-margin 500 } } tip-of-the-day-style set-global diff --git a/basis/html/streams/streams.factor b/basis/html/streams/streams.factor index 49a9225402..26a3d5f391 100644 --- a/basis/html/streams/streams.factor +++ b/basis/html/streams/streams.factor @@ -99,7 +99,8 @@ M: html-span-stream dispose : border-css, ( border -- ) "border: 1px solid #" % hex-color, "; " % ; -: padding-css, ( padding -- ) "padding: " % # "px; " % ; +: padding-css, ( padding -- ) + first2 "padding: " % # "px " % # "px; " % ; CONSTANT: pre-css "white-space: pre; font-family: monospace;" @@ -108,7 +109,7 @@ CONSTANT: pre-css "white-space: pre; font-family: monospace;" { { page-color bg-css, } { border-color border-css, } - { border-width padding-css, } + { inset padding-css, } } make-css ] [ wrap-margin swap at diff --git a/basis/io/styles/styles-docs.factor b/basis/io/styles/styles-docs.factor index a952c05dbf..19952b2b15 100755 --- a/basis/io/styles/styles-docs.factor +++ b/basis/io/styles/styles-docs.factor @@ -140,7 +140,7 @@ ARTICLE: "paragraph-styles" "Paragraph styles" "Paragraph styles for " { $link with-nesting } ":" { $subsection page-color } { $subsection border-color } -{ $subsection border-width } +{ $subsection inset } { $subsection wrap-margin } { $subsection presented } ; @@ -250,10 +250,10 @@ HELP: border-color { $code "H{ { border-color T{ rgba f 1 0 0 1 } } }\n[ \"A border\" write ] with-nesting nl" } } ; -HELP: border-width -{ $description "Paragraph style. Pixels between edge of text and border, an integer." } +HELP: inset +{ $description "Paragraph style. A pair of integers representing the number of pixels that the content should be inset from the border. The first number is the horizontal inset, and the second is the vertical inset." } { $examples - { $code "H{ { border-width 10 } }\n[ \"Some inset text\" write ] with-nesting nl" } + { $code "H{ { inset { 10 10 } } }\n[ \"Some inset text\" write ] with-nesting nl" } } ; HELP: wrap-margin diff --git a/basis/io/styles/styles.factor b/basis/io/styles/styles.factor index 108d4c9eb0..a3a1d991f3 100644 --- a/basis/io/styles/styles.factor +++ b/basis/io/styles/styles.factor @@ -135,7 +135,7 @@ SYMBOL: image ! Paragraph styles SYMBOL: page-color SYMBOL: border-color -SYMBOL: border-width +SYMBOL: inset SYMBOL: wrap-margin ! Table styles diff --git a/basis/ui/gadgets/panes/panes.factor b/basis/ui/gadgets/panes/panes.factor index 6f68c32ff0..50a609b897 100644 --- a/basis/ui/gadgets/panes/panes.factor +++ b/basis/ui/gadgets/panes/panes.factor @@ -242,11 +242,11 @@ MEMO: specified-font ( assoc -- font ) : apply-page-color-style ( style gadget -- style gadget ) page-color [ >>interior ] apply-style ; -: apply-border-width-style ( style gadget -- style gadget ) - border-width [ dup 2array ] apply-style ; +: apply-inset-style ( style gadget -- style gadget ) + inset [ ] apply-style ; : style-pane ( style pane -- pane ) - apply-border-width-style + apply-inset-style apply-border-color-style apply-page-color-style apply-presentation-style From 51f40e27271b0e8c2b9cc66e644ddd6737fa1e99 Mon Sep 17 00:00:00 2001 From: Keith Lazuka Date: Wed, 9 Sep 2009 16:33:40 -0400 Subject: [PATCH 05/10] vocabs.prettyprint: made the auto-use vocab box a little easier to read --- basis/colors/constants/factor-colors.txt | 1 + basis/vocabs/prettyprint/prettyprint.factor | 5 ++++- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/basis/colors/constants/factor-colors.txt b/basis/colors/constants/factor-colors.txt index 9d7649ab3d..c032aae5c4 100644 --- a/basis/colors/constants/factor-colors.txt +++ b/basis/colors/constants/factor-colors.txt @@ -1,4 +1,5 @@ ! Factor UI theme colors +243 242 234 FactorLightLightTan 227 226 219 FactorLightTan 172 167 147 FactorDarkTan 81 91 105 FactorLightSlateBlue diff --git a/basis/vocabs/prettyprint/prettyprint.factor b/basis/vocabs/prettyprint/prettyprint.factor index 2813485da3..3c5059b5c0 100644 --- a/basis/vocabs/prettyprint/prettyprint.factor +++ b/basis/vocabs/prettyprint/prettyprint.factor @@ -4,6 +4,7 @@ USING: accessors arrays assocs colors colors.constants fry io io.styles kernel make math.order namespaces parser prettyprint.backend prettyprint.sections prettyprint.stylesheet sequences sets sorting vocabs vocabs.parser ; +FROM: io.styles => inset ; IN: vocabs.prettyprint : pprint-vocab ( vocab -- ) @@ -86,7 +87,9 @@ PRIVATE> "To avoid doing this in the future, add the following forms" print "at the top of the source file:" print nl ] with-style - { { page-color COLOR: FactorLightTan } } + { { page-color COLOR: FactorLightLightTan } + { border-color COLOR: FactorDarkTan } + { inset { 5 5 } } } [ manifest get pprint-manifest ] with-nesting nl nl ] print-use-hook set-global \ No newline at end of file From 6fdd005b5c0a19960bf06ab2779007569203fd94 Mon Sep 17 00:00:00 2001 From: Keith Lazuka Date: Wed, 9 Sep 2009 16:46:05 -0400 Subject: [PATCH 06/10] prettyprint.stylesheet: fixed an error found by help-lint --- basis/prettyprint/stylesheet/stylesheet.factor | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/basis/prettyprint/stylesheet/stylesheet.factor b/basis/prettyprint/stylesheet/stylesheet.factor index eaed7cba0c..f04893fab3 100644 --- a/basis/prettyprint/stylesheet/stylesheet.factor +++ b/basis/prettyprint/stylesheet/stylesheet.factor @@ -31,7 +31,7 @@ PRIVATE> ] bind ] keep ; -: string-style ( obj -- style ) +: string-style ( str -- style ) [ presented set COLOR: LightSalmon4 foreground set From e1979f5ad55aac4e043f801122e4bf144f739b99 Mon Sep 17 00:00:00 2001 From: Keith Lazuka Date: Thu, 10 Sep 2009 09:48:20 -0400 Subject: [PATCH 07/10] help browser: tweaked the colors based on user feedback. The USING: line should be easier to read now. I also decided to stop highlighting symbols. --- basis/help/help.factor | 5 +++- basis/help/stylesheet/stylesheet.factor | 12 ++++++---- .../prettyprint/stylesheet/stylesheet.factor | 23 ++++++++++--------- 3 files changed, 24 insertions(+), 16 deletions(-) diff --git a/basis/help/help.factor b/basis/help/help.factor index c7e7f225dc..214ff14632 100644 --- a/basis/help/help.factor +++ b/basis/help/help.factor @@ -99,12 +99,15 @@ M: word set-article-parent swap "help-parent" set-word-prop ; : $navigation-row ( content element label -- ) [ prefix 1array ] dip prefix , ; +: ($navigation-table) ( element -- ) + help-path-style get table-style set [ $table ] with-scope ; + : $navigation-table ( topic -- ) [ [ prev-article [ 1array \ $long-link "Prev:" $navigation-row ] when* ] [ next-article [ 1array \ $long-link "Next:" $navigation-row ] when* ] bi - ] { } make [ $table ] unless-empty ; + ] { } make [ ($navigation-table) ] unless-empty ; : ($navigation) ( topic -- ) help-path-style get [ diff --git a/basis/help/stylesheet/stylesheet.factor b/basis/help/stylesheet/stylesheet.factor index c2e8e98476..2475fba0f6 100644 --- a/basis/help/stylesheet/stylesheet.factor +++ b/basis/help/stylesheet/stylesheet.factor @@ -1,6 +1,6 @@ ! Copyright (C) 2005, 2009 Slava Pestov. ! See http://factorcode.org/license.txt for BSD license. -USING: io.styles namespaces colors colors.constants ; +USING: colors colors.constants io.styles literals namespaces ; IN: help.stylesheet SYMBOL: default-span-style @@ -34,12 +34,16 @@ H{ { font-style bold } { wrap-margin 500 } { foreground COLOR: gray20 } - { page-color COLOR: FactorLightTan } + { page-color COLOR: FactorLightLightTan } { inset { 5 5 } } } title-style set-global SYMBOL: help-path-style -H{ { font-size 10 } } help-path-style set-global +H{ + { font-size 10 } + { table-gap { 5 5 } } + { table-border $ transparent } +} help-path-style set-global SYMBOL: heading-style H{ @@ -71,7 +75,7 @@ H{ SYMBOL: code-style H{ - { page-color COLOR: FactorLightTan } + { page-color COLOR: FactorLightLightTan } { inset { 5 5 } } { wrap-margin f } } code-style set-global diff --git a/basis/prettyprint/stylesheet/stylesheet.factor b/basis/prettyprint/stylesheet/stylesheet.factor index f04893fab3..fbd95ecbd2 100644 --- a/basis/prettyprint/stylesheet/stylesheet.factor +++ b/basis/prettyprint/stylesheet/stylesheet.factor @@ -7,27 +7,28 @@ IN: prettyprint.stylesheet : word-style ( word -- style ) dup "word-style" word-prop >hashtable [ [ - [ presented set ] [ - { - { [ dup parsing-word? ] [ parsing-word-color ] } - { [ dup delimiter? ] [ drop COLOR: DarkSlateGray ] } - { [ dup symbol? ] [ drop COLOR: DarkSlateGray ] } - [ drop COLOR: black ] - } cond foreground set - ] bi + [ presented set ] [ word-color foreground set ] bi ] bind ] keep ; From 65e9c29a7bcd04b4cd3510805ded6feb363424c3 Mon Sep 17 00:00:00 2001 From: Keith Lazuka Date: Fri, 11 Sep 2009 20:35:56 -0400 Subject: [PATCH 08/10] io.styles: bailed out on the with-nested-styles combinator --- basis/help/help.factor | 6 ++++-- basis/io/styles/styles-docs.factor | 7 ------- basis/io/styles/styles.factor | 3 --- 3 files changed, 4 insertions(+), 12 deletions(-) diff --git a/basis/help/help.factor b/basis/help/help.factor index 214ff14632..e31c705e26 100644 --- a/basis/help/help.factor +++ b/basis/help/help.factor @@ -117,8 +117,10 @@ M: word set-article-parent swap "help-parent" set-word-prop ; : $title ( topic -- ) title-style get [ - [ ($title) ] [ ($navigation) ] bi - ] with-nested-style nl ; + title-style get [ + [ ($title) ] [ ($navigation) ] bi + ] with-nesting + ] with-style nl ; : print-topic ( topic -- ) >link diff --git a/basis/io/styles/styles-docs.factor b/basis/io/styles/styles-docs.factor index 19952b2b15..8d29cffb04 100755 --- a/basis/io/styles/styles-docs.factor +++ b/basis/io/styles/styles-docs.factor @@ -90,13 +90,6 @@ HELP: with-style { $notes "Details are in the documentation for " { $link make-span-stream } "." } $io-error ; -HELP: with-nested-style -{ $values { "style" assoc } { "quot" quotation } } -{ $description "Calls the quotation in a new dynamic scope where calls to " { $link write } ", " { $link format } " and other stream output words automatically inherit style settings from " { $snippet "style" } "." } -$nl -"This word is intended to be used when you have a single style assoc that contains both character and paragraph styles." -$io-error ; - ARTICLE: "formatted-stream-protocol" "Formatted stream protocol" "The " { $vocab-link "io.styles" } " vocabulary defines a protocol for output streams that support rich text." { $subsection stream-format } diff --git a/basis/io/styles/styles.factor b/basis/io/styles/styles.factor index a3a1d991f3..b141d8d2f7 100644 --- a/basis/io/styles/styles.factor +++ b/basis/io/styles/styles.factor @@ -42,9 +42,6 @@ make-cell-stream stream-write-table ; [ output-stream get make-block-stream ] dip with-output-stream ; inline -: with-nested-style ( style quot -- ) - over [ with-nesting ] with-style ; inline - TUPLE: filter-writer stream ; CONSULT: output-stream-protocol filter-writer stream>> ; From 0e176d8de7027985592bf9d2349dfc0199f3d8a2 Mon Sep 17 00:00:00 2001 From: Keith Lazuka Date: Fri, 11 Sep 2009 21:45:03 -0400 Subject: [PATCH 09/10] prettyprint.stylesheet: more idiomatic Factor style (thanks Slava) --- .../prettyprint/stylesheet/stylesheet.factor | 63 +++++++++---------- 1 file changed, 29 insertions(+), 34 deletions(-) diff --git a/basis/prettyprint/stylesheet/stylesheet.factor b/basis/prettyprint/stylesheet/stylesheet.factor index fbd95ecbd2..a593f23d99 100644 --- a/basis/prettyprint/stylesheet/stylesheet.factor +++ b/basis/prettyprint/stylesheet/stylesheet.factor @@ -1,51 +1,46 @@ -! Copyright (C) 2009 Your name. +! Copyright (C) 2009 Keith Lazuka. ! See http://factorcode.org/license.txt for BSD license. -USING: colors.constants combinators combinators.short-circuit -hashtables io.styles kernel namespaces sequences words -words.symbol ; +USING: assocs colors.constants combinators +combinators.short-circuit hashtables io.styles kernel literals +namespaces sequences words words.symbol ; IN: prettyprint.stylesheet -: word-style ( word -- style ) - dup "word-style" word-prop >hashtable [ - [ - [ presented set ] [ word-color foreground set ] bi - ] bind - ] keep ; +GENERIC: word-style ( word -- style ) + +M: word word-style + [ presented associate ] + [ "word-style" word-prop >hashtable ] bi assoc-union ; + +M: highlighted-word word-style + call-next-method COLOR: DarkSlateGray foreground associate + swap assoc-union ; + + : string-style ( str -- style ) - [ - presented set - COLOR: LightSalmon4 foreground set - ] H{ } make-assoc ; + COLOR: LightSalmon4 colored-presentation-style ; : vocab-style ( vocab -- style ) - [ - presented set - dim-color foreground set - ] H{ } make-assoc ; + dim-color colored-presentation-style ; : effect-style ( effect -- style ) - [ - presented set - COLOR: DarkGreen foreground set - ] H{ } make-assoc ; \ No newline at end of file + COLOR: DarkGreen colored-presentation-style ; From 12446adfc248ad263ef621cfed00444cf305d1e6 Mon Sep 17 00:00:00 2001 From: Keith Lazuka Date: Fri, 11 Sep 2009 21:46:16 -0400 Subject: [PATCH 10/10] vocabs.prettyprint: fixed indentation --- basis/vocabs/prettyprint/prettyprint.factor | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/basis/vocabs/prettyprint/prettyprint.factor b/basis/vocabs/prettyprint/prettyprint.factor index 3c5059b5c0..6b759dddde 100644 --- a/basis/vocabs/prettyprint/prettyprint.factor +++ b/basis/vocabs/prettyprint/prettyprint.factor @@ -87,9 +87,10 @@ PRIVATE> "To avoid doing this in the future, add the following forms" print "at the top of the source file:" print nl ] with-style - { { page-color COLOR: FactorLightLightTan } - { border-color COLOR: FactorDarkTan } - { inset { 5 5 } } } - [ manifest get pprint-manifest ] with-nesting + { + { page-color COLOR: FactorLightLightTan } + { border-color COLOR: FactorDarkTan } + { inset { 5 5 } } + } [ manifest get pprint-manifest ] with-nesting nl nl ] print-use-hook set-global \ No newline at end of file