search

jcaillon / 3P

3P (short for Progress Programmers Pal), is a notepad++ plugin designed to help writing progress 4GL (OpenEdge ABL) code. It transforms notepad++ into a full featured IDE for openedge code editor, providing syntax checking, compilation, auto-completion, tool-tips and much more!
https://jcaillon.github.io/3P/
GNU General Public License v3.0
80 stars 35 forks source link

Bug with new syntax highlighting - AND is not highlighted #295

Closed AlbusPortucalis closed 2 years ago

AlbusPortucalis commented 2 years ago

Thanks for updating 3P so fast. It facilitates programming in Progress a lot. I share below a bug I saw with the new update.

image

AlbusPortucalis commented 2 years ago

The problem is only found with the developer studio mode of syntax highlighting.

jcaillon commented 2 years ago

Hi @AlbusPortucalis , glad that it is useful for you.

Took a few minutes to look at the bug. Actually you accidently found an issue that has been there for the very beggining x) Let me explain...

The screencap you attached is actually totally legit; this is how this theme is programmed; Operator #000000 @Default 0 operator keywords like AND OR... are in black on default background and with no font weight.

The issue is that if you try the different syntax highlighting theme through the UI: image

then you will see soemthing like that for dev studio theme:

image

and if you restart notepad++ after that, you will see it like that:

image

The difference is that some of the highlighted words are no longer in BOLD.

The actual bug is that, when switching the themes, i do not correctly reset the font size, giving the impression that some words are bold for the dev studio theme where everything is actually in normal font.

TL;DR; I will correct the bug and modify the theme so that the operators are more visible.

In the mean time, you can use this page to learn how to customize the style in question. Here is my proposal (i just changed the last digit, 2 for bold and 4 for italic):

> Developer studio
Default #000000 #FFF    0
Comment #3F7F5F @Default    0
SingleLineComment   @Comment    @Default    0
Preprocessor    #B3A68E @Default    4
JumpStatement   #7F0055 @Default    2
Statement   #7F0055 @Default    2
Type    #CD3A3A @Default    2
Keyword #7F0055 @Default    2
Operator    #000000 @Default    2
Abbreviation    #EC4D00 @Default    2
Include #FF0080 @Default    4
DoubleQuote #2A00FF @Default    0
SimpleQuote #2A00FF @Default    0
NormedVariables #FF8000 @Default    0
Number  #2A00FF @Default    0
CaretLine   #000000 #E8E8FF 0
Selection   #000000 lightgrey   0
WhiteSpace  #CCC    @Default    0
NoError #007B74 #C6EFCE 0
Error0  #3F3F3F #F2F2F2 0
Error1  #9C6500 #FFEB9C 0
Error2  #833C0C #FFCC99 0
Error3  #9C0006 #FFC7CE 0
Error4  #58267E #CC99FF 0
CaretColor  @Default    #000000 0
FoldMargin  #595959 #F3F3F3 0
FoldActiveMarker    @SimpleQuote    #000000 0
LineNumberMargin    #595959 #E4E4E4 0
SmartHighLighting   #09FF00 #000000 0
BraceHighLight  #333    #5EFF00 0
BadBraceHighLight   #FFFFFF #FF42FF 0
AlbusPortucalis commented 2 years ago

Got it! Thanks for the detailed answer.