Community
    • Login

    Highlighting with self created words in "langs.xml" does not work

    Scheduled Pinned Locked Moved Help wanted · · · – – – · · ·
    langs.xmlhighlightingphpstylers.xml
    29 Posts 4 Posters 2.3k Views
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • Manfred DrechselM
      Manfred Drechsel @PeterJones
      last edited by

      Classic repeating without thinking from my side 😂

      1 Reply Last reply Reply Quote 0
      • EkopalypseE
        Ekopalypse @PeterJones
        last edited by

        @PeterJones

        Your understanding of how the current version of EnhanceAnyLexer works is correct :-)

        If you need help getting started with V, let me know. There are some hurdles that are not so obvious. Either message me or open an issue on github.

        As for substyles for existing lexers, hmm … without having given it much thought, I assume it can be added. Basically we just need some additional styles and their configuration and apply them when activating the buffer.
        Should be doable.

        1 Reply Last reply Reply Quote 2
        • PeterJonesP
          PeterJones @PeterJones
          last edited by

          I said earlier,

          No one has put in a feature request with Notepad++ to implement these substyles. Until someone does, it’s guaranteed to never be implemented in core Notepad++.

          I actually just put in the feature request for the main app.

          The more I thought about it, the more I thought it would work best in the main app, where the keyword list and color definitions could all go in the Style Configurator, alongside the normal keyword lists. I think I’ve figured out the places I’d need to edit, and I’ve offered to do the work and put in the PR, if Don gives his stamp of approval on the general concept.

          If he rejects the concept, I’ll start exploring other options.

          Manfred DrechselM 1 Reply Last reply Reply Quote 2
          • Manfred DrechselM
            Manfred Drechsel @PeterJones
            last edited by PeterJones

            @PeterJones

            Tried now the PythonScript solution. Could not really reliable get it working.

            First, it did not colorize on startup of NPP. Accidentally, I found out, that it starts colorizing if I open the PythonScript console. Output by the way is below. Solution after some looking around was to change the initialization type from “LAZY” (default after installation) to “ATSTARTUP”.

            Second, trying with an arbitrary PHP keyword (here CURLOPT_STDERR) does not work. No clue why.

            This is the relevant part of SubStylesForLexer.py:

                    self._style[SCE_HPHP_WORD].append(dict(fg=(0,135,68), bg=(255,255,255), keywords="pryrt_a"))
                    self._style[SCE_HPHP_WORD].append(dict(fg=(0,0,255), bg=(255,255,0), keywords="pryrt_b"))
                    self._style[SCE_HPHP_WORD].append(dict(fg=(0,135,68), bg=(255,255,255), keywords="CURLOPT_STDERR"))
            

            This is my test PHP script (results see comments):

            <?php
            pryrt_a "xyzzy";         # works as expected :-)
            pryrt_b "xyzzy";         # works as expected :-)
            $test = CURLOPT_STDERR;  # does not colorize :-\
            ?>
            

            What am I doing wrong?

            However, if the feature would be implemented nativ in NPP, that would be much better :-)
            Placed my 👍 already at GitHub :-)

            –
            Just for information, the PythonScript console output:

            Initialized SubstyleLexerInterface
            Python 2.7.18 (v2.7.18:8d21aa21f2, Apr 20 2020, 13:25:05) [MSC v.1500 64 bit (AMD64)]
            Initialisation took 15ms
            Ready.
            
            PeterJonesP 1 Reply Last reply Reply Quote 0
            • PeterJonesP
              PeterJones @Manfred Drechsel
              last edited by

              First, it did not colorize on startup of NPP. Accidentally, I found out, that it starts colorizing if I open the PythonScript console. Output by the way is below. Solution after some looking around was to change the initialization type from “LAZY” (default after installation) to “ATSTARTUP”.

              That was explained in the FAQ: How to install and run a script in PythonScript I linked you to originally, in the instructions for how to get it to run automatically. I am sorry you didn’t notice that.

              Second, trying with an arbitrary PHP keyword (here CURLOPT_STDERR) does not work. No clue why.

              This is the relevant part of SubStylesForLexer.py:

                      self._style[SCE_HPHP_WORD].append(dict(fg=(0,135,68), bg=(255,255,255), keywords="pryrt_a"))
                      self._style[SCE_HPHP_WORD].append(dict(fg=(0,0,255), bg=(255,255,0), keywords="pryrt_b"))
                      self._style[SCE_HPHP_WORD].append(dict(fg=(0,135,68), bg=(255,255,255), keywords="CURLOPT_STDERR"))
              

              What am I doing wrong?

              Apparently, the lexer used for PHP requires all the keywords to be in lowercase. To make it work, just change the case of your keyword to lowercase. (This makes it match the case it was before you removed curlopt_stderr from the list in stylers.xml, too):

              self._style[SCE_HPHP_WORD].append(dict(fg=(0,135,68), bg=(255,255,255), keywords="curlopt_stderr"))
              

              f844d19a-ad05-4cb1-884b-b3f0b95e06b2-image.png

              But as a second note: I noticed that you did two different .append() calls for the same color. Each .append() creates a new list. The intention, if you have multiple words you want the same color, is to have multiple words in the string for the same .append(), like:

                      self._style[SCE_HPHP_WORD].append(dict(fg=(0,135,68), bg=(255,255,255), keywords="pryrt_a curlopt_stderr more words go here"))
                      self._style[SCE_HPHP_WORD].append(dict(fg=(0,0,255), bg=(255,255,0), keywords="pryrt_b and other secondcolor wordish thingies"))
              

              Here’s a screenshot showing various words from those multi-word lists highlighted, proving it only needs
              c5624e0d-8cbf-4f10-84f7-45150a79125e-image.png

              However, if the feature would be implemented nativ in NPP, that would be much better :-)

              That will take a while, since I’ll just be working on it in my free time. Like you, I am not paid to develop the Notepad++ application. In fact, until this year, I had never contributed any actual code to the codebase (though I had done a couple of XML config default updates over the years, and I am heavily involved in the Notepad++ documentation). But as a code-contributor, I’m a newbie to this project, and the Notepad++ codebase is a large, complicated critter to navigate.

              Depending on how long it takes me, it might not even be done in before the next release. But I will work on it as I have time, and I will report back here when/if the PR gets merged – after that happens, it will be the version after that announcement that it makes it into the Notepad++ application.

              Apparently, I’ll have to pay attention to which lexers require the keyword lists to be in lowercase, and either document that well, or have my code fix the case for those lexers. So thanks for that heads-up.

              Alan KilbornA Manfred DrechselM 2 Replies Last reply Reply Quote 2
              • Alan KilbornA
                Alan Kilborn @PeterJones
                last edited by

                if the feature would be implemented native in NPP, that would be much better

                Actually, the end result would be the same.
                You’re probably just saying that because the script setup takes you outside your comfort zone.
                You’re lucky that the author of Notepad++ has agreed to accept changes to have it be native; this often does not happen, and add-on scripts to add features or change functionality are all one has.

                1 Reply Last reply Reply Quote 1
                • Manfred DrechselM
                  Manfred Drechsel @PeterJones
                  last edited by

                  @PeterJones
                  Can confirm now, that for me the PythonScript solution fully works. I added 1963 constant and 103 language construct keywords and special vars to SubStylesForLexer. py with different colors and all lowercase. The 1270 function keywords still are in the NPP definitions (langs.xml and stylers.xml). >Thanks a lot for your great job!<

                  So I will uninstall the EAL plugin now and stay with substyles.

                  @Alan-Kilborn
                  No, I do not have a “need native solution” comfort zone. My only argument to have a NPP solution was performance. Nothing else. I never had and have a problem with script setups or similar.

                  PeterJonesP 1 Reply Last reply Reply Quote 0
                  • PeterJonesP
                    PeterJones @Manfred Drechsel
                    last edited by

                    @Manfred-Drechsel said in Highlighting with self created words in "langs.xml" does not work:

                    for me the PythonScript solution fully works.

                    Glad to hear it. (For future readers of this discussion, I have updated the downloadable script to make the ATSTARTUP more obvious (it’s now in the comments near the top of the script, not just in the instructions-FAQ)

                    My only argument to have a NPP solution was performance. I never had and have a problem with script setups or similar

                    If you haven’t had performance problems with other scripts, you likely won’t with this one, either.

                    The only time the script comes into play is when you change from one document to another (the on_bufferactivated) – and that’s a brief number of commands that shouldn’t take a noticeable amount of time. (There might technically be a difference, but with how few commands it is, it would be on the order of a tiny fraction of a second of difference.)

                    Once the on_bufferactivated has been run, Scintilla will do the actual lexing and syntax highlighting using the code compiled into Notepad++, whether the on_bufferactivated stuff was run from a script or from native Notepad++, so the syntax highlighting will not have any performance difference.

                    Manfred DrechselM 1 Reply Last reply Reply Quote 3
                    • Manfred DrechselM
                      Manfred Drechsel @PeterJones
                      last edited by

                      @PeterJones said in Highlighting with self created words in "langs.xml" does not work:

                      If you haven’t had performance problems with other scripts, you likely won’t with this one, either.

                      My comment was wrong and misleading. For whatever reason, I had the regex way of EnhanceAnyLexer in my mind. Using substyles is native, as you already explained earlier. Sorry for the confusion…

                      Manfred DrechselM 1 Reply Last reply Reply Quote 1
                      • Manfred DrechselM
                        Manfred Drechsel @Manfred Drechsel
                        last edited by

                        @PeterJones

                        I’m sure you already know that this is possible:

                        I added the option to set my PHP constant words to bold:

                        editor.styleSetBold(subStyle, self._style[parentStyle][idx]['bold'])
                        
                        self._style[SCE_HPHP_WORD].append(dict(fg=(0,0,204), bg=(255,255,255), bold=True, keywords="..."))
                        

                        More options available here or here (search for STYLESETBOLD or STYLESETITALIC or STYLESETUNDERLINE …)

                        PeterJonesP 1 Reply Last reply Reply Quote 4
                        • PeterJonesP
                          PeterJones @Manfred Drechsel
                          last edited by

                          @Manfred-Drechsel ,

                          FYI: The developer accepted my PR, which means it will be native Notepad++ (showing up in Style Configurator) at the next release.

                          Manfred DrechselM PeterJonesP 2 Replies Last reply Reply Quote 4
                          • Manfred DrechselM
                            Manfred Drechsel @PeterJones
                            last edited by

                            @PeterJones

                            Yes, I followed the other discussions. Great job!

                            1 Reply Last reply Reply Quote 0
                            • PeterJonesP
                              PeterJones @PeterJones
                              last edited by

                              which means it will be native Notepad++ (showing up in Style Configurator) at the next release

                              It can be seen in v8.7 RC Announcement item 13.

                              The PHP keyword list has also been split from the original single huge list into a couple of huge lists instead.

                              My recommendation, if you are trying out v8.7 (whether it’s the release candidate, orthe final v8.7 in a few weeks, or if you are reading this later and updating from something before v8.7 to anything at or after v8.7) and want to be able to see the new styles in Style Configurator:

                              • If you just download the Portable version of v8.7, you can try it out without affecting your installed copy, and you will be
                              • If you install v8.7 overtop an older copy (upgrade or otherwise just run the installer), it will not update your stylers.xml, other themes, and langs.xml, so you will not see the new styles in the style configurator right away. Instead, follow one of the following sequences:
                                1. If you don’t have the PythonScript plugin and don’t want to use it, then you can manually compare the new <installed directory>\stylers.model.xml to your %AppData%\Notepad++\stylers.xml or theme file, and bring in the new stuff from the .model. into your active file, and restart. (This is mentioned in the User Manual here.)
                                2. If you have the PythonScript plugin, or are willing to install it, then I highly recommend following the instructions in “Config Files Need Updating, Too” (I suggest with PythonScript 3, but it will work with the PluginsAdmin-version of PythonScript 2 also)

                              —

                              For those who are frequent users of PythonScript, I recommend the following customization after running my script on v8.7: open stylers.xml (or your active theme’s XML), go to the “Python” section, and replace the rows for “User Keywords 1-5” (styleID 128-132, substyle1-5) with something similar to what’s in this gist. (Sorry, it’s too big to fit in a post in the forum: there are a lot of keywords)

                              After restarting Notepad++, then the Style Configurator will call the “user keyword 1-5” styles as “PythonScript xxxx”, and the user-defined list will be pre-populated with PythonScript’s built-in objects, editor-object methods, notepad-object methods, enum classes and enum elements. You might not like my choice of colors (I am not a graphic designer, after all), but it’s split up in such a way that you could make them all the same color if you want (if you don’t want to mentally distinguish those categories), or you could make the colors wildly different, if my subtle differences in color from the standard Python KEYWORD style isn’t distinct enough for you.

                              original v8.7 Python styling: 01b57f07-7118-4a64-a98d-461d20707092-image.png
                              with PythonScript keyword lists: bf3ab822-61dd-47f4-b01b-eecd84002147-image.png
                              Manfred DrechselM 1 Reply Last reply Reply Quote 2
                              • Manfred DrechselM
                                Manfred Drechsel @PeterJones
                                last edited by Manfred Drechsel

                                @PeterJones

                                Just installed a clean version of NPP V8.7 after i read your comments here ;-)
                                Tomorrow, I will insert all the keywords from my PHP V8.4.0beta5 into the style definitions and restore my colors.

                                Only a minor issue which I’ve seen is the width of the style selection list control. It should be wider to see the full text of the styles. See attachment. Guess I directly should file an issue for NPP?

                                The NPP SubStyle functionality looks really great now and again, many thanks and I appreciate your efforts very much :-)

                                npp_php_style_list.jpg

                                PeterJonesP 1 Reply Last reply Reply Quote 0
                                • PeterJonesP
                                  PeterJones @Manfred Drechsel
                                  last edited by PeterJones

                                  @Manfred-Drechsel said in Highlighting with self created words in "langs.xml" does not work:

                                  Only a minor issue which I’ve seen is the width of the style selection list control. It should be wider to see the full text of the styles. See attachment. Guess I directly should file an issue for NPP?

                                  It’s always been that way (see for example, the “INSTRUCTION WORD” on ActionScript, or the “Indent guideline style” in Global Styles, both of which have gone beyond the width for multiple versions of N++).

                                  If you want that aspect of the GUI changed, you would need to put in a feature request in N++'s GitHub repo. I would suggest asking for either resizable, or wider-by-default, or at least having a hover (or having a hover plus resizable/wider).

                                  I am highly doubtful that the Developer would implement be wider-by-default; there’s slightly more chance that he’d make it user-resizable; I would say the best-chance for implementation is using the full Style name as the hover text for each entry, which is why I suggested it, but no guarantees it would be implemented.

                                  1 Reply Last reply Reply Quote 3
                                  • First post
                                    Last post
                                  The Community of users of the Notepad++ text editor.
                                  Powered by NodeBB | Contributors