I've read your comments on the issue and was actually in the process of writing 
to you when I received your message.
Let me clarify a critically important point.

Currently, to style segment text using CSS style classes, we use this method:
RichParagraph.Builder.addWithStyleNames(String text, String... css) - ONE 
METHOD FOR ALL STYLE CLASSES.

If I understand correctly (and I might be wrong!), you're suggesting using a 
DIFFERENT method for background color styling:
RichParagraph.Builder#addHighlight(int start, int end, String... styles).

Could you please confirm whether RichParagraph.Builder.addWithStyleNames(String 
text, String... css) can also be used for background coloring?

For example, we know that user wants to add style classes "foo" and "bar" to a 
text from from 5 to 20. We do not know IF THESE STYLES
SET BACKGROUND COLOR. Or should we ask an user to name a class this way - 
"foo-bg"?

Best regards, Pavel

On 5/7/25 21:16, Andy Goryachev wrote:

Thank /you/ for the feedback!

I finally got the ticket - https://bugs.openjdk.org/browse/JDK-8356436 .  It's 
a dup of https://bugs.openjdk.org/browse/JDK-8355774 , essentially.  Once the 
missing APIs are added, you'll be able to style the highlights with CSS:

RichParagraph.Builder#addHighlight(int start, int length, String ... css)

-andy

P.S. In some of your code, you do css.toArray() for each paragraph.  I wonder 
if you should just create a bunch of styles statically instead - like a 
document stylesheet, since these names do not change at all.

*From: *openjfx-dev <openjfx-dev-r...@openjdk.org> on behalf of PavelTurk 
<pavelturk2...@gmail.com>
*Date: *Wednesday, May 7, 2025 at 10:30
*To: *openjfx-dev@openjdk.org <openjfx-dev@openjdk.org>
*Subject: *Re: CodeArea: -fx-background-color doesn't work.

Andy, thank you very much for your help!

Best regards, Pavel

On 5/7/25 20:29, Andy Goryachev wrote:

    This bug is still percolating through the system.  I'll respond once I get 
it.

    Thank you for all the questions and the feedback!

    -andy

    *From: *openjfx-dev <openjfx-dev-r...@openjdk.org> 
<mailto:openjfx-dev-r...@openjdk.org> on behalf of PavelTurk <pavelturk2...@gmail.com> 
<mailto:pavelturk2...@gmail.com>
    *Date: *Monday, May 5, 2025 at 14:13
    *To: *openjfx-dev@openjdk.org <openjfx-dev@openjdk.org> 
<mailto:openjfx-dev@openjdk.org>
    *Subject: *Re: CodeArea: -fx-background-color doesn't work.

    No, I am afraid you misunderstood me. I am not talking about compatibility 
with a third party library.
    The "compatibility" already exists and its name is CSS. So, the only thing 
I need is to support
    '-fx-background-color' for RichParagraph segment.

    I've opened an issue today, so, when it is approved there won't be any 
questions.

    Best regards, Pavel

    On 5/6/25 00:04, Andy Goryachev wrote:

        Will not happen.  CSS compatibility with a third party library is not a 
requirement in this case.

        -andy

        *From: *openjfx-dev <openjfx-dev-r...@openjdk.org> 
<mailto:openjfx-dev-r...@openjdk.org> on behalf of PavelTurk <pavelturk2...@gmail.com> 
<mailto:pavelturk2...@gmail.com>
        *Date: *Monday, May 5, 2025 at 13:59
        *To: *openjfx-dev@openjdk.org <openjfx-dev@openjdk.org> 
<mailto:openjfx-dev@openjdk.org>
        *Subject: *Re: CodeArea: -fx-background-color doesn't work.

        To be as clear as possible, here's what I need:

        .test {
        -fx-set-somehow-background-color: green;
        }

        var builder = RichParagraph.builder();
        builder.addWithStyleNames(segment, "test");

        because, in parallel for RTFX I simply use its stylesheet:

        .test {
             -rtfx-background-color: green;
        }


        So, for both code areas, I use same style classes for styling paragraph 
segments.

        Best regards, Pavel

        On 5/5/25 23:20, Andy Goryachev wrote:

            Right, the API are missing currently.  I just point out the fact 
that you must use highlights for the purpose of highlighting text range(s) 
within the paragraph, since these ranges might contain semi-transparent Nodes.  
Once JDK-8355774 is fixed you'll be able to style highlights with CSS and all 
will be well. Right?

            -andy

            *From: *openjfx-dev <openjfx-dev-r...@openjdk.org> 
<mailto:openjfx-dev-r...@openjdk.org> on behalf of PavelTurk <pavelturk2...@gmail.com> 
<mailto:pavelturk2...@gmail.com>
            *Date: *Monday, May 5, 2025 at 13:10
            *To: *openjfx-dev@openjdk.org <openjfx-dev@openjdk.org> 
<mailto:openjfx-dev@openjdk.org>
            *Subject: *Re: CodeArea: -fx-background-color doesn't work.

            Andy, thank you very much for your help.

            That doesn’t work for me. As I mentioned before, I’m working with 
two CodeArea implementations - JFX and RTFX,
            and CSS is the common styling mechanism. I can’t use code for one 
and CSS for the other.

            Well, technically I could, but it just wouldn’t make any sense.

            Best regards, Pavel

            On 5/5/25 22:54, Andy Goryachev wrote:

                You have to use highlights for that, since you can't assign CSS 
style to a part of the Text instance, and you can't set a background on it.

                JDK-8355774 is pretty high on my list of things to do.

                -andy

                *From: *openjfx-dev <openjfx-dev-r...@openjdk.org> 
<mailto:openjfx-dev-r...@openjdk.org> on behalf of PavelTurk <pavelturk2...@gmail.com> 
<mailto:pavelturk2...@gmail.com>
                *Date: *Monday, May 5, 2025 at 12:47
                *To: *openjfx-dev@openjdk.org <openjfx-dev@openjdk.org> 
<mailto:openjfx-dev@openjdk.org>
                *Subject: *Re: CodeArea: -fx-background-color doesn't work.

                Hello, Andy

                Yes, I need to add a background color for a text segment within 
a paragraph using CSS.

                I have just opened an issue with ID : 9078469.

                Can anyone give an estimate of when this issue might be 
addressed? Background color is such a fundamental feature
                that it's impossible to work without it.

                Best regards, Pavel

                On 5/5/25 22:09, Andy Goryachev wrote:

                    Dear Pavel:

                    Can you clarify what you are trying to do exactly?

                    If you are trying to add a background to a text segment 
within the paragraph (as your code seem to indicate), then the only way to do 
it is to call Builder.addHighlight().

                    If you are trying to set the background of the whole 
paragraph, you've hit another missing API similar to JDK-8355774 - there is 
currently no way to style the paragraphs via CSS. There is the 
Builder.setParagraphAttributes(StyleAttributeMap) but a CSS one is missing.

                    -andy

                    *From: *openjfx-dev <openjfx-dev-r...@openjdk.org> 
<mailto:openjfx-dev-r...@openjdk.org> on behalf of PavelTurk <pavelturk2...@gmail.com> 
<mailto:pavelturk2...@gmail.com>
                    *Date: *Saturday, May 3, 2025 at 10:07
                    *To: *openjfx-dev@openjdk.org <openjfx-dev@openjdk.org> 
<mailto:openjfx-dev@openjdk.org>
                    *Subject: *CodeArea: -fx-background-color doesn't work.

                    For styling CodeArea, I use exclusively style classes, and 
this is the foundation of my entire architecture.
                    Today I tried to implement search highlighting (via 
background color), but it didn't work. Below is my test code.

                    Can anyone tell me how to set the background color using 
CSS? For example, in RichTextFX's CodeArea,
                    they have -rtfx-background-color.

                    public class JfxCodeArea extends Application {

                         @Override
                         public void start(Stage primaryStage) throws Exception 
{
                             String text = """
                                           Lorem ipsum dolor sit amet, 
consectetur adipiscing elit, sed do eiusmod tempor incididunt
                                           ut labore et dolore magna aliqua. Ut 
enim ad minim veniam, quis nostrud exercitation ullamco
                                           laboris nisi ut aliquip ex ea 
commodo consequat. Duis aute irure dolor in reprehenderit
                                           in voluptate velit esse cillum 
dolore eu fugiat nulla pariatur.
                                           """;

                             String css = """
                                 .test {
                                     -fx-font-weight: bold;
                                     -fx-fill: red;
                                     -fx-background-color: green;
                                 }

                             """;
                             String data = "data:text/css;base64," + 
Base64.getEncoder().encodeToString(css.getBytes(StandardCharsets.UTF_8));

                             CodeArea codeArea = new CodeArea();
                             codeArea.getStylesheets().add(data);

                             codeArea.setSyntaxDecorator(new SyntaxDecorator() {
                                 @Override
                                 public RichParagraph 
createRichParagraph(CodeTextModel model, int index) {
                                     var builder = RichParagraph.builder();
                                     builder.addWithStyleNames(model.getPlainText(index), 
"test");
                                     return builder.build();
                                 }

                                 @Override
                                 public void handleChange(CodeTextModel m, 
TextPos start, TextPos end, int charsTop, int linesAdded, int charsBottom) {

                                 }
                             });

                             VBox.setVgrow(codeArea, Priority.ALWAYS);
                             var button = new Button("Go!");
                             button.setOnAction(e -> codeArea.setText(text));
                             VBox root = new VBox(codeArea, button);
                             Scene scene = new Scene(root, 600, 200);
                             primaryStage.setScene(scene);
                             primaryStage.show();

                         }

                         public static void main(String[] args) {
                             launch(args);
                         }
                    }

                    Best regards, Pavel

Reply via email to