Bug #2773
Bug #2677: fix drawing and functional differences between P2J GUI and 4GL GUI
simpler_alert_box.p fails in web GUI client
100%
Related issues
History
#1 Updated by Greg Shah over 8 years ago
Run ./simpler_alert_box.p
in the web GUI. You get this:
It is unresponsive to key or mouse processing, however I think this is true for Swing too and is something to be fixed in #2770.
In this task, we need to fix the broken drawing.
#2 Updated by Hynek Cihlar over 8 years ago
In Swing ALERT-BOX processes mouse and key ok. Can you post simpler_alert_box.p?
#3 Updated by Sergey Ivanovskiy over 8 years ago
In Swing ALERT-BOX processes mouse and key ok. Can you post simpler_alert_box.p?
It needs to update testcases project.
#4 Updated by Greg Shah over 8 years ago
It is checked into the testcases project. But here is the code:
message "Hello World!" update res as logical view-as alert-box information buttons yes-no. message "Result =" res.
#5 Updated by Greg Shah over 8 years ago
In Swing ALERT-BOX processes mouse and key ok.
In 11023 mouse works OK. But keys do not. Perhaps you have something in your build that is not checked in to 1811s?
#6 Updated by Sergey Ivanovskiy over 8 years ago
There is this exception on the js side:
21:53:31.126 TypeError: fontTable[font] is undefined1 p2j.fonts.js:262:7
#7 Updated by Sergey Ivanovskiy over 8 years ago
It seems that SPACE (32) key is eaten by some widgets on the java side.
#8 Updated by Hynek Cihlar over 8 years ago
Greg Shah wrote:
In Swing ALERT-BOX processes mouse and key ok.
In 11023 mouse works OK. But keys do not. Perhaps you have something in your build that is not checked in to 1811s?
I double checked, I have all the changes in repo. Can you please check whether you can mouse drag the alert-box window around?
#9 Updated by Hynek Cihlar over 8 years ago
Hynek Cihlar wrote:
Greg Shah wrote:
In Swing ALERT-BOX processes mouse and key ok.
In 11023 mouse works OK. But keys do not. Perhaps you have something in your build that is not checked in to 1811s?
I double checked, I have all the changes in repo. Can you please check whether you can mouse drag the alert-box window around?
Alright, mystery solved. The key input is really stuck when there is no main window. I always tested with main window on so I didn't see it. If you put message ""
as the first intruction or anything else that displays the main window, the key input will start work. I'm looking at it.
#10 Updated by Sergey Ivanovskiy over 8 years ago
Constantin, could you help how to eliminate this message on the server side for ./simpler_alert_box.p
[10/22/2015 22:27:45 MSK] (com.goldencode.p2j.ui.FontTable:SEVERE) Text [txt2] has no legacy metrics with font [ms sans serif,8,false,false,false].
Can it lead to the font undefined error?
Now
gd.getParagraphHeight
is called from MessagePanel
on doLayout
AlertBoxGuiImpl$MessagePanel.doLayout() line: 422 AlertBoxGuiImpl.doLayout() line: 131 AlertBoxGuiImpl(ModalWindow).show() line: 260 AlertBoxGuiImpl.show() line: 170 AlertBoxGuiImpl(AbstractWidget<O>).setVisible(boolean) line: 1053
and it leads to the font undefined error on the JS side and
gd.getParagraphHeight
returns zero value.TopLevelWindow<?> defWindow = getOwner(); gd.selectWindow(defWindow.getId().asInt()); // determine message height int paragraphHeight = gd.getParagraphHeight(messageText, font, paragraphWidth); gd.releaseWindow();
Is it a correct usage of
gd.getParagraphHeight(messageText, font, paragraphWidth);
?#11 Updated by Constantin Asofiei over 8 years ago
Sergey Ivanovskiy wrote:
Constantin, could you help how to eliminate this message on the server side for ./simpler_alert_box.p
[...]
Can it lead to the font undefined error?
No, this is related to the legacy text metrics - if not found, it relies on the driver to measure the text.
Is it a correct usage of
gd.getParagraphHeight(messageText, font, paragraphWidth);
?
Yes, this should work. Can you debug the font
value (index + font name) passed to gd.getParagraphHeight
and the font-table registered in p2j.fonts.js
? The fontTable[font] is undefined
un JS-side means an invalid font
or fontTable[font]
does not exist.
#12 Updated by Hynek Cihlar over 8 years ago
Hynek Cihlar wrote:
Alright, mystery solved. The key input is really stuck when there is no main window. I always tested with main window on so I didn't see it. If you put
message ""
as the first intruction or anything else that displays the main window, the key input will start work. I'm looking at it.
This seems to be an issue on the Swing driver level. The Swing window doesn't receive any key input when its owner is not visible.
#13 Updated by Sergey Ivanovskiy over 8 years ago
Please look at ./demo/demo_widgets.p. There is the definite issue with fill-in! SPACEs are accepted only in the middle of the text and can't be added to the end! Attention!!! May be the SPACES are cut.
#14 Updated by Constantin Asofiei over 8 years ago
Sergey Ivanovskiy wrote:
Please look at ./demo/demo_widgets.p. There is the definite issue with fill-in! SPACEs are accepted only in the middle of the text and can't be added to the end! Attention!!! May be the SPACES are cut.
Is this something new or something you know it used to work? Because as I think this is a behavior which was "inherited" from ChUI, to right-trim the text.
#15 Updated by Sergey Ivanovskiy over 8 years ago
Constantin Asofiei wrote:
Sergey Ivanovskiy wrote:
Please look at ./demo/demo_widgets.p. There is the definite issue with fill-in! SPACEs are accepted only in the middle of the text and can't be added to the end! Attention!!! May be the SPACES are cut.
Is this something new or something you know it used to work? Because as I think this is a behavior which was "inherited" from ChUI, to right-trim the text.
No, I haven't noticed this behaviour before.
#16 Updated by Greg Shah over 8 years ago
I'm separating the missing key events in Swing issue into #2775.
#17 Updated by Greg Shah over 8 years ago
Sergey Ivanovskiy wrote:
Constantin Asofiei wrote:
Sergey Ivanovskiy wrote:
Please look at ./demo/demo_widgets.p. There is the definite issue with fill-in! SPACEs are accepted only in the middle of the text and can't be added to the end! Attention!!! May be the SPACES are cut.
Is this something new or something you know it used to work? Because as I think this is a behavior which was "inherited" from ChUI, to right-trim the text.
No, I haven't noticed this behaviour before.
Sergey: what does Progress GUI do in this same situation? Does it keep trailing spaces?
#18 Updated by Sergey Ivanovskiy over 8 years ago
Greg Shah wrote:
Sergey Ivanovskiy wrote:
Constantin Asofiei wrote:
Sergey Ivanovskiy wrote:
Please look at ./demo/demo_widgets.p. There is the definite issue with fill-in! SPACEs are accepted only in the middle of the text and can't be added to the end! Attention!!! May be the SPACES are cut.
Is this something new or something you know it used to work? Because as I think this is a behavior which was "inherited" from ChUI, to right-trim the text.
No, I haven't noticed this behaviour before.
Sergey: what does Progress GUI do in this same situation? Does it keep trailing spaces?
I have checked it on windev01 for demo_widgets.p, the "fill-in" field accepts spaces at the end. I think it is a regression in our code and it happens recently. I worked with keyboards, and if it had been present, then I would notice it before deadline.
#19 Updated by Sergey Ivanovskiy over 8 years ago
Constantin Asofiei wrote:
Sergey Ivanovskiy wrote:
Constantin, could you help how to eliminate this message on the server side for ./simpler_alert_box.p
[...]
Can it lead to the font undefined error?No, this is related to the legacy text metrics - if not found, it relies on the driver to measure the text.
Is it a correct usage of
gd.getParagraphHeight(messageText, font, paragraphWidth);
?Yes, this should work. Can you debug the
font
value (index + font name) passed togd.getParagraphHeight
and the font-table registered inp2j.fonts.js
? ThefontTable[font] is undefined
un JS-side means an invalidfont
orfontTable[font]
does not exist.
The application tried to get font 0 and font 62 but there were only available these fonts:
getFontHeight for font = 0 p2j.fonts.js:262:7 11:08:29.331 key = 1 p2j.fonts.js:269:16 11:08:29.332 val = {"name":"MS Sans Serif","size":8,"bold":false,"italic":false,"underline":false,"font":"8pt 'MS Sans Serif'","height":12,"widths":[13,13,13,13,13,13,13,13,13,3,3,3,3,3,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,3,4,5,9,7,10,10,3,4,4,5,9,3,4,3,4,7,7,7,7,7,7,7,7,7,7,4,4,9,9,9,6,11,8,8,8,9,8,7,9,9,4,4,8,7,11,9,9,7,9,8,7,7,9,8,11,8,7,7,4,4,4,9,5,5,6,7,6,7,6,4,7,7,3,3,6,3,10,7,6,7,7,5,5,4,7,6,9,6,6,6,7,4,7,9,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,3,4,7,7,7,7,4,5,5,11,5,7,9,0,11,5,5,9,4,4,5,7,7,3,5,4,5,7,10,10,10,6,8,8,8,8,8,8,11,8,8,8,8,8,4,4,4,4,9,9,9,9,9,9,9,9,9,9,9,9,9,7,7,7,6,6,6,6,6,6,10,6,6,6,6,6,3,3,3,3,6,7,6,6,6,6,6,9,6,7,7,7,7,6,7,6],"maxWidth":13} p2j.fonts.js:270:16 11:08:29.333 key = 2 p2j.fonts.js:269:16 11:08:29.334 val = {"name":"Courier New","size":8,"bold":false,"italic":false,"underline":false,"font":"8pt 'Courier New'","height":12,"widths":[13,13,13,13,13,13,13,13,13,6,6,6,6,6,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,6,6,6,6,6,6,6,6,6,6,6,6,6,0,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6],"maxWidth":13} p2j.fonts.js:270:16 11:08:29.334 key = 3 p2j.fonts.js:269:16 11:08:29.335 val = {"name":"Tahoma","size":8,"bold":false,"italic":false,"underline":false,"font":"8pt 'Tahoma'","height":12,"widths":[13,13,13,13,13,13,13,13,13,3,3,3,3,3,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,3,4,5,9,7,10,10,3,4,4,5,9,3,4,3,4,7,7,7,7,7,7,7,7,7,7,4,4,9,9,9,6,11,8,8,8,9,8,7,9,9,4,4,8,7,11,9,9,7,9,8,7,7,9,8,11,8,7,7,4,4,4,9,5,5,6,7,6,7,6,4,7,7,3,3,6,3,10,7,6,7,7,5,5,4,7,6,9,6,6,6,7,4,7,9,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,3,4,7,7,7,7,4,5,5,11,5,7,9,0,11,5,5,9,4,4,5,7,7,3,5,4,5,7,10,10,10,6,8,8,8,8,8,8,11,8,8,8,8,8,4,4,4,4,9,9,9,9,9,9,9,9,9,9,9,9,9,7,7,7,6,6,6,6,6,6,10,6,6,6,6,6,3,3,3,3,6,7,6,6,6,6,6,9,6,7,7,7,7,6,7,6],"maxWidth":13} p2j.fonts.js:270:16 11:08:29.335 key = 4 p2j.fonts.js:269:16 11:08:29.336 val = {"name":"MS Sans Serif","size":10,"bold":false,"italic":false,"underline":false,"font":"10pt 'MS Sans Serif'","height":15,"widths":[13,13,13,13,13,13,13,13,13,4,4,4,4,4,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,4,4,5,7,7,12,9,3,4,4,5,8,4,4,4,4,7,7,7,7,7,7,7,7,7,7,4,4,8,8,8,7,14,9,9,10,10,9,8,10,10,4,7,9,7,11,10,10,9,10,10,9,8,10,9,13,9,9,8,4,4,4,6,7,4,7,7,7,7,7,4,7,7,3,3,7,3,11,7,7,7,7,4,7,4,7,7,10,7,7,7,4,3,4,8,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,4,4,7,7,7,7,3,7,4,10,5,7,8,0,10,7,5,8,4,4,4,7,7,4,4,4,5,7,11,11,11,7,9,9,9,9,9,9,13,10,9,9,9,9,4,4,4,4,10,10,10,10,10,10,10,8,10,10,10,10,10,9,9,8,7,7,7,7,7,7,12,7,7,7,7,7,3,3,3,3,7,7,7,7,7,7,7,8,7,7,7,7,7,7,7,7],"maxWidth":14} p2j.fonts.js:270:16 11:08:29.337 key = 5 p2j.fonts.js:269:16 11:08:29.338 val = {"name":"MS Sans Serif","size":8,"bold":true,"italic":false,"underline":false,"font":"bold 8pt 'MS Sans Serif'","height":13,"widths":[13,13,13,13,13,13,13,13,13,3,3,3,3,3,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,3,3,4,6,6,9,7,2,4,4,4,6,3,4,3,3,6,6,6,6,6,6,6,6,6,6,3,3,6,6,6,6,11,7,7,8,8,7,7,8,8,3,5,7,6,9,8,8,7,8,8,7,7,8,7,10,7,7,7,3,3,3,5,6,4,6,6,5,6,6,3,6,6,2,2,5,2,9,6,6,6,6,4,5,3,6,5,8,5,5,5,4,3,4,6,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,3,3,6,6,6,6,3,6,4,8,4,6,6,0,8,5,4,6,4,4,4,6,6,3,4,4,4,6,9,9,9,6,7,7,7,7,7,7,11,8,7,7,7,7,3,3,3,3,8,8,8,8,8,8,8,6,8,8,8,8,8,7,7,7,6,6,6,6,6,6,9,5,6,6,6,6,2,2,2,2,6,6,6,6,6,6,6,6,6,6,6,6,6,5,6,5],"maxWidth":13} p2j.fonts.js:270:16 11:08:29.338 key = -1 p2j.fonts.js:269:16 11:08:29.339 val = {"name":"sans-serif","size":8,"bold":true,"italic":false,"underline":false,"font":"bold 8pt 'sans-serif'","height":12,"widths":[13,13,13,13,13,13,13,13,13,4,4,4,4,4,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,4,5,6,9,7,11,9,3,5,5,6,9,4,4,4,4,7,7,7,7,7,7,7,7,7,7,4,4,9,9,9,6,11,8,8,8,9,7,7,9,9,4,4,8,7,11,9,9,8,9,8,8,7,9,8,12,8,8,8,5,4,5,9,5,5,7,8,6,8,7,5,8,8,4,4,7,4,11,8,7,8,8,5,6,5,8,7,10,7,7,6,8,4,8,9,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,4,5,7,7,7,7,4,5,5,11,6,7,9,0,11,5,5,9,5,5,5,8,7,4,5,5,6,7,11,11,11,6,8,8,8,8,8,8,12,8,7,7,7,7,4,4,4,4,9,9,9,9,9,9,9,9,9,9,9,9,9,8,8,8,7,7,7,7,7,7,11,6,7,7,7,7,4,4,4,4,7,8,7,7,7,7,7,9,7,8,8,8,8,7,8,7],"maxWidth":13} p2j.fonts.js:270:16 11:08:29.339 TypeError: fontTable[font] is undefined p2j.fonts.js:274:7 11:08:39.440 resize width = 296 height = 144 p2j.screen.js:589:7 11:08:39.916 txt = 1 font = 62 p2j.screen.js:2029:7 11:08:39.916 getFontHeight for font = 62 p2j.fonts.js:262:7 11:08:39.917 key = 1 p2j.fonts.js:269:16 11:08:39.917 val = {"name":"MS Sans Serif","size":8,"bold":false,"italic":false,"underline":false,"font":"8pt 'MS Sans Serif'","height":12,"widths":[13,13,13,13,13,13,13,13,13,3,3,3,3,3,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,3,4,5,9,7,10,10,3,4,4,5,9,3,4,3,4,7,7,7,7,7,7,7,7,7,7,4,4,9,9,9,6,11,8,8,8,9,8,7,9,9,4,4,8,7,11,9,9,7,9,8,7,7,9,8,11,8,7,7,4,4,4,9,5,5,6,7,6,7,6,4,7,7,3,3,6,3,10,7,6,7,7,5,5,4,7,6,9,6,6,6,7,4,7,9,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,3,4,7,7,7,7,4,5,5,11,5,7,9,0,11,5,5,9,4,4,5,7,7,3,5,4,5,7,10,10,10,6,8,8,8,8,8,8,11,8,8,8,8,8,4,4,4,4,9,9,9,9,9,9,9,9,9,9,9,9,9,7,7,7,6,6,6,6,6,6,10,6,6,6,6,6,3,3,3,3,6,7,6,6,6,6,6,9,6,7,7,7,7,6,7,6],"maxWidth":13} p2j.fonts.js:270:16 11:08:39.918 key = 2 p2j.fonts.js:269:16 11:08:39.918 val = {"name":"Courier New","size":8,"bold":false,"italic":false,"underline":false,"font":"8pt 'Courier New'","height":12,"widths":[13,13,13,13,13,13,13,13,13,6,6,6,6,6,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,6,6,6,6,6,6,6,6,6,6,6,6,6,0,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6,6],"maxWidth":13} p2j.fonts.js:270:16 11:08:39.919 key = 3 p2j.fonts.js:269:16 11:08:39.919 val = {"name":"Tahoma","size":8,"bold":false,"italic":false,"underline":false,"font":"8pt 'Tahoma'","height":12,"widths":[13,13,13,13,13,13,13,13,13,3,3,3,3,3,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,3,4,5,9,7,10,10,3,4,4,5,9,3,4,3,4,7,7,7,7,7,7,7,7,7,7,4,4,9,9,9,6,11,8,8,8,9,8,7,9,9,4,4,8,7,11,9,9,7,9,8,7,7,9,8,11,8,7,7,4,4,4,9,5,5,6,7,6,7,6,4,7,7,3,3,6,3,10,7,6,7,7,5,5,4,7,6,9,6,6,6,7,4,7,9,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,3,4,7,7,7,7,4,5,5,11,5,7,9,0,11,5,5,9,4,4,5,7,7,3,5,4,5,7,10,10,10,6,8,8,8,8,8,8,11,8,8,8,8,8,4,4,4,4,9,9,9,9,9,9,9,9,9,9,9,9,9,7,7,7,6,6,6,6,6,6,10,6,6,6,6,6,3,3,3,3,6,7,6,6,6,6,6,9,6,7,7,7,7,6,7,6],"maxWidth":13} p2j.fonts.js:270:16 11:08:39.920 key = 4 p2j.fonts.js:269:16 11:08:39.920 val = {"name":"MS Sans Serif","size":10,"bold":false,"italic":false,"underline":false,"font":"10pt 'MS Sans Serif'","height":15,"widths":[13,13,13,13,13,13,13,13,13,4,4,4,4,4,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,4,4,5,7,7,12,9,3,4,4,5,8,4,4,4,4,7,7,7,7,7,7,7,7,7,7,4,4,8,8,8,7,14,9,9,10,10,9,8,10,10,4,7,9,7,11,10,10,9,10,10,9,8,10,9,13,9,9,8,4,4,4,6,7,4,7,7,7,7,7,4,7,7,3,3,7,3,11,7,7,7,7,4,7,4,7,7,10,7,7,7,4,3,4,8,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,4,4,7,7,7,7,3,7,4,10,5,7,8,0,10,7,5,8,4,4,4,7,7,4,4,4,5,7,11,11,11,7,9,9,9,9,9,9,13,10,9,9,9,9,4,4,4,4,10,10,10,10,10,10,10,8,10,10,10,10,10,9,9,8,7,7,7,7,7,7,12,7,7,7,7,7,3,3,3,3,7,7,7,7,7,7,7,8,7,7,7,7,7,7,7,7],"maxWidth":14} p2j.fonts.js:270:16 11:08:39.920 key = 5 p2j.fonts.js:269:16 11:08:39.921 val = {"name":"MS Sans Serif","size":8,"bold":true,"italic":false,"underline":false,"font":"bold 8pt 'MS Sans Serif'","height":13,"widths":[13,13,13,13,13,13,13,13,13,3,3,3,3,3,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,3,3,4,6,6,9,7,2,4,4,4,6,3,4,3,3,6,6,6,6,6,6,6,6,6,6,3,3,6,6,6,6,11,7,7,8,8,7,7,8,8,3,5,7,6,9,8,8,7,8,8,7,7,8,7,10,7,7,7,3,3,3,5,6,4,6,6,5,6,6,3,6,6,2,2,5,2,9,6,6,6,6,4,5,3,6,5,8,5,5,5,4,3,4,6,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,3,3,6,6,6,6,3,6,4,8,4,6,6,0,8,5,4,6,4,4,4,6,6,3,4,4,4,6,9,9,9,6,7,7,7,7,7,7,11,8,7,7,7,7,3,3,3,3,8,8,8,8,8,8,8,6,8,8,8,8,8,7,7,7,6,6,6,6,6,6,9,5,6,6,6,6,2,2,2,2,6,6,6,6,6,6,6,6,6,6,6,6,6,5,6,5],"maxWidth":13} p2j.fonts.js:270:16 11:08:39.921 key = -1 p2j.fonts.js:269:16 11:08:39.922 val = {"name":"sans-serif","size":8,"bold":true,"italic":false,"underline":false,"font":"bold 8pt 'sans-serif'","height":12,"widths":[13,13,13,13,13,13,13,13,13,4,4,4,4,4,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,4,5,6,9,7,11,9,3,5,5,6,9,4,4,4,4,7,7,7,7,7,7,7,7,7,7,4,4,9,9,9,6,11,8,8,8,9,7,7,9,9,4,4,8,7,11,9,9,8,9,8,8,7,9,8,12,8,8,8,5,4,5,9,5,5,7,8,6,8,7,5,8,8,4,4,7,4,11,8,7,8,8,5,6,5,8,7,10,7,7,6,8,4,8,9,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,13,4,5,7,7,7,7,4,5,5,11,6,7,9,0,11,5,5,9,5,5,5,8,7,4,5,5,6,7,11,11,11,6,8,8,8,8,8,8,12,8,7,7,7,7,4,4,4,4,9,9,9,9,9,9,9,9,9,9,9,9,9,8,8,8,7,7,7,7,7,7,11,6,7,7,7,7,4,4,4,4,7,8,7,7,7,7,7,9,7,8,8,8,8,7,8,7],"maxWidth":13} p2j.fonts.js:270:16 11:08:39.922 TypeError: fontTable[font] is undefined
What could it mean? (The font 0 and 62 aren't registered)
#20 Updated by Constantin Asofiei over 8 years ago
Sergey, fontTable
in p2j.fonts.js should be an associative array - with keys both numbers and texts. In p2j.fonts.js:createFont
, fontTable
uses as keys both the font ID (key is numeric, the value is where font details are saved) and the font-key (which is a string key and refers to the font ID). Maybe there's something wrong there?
#21 Updated by Sergey Ivanovskiy over 8 years ago
- File font_issue_1.txt added
Constantin, please review this font fix, it is due to incorrect provided arguments in js functions calls. We need a hot fix to 1811s. It also fixes this issue.
#22 Updated by Constantin Asofiei over 8 years ago
Sergey Ivanovskiy wrote:
Constantin, please review this font fix, it is due to incorrect provided arguments in js functions calls. We need a hot fix to 1811s. It also fixes this issue.
I;m OK with the changes, please commit the fix to 1811s.
#23 Updated by Sergey Ivanovskiy over 8 years ago
Committed revision 11029 fixes the broken alarm dialog layout.
#24 Updated by Greg Shah over 8 years ago
- Status changed from New to Closed
#25 Updated by Greg Shah over 8 years ago
- % Done changed from 0 to 100
#26 Updated by Greg Shah over 7 years ago
- Target version changed from Milestone 12 to GUI Support for a Complex ADM2 App