FWD » Core Development » User Interface

04/10/2020

06:23 PM Bug #4612: Fix ChUI browse drawing when the current row is out of the view

In a ChUI browse the current row can be out of the view. It is a rare case and I'm aware only of one case when this h... Stanislav Lomany

05:54 PM Bug #4612 (New): Fix ChUI browse drawing when the current row is out of the view

Stanislav Lomany

09:23 AM Feature #3876: add support for "direct font control" at the widget level

Greg Shah wrote:
> Please create a new task for that. We will defer work on it for now.
Done. Hynek Cihlar

09:13 AM Feature #3876: add support for "direct font control" at the widget level

Please create a new task for that. We will defer work on it for now. In the wiki page, explain the idea and link to... Greg Shah

09:12 AM Feature #3876: add support for "direct font control" at the widget level

Greg Shah wrote:
> > He could do this using the standard system tools. Or do you want to add other 4GL extensions th... Hynek Cihlar

09:07 AM Feature #3876: add support for "direct font control" at the widget level

> He could do this using the standard system tools. Or do you want to add other 4GL extensions that would allow this?... Greg Shah

09:00 AM Feature #3876: add support for "direct font control" at the widget level

Greg Shah wrote:
> I've just edited the "Font Control in 4GL Widgets":/projects/p2j/wiki/Font_Control_in_4GL_Widgets... Hynek Cihlar

08:16 AM Feature #3876: add support for "direct font control" at the widget level

I've just edited the "Font Control in 4GL Widgets":/projects/p2j/wiki/Font_Control_in_4GL_Widgets. Please review and... Greg Shah

09:21 AM Bug #4611: Implement a font enumeration Java API directly callable from legacy code

The motivation here is to provide a way to allow to enumerate available font faces in legacy code. That is FWD/4GL co... Hynek Cihlar

09:16 AM Bug #4611 (New): Implement a font enumeration Java API directly callable from legacy code

Hynek Cihlar

04/09/2020

02:24 PM Feature #4164: implement misc UI features (frame options, misc attributes)

As Constantin notes, you are telling the code to only have the frame positioned in @col 1 row 1@ because only the FIR... Greg Shah

01:05 PM Feature #4164: implement misc UI features (frame options, misc attributes)

Roger, the @AT COL ROW@ in the frame phrase (for DISPLAY) sets the frame's definition location, not at runtime.
To... Constantin Asofiei

01:01 PM Feature #4164: implement misc UI features (frame options, misc attributes)

New testcase has expanded my ability to validate values, but I'm not sure it's on the mark yet. I was expecting the f... Roger Borrello

04/08/2020

11:29 AM Feature #4394: add some frame options and attributes

The @frame-name@ is just a string, right? So you can do anything with it to see the value (e.g. @message widget:fram... Greg Shah

11:03 AM Feature #4394: add some frame options and attributes

I cannot find any useful documentation related to how @FRAME-NAME@ would be used in a testcase. Searching through our... Roger Borrello

09:05 AM Feature #4609: rework scrolling approach to use an cleaner, MVC design

For a customer-specific grid widget, Vladimir wrote an alternative MVC scrolling implementation. The details are doc... Greg Shah

09:02 AM Feature #4609 (New): rework scrolling approach to use an cleaner, MVC design

Greg Shah

04/06/2020

04:33 PM Feature #4164: implement misc UI features (frame options, misc attributes)

Greg Shah wrote:
> Make a list of the problems here and fix them.
>1. What do these return before the frame is real... Roger Borrello

04/03/2020

01:00 PM Feature #4164: implement misc UI features (frame options, misc attributes)

Make a list of the problems here and fix them. Greg Shah

12:33 PM Feature #4164: implement misc UI features (frame options, misc attributes)

I'll update, now that I (finally) understand. I do have some initial items, some not related to this testcase, but be... Roger Borrello

11:19 AM Feature #4164: implement misc UI features (frame options, misc attributes)

I think you should check the @FRAME-ROW(frame)@ builtin for all of the same case.
Make sure to test:
* What do ... Greg Shah

10:38 AM Feature #4164: implement misc UI features (frame options, misc attributes)

Greg Shah wrote:
> > That is not really correct, because the attribute is off the widget, in this case i.
>
> Pl... Roger Borrello

10:16 AM Feature #4164: implement misc UI features (frame options, misc attributes)

> That is not really correct, because the attribute is off the widget, in this case i.
Please read #4164-1 more c... Greg Shah

09:54 AM Feature #4164: implement misc UI features (frame options, misc attributes)

Greg Shah wrote:
> All you care about is positioning the frame in different locations.
That is not really corre... Roger Borrello

09:34 AM Feature #4164: implement misc UI features (frame options, misc attributes)

I had a testcase:... Roger Borrello

08:53 AM Feature #4164: implement misc UI features (frame options, misc attributes)

All you care about is positioning the frame in different locations. This is trivial to do in the AT phrase of the fr... Greg Shah

04/02/2020

05:40 PM Bug #4604: DCOLOR on FRAME or BROWSE TITLE phrase results in title not being displayed

Using @4231b-11418@:
Testcase @./uast/frame_title_color_default_values.p@ shows that @define frame@ with a title p... Roger Borrello

09:04 AM Bug #4604: DCOLOR on FRAME or BROWSE TITLE phrase results in title not being displayed

Please document the testcase and the trunk revision in which this can be recreated. Greg Shah

04/01/2020

06:54 PM Bug #4604 (New): DCOLOR on FRAME or BROWSE TITLE phrase results in title not being displayed

Roger Borrello

01:58 PM Feature #4393: add TITLE-DCOLOR attribute support

Yes, it could be a version difference. Run the testcases on both 10.x and 11.x and record the differences only. The... Greg Shah

12:22 PM Feature #4393: add TITLE-DCOLOR attribute support

I was re-running @frame_title_color_attributes.p@ to try to close this task down, and I was running on OE release 10.... Roger Borrello

10:53 AM Feature #4393: add TITLE-DCOLOR attribute support

h2. Browse TITLE Attribute Defaults (before/after realization)
|_. |_. ChUI (bef) |_. ChUI (aft) |_. GUI (... Roger Borrello

08:09 PM Feature #4393: add TITLE-DCOLOR attribute support

> Should I remove the comment: "For the browse options phrase, only the TITLE DCOLOR <num> "title" form can be used..... Greg Shah

03/31/2020

07:00 PM Feature #4393: add TITLE-DCOLOR attribute support

Made updates in Rev11413. I was attempting to get the title to appear when @DCOLOR@ was part of the title phrase. Thi... Roger Borrello

05:05 PM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> No, looking back on things they are all possible. I didn't update the comment in @frame_generato... Roger Borrello

04:02 PM Feature #4393: add TITLE-DCOLOR attribute support

No, looking back on things they are all possible. I didn't update the comment in @frame_generator.xml@ but the notes... Greg Shah

03:45 PM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> See the @gen_color@ in @frame_generator.xml@. It should only be generating code for @TITLE DCOLO... Roger Borrello

03:18 PM Feature #4393: add TITLE-DCOLOR attribute support

Not sure the @realized@ flag is correct.
See the @gen_color@ in @frame_generator.xml@. It should only be generati... Greg Shah

03:13 PM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> I guess you are referencing @testcases/uast/browse_title_color_options.p@. Options are not attri... Roger Borrello

03:12 PM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> What testcase are you using? The code in @testcases/uast/browse_title_color_attributes.p@ doesn'... Roger Borrello

03:11 PM Feature #4393: add TITLE-DCOLOR attribute support

I guess you are referencing @testcases/uast/browse_title_color_options.p@. Options are not attributes. The text abo... Greg Shah

02:46 PM Feature #4393: add TITLE-DCOLOR attribute support

What testcase are you using? The code in @testcases/uast/browse_title_color_attributes.p@ doesn't look anything like... Greg Shah

02:16 PM Feature #4393: add TITLE-DCOLOR attribute support

Another issue I am looking at. The browse title is missing, if @DCOLOR@ is part of the options:
!GUI_browse_title_... Roger Borrello

01:45 PM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> Be more specific. In which testscase and for which lines of code is this happening? GUI, ChUI? ... Roger Borrello

01:11 PM Feature #4393: add TITLE-DCOLOR attribute support

> I see the testcase in FWD displays 4052 in a popup for every browse widget.
Be more specific. In which testsca... Greg Shah

12:42 PM Feature #4393: add TITLE-DCOLOR attribute support

Done, but in my testing, I see the testcase in FWD displays 4052 in a popup for every browse widget. Either I need to... Roger Borrello

12:22 PM Feature #4393: add TITLE-DCOLOR attribute support

So @setTitleColor(ColorSpec)@ was just missing. My changes fixed the conversion, but you will need to add the runtim... Greg Shah

12:14 PM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> What happens in trunk?... Roger Borrello

12:07 PM Feature #4393: add TITLE-DCOLOR attribute support

> It looks like setTitleColor needs to be implemented. I have added it to BrowseWidget and I believe the interface is... Greg Shah

11:55 AM Feature #4393: add TITLE-DCOLOR attribute support

Compilation of the testcase:... Roger Borrello

11:52 AM Feature #4393: add TITLE-DCOLOR attribute support

Roger Borrello wrote:
> Converting the @browse_title_color_options.p@ testcase, @setTitleColor@ is unresolved. Did t... Greg Shah

11:43 AM Feature #4393: add TITLE-DCOLOR attribute support

Converting the @browse_title_color_options.p@ testcase, @setTitleColor@ is unresolved. Did that method ne get overloo... Roger Borrello

11:31 AM Feature #4393: add TITLE-DCOLOR attribute support

Code Review updates made:... Roger Borrello

10:55 AM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> For now, I'm just suggesting that you create a helper for @BrowseWidget@, so make common code ins... Roger Borrello

10:14 AM Feature #4393: add TITLE-DCOLOR attribute support

For now, I'm just suggesting that you create a helper for @BrowseWidget@, so make common code instead of the duplicat... Greg Shah

10:12 AM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> Anytime you are writing the same code over and over, you can make a helper. It is ALWAYS better ... Roger Borrello

09:52 AM Feature #4393: add TITLE-DCOLOR attribute support

Anytime you are writing the same code over and over, you can make a helper. It is ALWAYS better than copying the cod... Greg Shah

09:45 AM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> Code Review Task Branch 4231b Revision 11407
> 4. In @BrowseWidget@, you would have much less co... Roger Borrello

09:06 AM Feature #4393: add TITLE-DCOLOR attribute support

> Just to be sure... you want comments removed?
Yes.
> Isn't there any usefulness to tieing the keyword to the... Greg Shah

09:03 AM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> Code Review Task Branch 4231b Revision 11407
>
> 1. In @gaps/expressions.rules@, the @kw_tit*@... Roger Borrello

08:39 AM Feature #4393: add TITLE-DCOLOR attribute support

Code Review Task Branch 4231b Revision 11407
1. In @gaps/expressions.rules@, the @kw_tit*@ comments should be remo... Greg Shah

11:00 AM Feature #4394: add some frame options and attributes

@SCROLL@ and @NO-ATTR-SPACE@ will be finished in #4164. The only thing needed for this task is @FRAME-NAME@. Greg Shah

03/30/2020

04:14 PM Feature #4393: add TITLE-DCOLOR attribute support

Committed updates:... Roger Borrello

03:59 PM Feature #4393: add TITLE-DCOLOR attribute support

Roger Borrello wrote:
> Currently looking at why the @./uast/menu/menu_title_color_attributes.p@ testcase fails in *... Roger Borrello

03:21 PM Feature #4393: add TITLE-DCOLOR attribute support

Currently looking at why the @./uast/menu/menu_title_color_attributes.p@ testcase fails in *Core Code Conversion*:
<... Roger Borrello

02:17 PM Feature #4393: add TITLE-DCOLOR attribute support

Same issues occur with @trunk@. I'll focus on implementation, but these will prevent validation of the implementation... Roger Borrello

02:10 PM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> What do you mean by "build"? It parsed fine.
When I tried to convert the @./uast/browse_title... Roger Borrello

02:04 PM Feature #4393: add TITLE-DCOLOR attribute support

What do you mean by "build"? It parsed fine. Greg Shah

02:03 PM Feature #4393: add TITLE-DCOLOR attribute support

Suddenly that implementation seems secondary to an issue that came up when I tried to build @./uast/browse_title_colo... Roger Borrello

02:00 PM Feature #4393: add TITLE-DCOLOR attribute support

The frame definitions will use @setTitle*(long)@. It is possible for the attribute usage to convert this way too, un... Greg Shah

01:51 PM Feature #4393: add TITLE-DCOLOR attribute support

Doesn't the @int64@ interface to the converted 4GL? Perhaps that's what is confusing me. Is there a @long@ interface ... Roger Borrello

12:52 PM Feature #4393: add TITLE-DCOLOR attribute support

No.
> Just reverse the logic. The long version calls the wrapper version.
This means that @setTitleFgColor(... Greg Shah

12:25 PM Feature #4393: add TITLE-DCOLOR attribute support

I see the missing "new" in there. Roger Borrello

12:20 PM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> You did not reverse the logic.
Like this?... Roger Borrello

11:37 AM Feature #4393: add TITLE-DCOLOR attribute support

You did not reverse the logic. Greg Shah

11:23 AM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> Just reverse the logic. The @long@ version calls the wrapper version.
Can you review this imp... Roger Borrello

11:19 AM Feature #4393: add TITLE-DCOLOR attribute support

Just reverse the logic. The @long@ version calls the wrapper version. Greg Shah

10:49 AM Feature #4393: add TITLE-DCOLOR attribute support

The previous convention for the setters had the @int64@ versions calling the @long@ versions:... Roger Borrello

10:24 AM Feature #4393: add TITLE-DCOLOR attribute support

> For the cases where menu TITLE- attributes are actually handled (TITLE-BGCOLOR, TITLE-FGCOLOR and TITLE-FONT), but ... Greg Shah

10:07 AM Feature #4393: add TITLE-DCOLOR attribute support

For the cases where @menu TITLE-@ attributes are actually handled (TITLE-BGCOLOR, TITLE-FGCOLOR and TITLE-FONT), but ... Roger Borrello

09:20 AM Feature #4393: add TITLE-DCOLOR attribute support

Yes. Greg Shah

09:20 AM Feature #4393: add TITLE-DCOLOR attribute support

In @MenuWidget@, is it adequate to utilize @config.popupOnly@ to determine between popup and non-popup menus for the ... Roger Borrello

03/27/2020

06:53 PM Feature #4393: add TITLE-DCOLOR attribute support

Cleaned up and checked in:... Roger Borrello

05:48 PM Feature #4393: add TITLE-DCOLOR attribute support

> All the ChUI tests passed side-by-side, except for the fact that the color schemes seemed different. There were oft... Greg Shah

05:44 PM Feature #4393: add TITLE-DCOLOR attribute support

Remove all getters. Remove the @int64@ setters. Only the @long@ setters should be in either @CommonFrame@ or @Gener... Greg Shah

05:43 PM Feature #4393: add TITLE-DCOLOR attribute support

Roger Borrello wrote:
> Greg Shah wrote:
>
> > We need the @setTitle*(long)@ methods in @CommonFrame@. That is i... Greg Shah

05:42 PM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> We need the @setTitle*(long)@ methods in @CommonFrame@. That is it. Don't add the full @Title... Roger Borrello

05:22 PM Feature #4393: add TITLE-DCOLOR attribute support

I will go back and update per your last note. I was writing the below while you made your update:
All the ChUI tes... Roger Borrello

05:19 PM Feature #4393: add TITLE-DCOLOR attribute support

> Menus and Browse don't have the same complexity of the class structure, do they?
No, they don't.
> All the fr... Greg Shah

04:39 PM Feature #4393: add TITLE-DCOLOR attribute support

Roger Borrello wrote:
> The frames are CommonFrame, and trying to access the methods which aren't in that interface.... Roger Borrello

03:57 PM Feature #4393: add TITLE-DCOLOR attribute support

The frames are CommonFrame, and trying to access the methods which aren't in that interface.... Roger Borrello

03:45 PM Feature #4393: add TITLE-DCOLOR attribute support

Roger Borrello wrote:
> Updates for frames in *4231b-11400*.
Hold on... a couple more frame testcases to validate... Roger Borrello

03:41 PM Feature #4393: add TITLE-DCOLOR attribute support

Updates for frames in *4231b-11400*. Roger Borrello

03:26 PM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> Try @frame df:title-dcolor = 1@ instead of @3@ on line 208 of @testcases/uast/frame_title_color_a... Roger Borrello

03:22 PM Feature #4393: add TITLE-DCOLOR attribute support

Try @frame df:title-dcolor = 1@ instead of @3@ on line 208 of @testcases/uast/frame_title_color_attributes.p@. You s... Greg Shah

03:11 PM Feature #4393: add TITLE-DCOLOR attribute support

My previous note was stating the "current" state, so thank you for clarifying.
With respect to the ChUI testing, I... Roger Borrello

02:17 PM Feature #4393: add TITLE-DCOLOR attribute support

> TitiledElement - leave alone
Yes
> CommonFrame - currently an inteface that extends TitledElement. No getters... Greg Shah

01:51 PM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> Yes, I'm thinking that we leave the frame options in @CommonFrame@/@GenericFrame@. These are jus... Roger Borrello

09:56 AM Feature #4393: add TITLE-DCOLOR attribute support

Yes, I'm thinking that we leave the frame options in @CommonFrame@/@GenericFrame@. These are just the setters (@setT... Greg Shah

09:52 AM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> > Or, do you mean to convert frame f1:<widget/method> differently, and access the asWidget() meth... Constantin Asofiei

09:48 AM Feature #4393: add TITLE-DCOLOR attribute support

> Or, do you mean to convert frame f1:<widget/method> differently, and access the asWidget() method directly?
Yes,... Greg Shah

09:46 AM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> I was thinking about the dual nature, but I guess I got part of it backwards. If not present in ... Constantin Asofiei

09:39 AM Feature #4393: add TITLE-DCOLOR attribute support

I was thinking about the dual nature, but I guess I got part of it backwards. If not present in @widget_references.r... Greg Shah

09:25 AM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> Roger: If I'm right, then you would remove all the changes from @FrameWidget@ and just put them i... Constantin Asofiei

09:12 AM Feature #4393: add TITLE-DCOLOR attribute support

Roger: If I'm right, then you would remove all the changes from @FrameWidget@ and just put them in @GenericFrame@. Greg Shah

09:12 AM Feature #4393: add TITLE-DCOLOR attribute support

Code Review Task Branch 4231b Revision 11399
OK, you are close here.
Looking deeper at the changes in @widget_r... Greg Shah

03/26/2020

07:34 PM Feature #4393: add TITLE-DCOLOR attribute support

Roger Borrello wrote:
> Right now, the error message is being displayed, but the client is cycling in a loop, and no... Roger Borrello

07:06 PM Feature #4393: add TITLE-DCOLOR attribute support

Look at the Hotel ChUI @p2j.cfg.xml@ for clues. You can also add things to that project to test, though the regular ... Greg Shah

06:43 PM Feature #4393: add TITLE-DCOLOR attribute support

For the ChUI, what is the key to building it as-if using ChUI development, so that @&quot;{&amp;window-system}&quot; ... Roger Borrello

05:08 PM Feature #4393: add TITLE-DCOLOR attribute support

I have the @frame_title_color_attributes.p@ testcase working on GUI. There are some differences, between FWD and 4GL,... Roger Borrello

04:48 PM Feature #4393: add TITLE-DCOLOR attribute support

Correct. Greg Shah

04:34 PM Feature #4393: add TITLE-DCOLOR attribute support

In @common-progress.rules:read_only_attribute@ is included @prog.kw_titl_bgc@. I am going to remove this, since we ca... Roger Borrello

03:52 PM Feature #4393: add TITLE-DCOLOR attribute support

Nevermind. Found the sources to our @readOnlyError@. Roger Borrello

03:43 PM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> Figure out why the @getTitleFont()@ is converting differently than @getTitleBgColor()@. The font... Roger Borrello

02:59 PM Feature #4393: add TITLE-DCOLOR attribute support

Figure out why the @getTitleFont()@ is converting differently than @getTitleBgColor()@. The font is not converting t... Greg Shah

02:57 PM Feature #4393: add TITLE-DCOLOR attribute support

No, I don't think so. Greg Shah

02:56 PM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> Why would we want it to be @extends@?
I don't want to do that. I believe I need to add our get... Roger Borrello

02:38 PM Feature #4393: add TITLE-DCOLOR attribute support

Why would we want it to be @extends@? Greg Shah

02:23 PM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> Did you add it to @FrameWidget@?
ahh... @TitledElement@ is in there as @implements@, but not @... Roger Borrello

02:21 PM Feature #4393: add TITLE-DCOLOR attribute support

Did you add it to @FrameWidget@? Greg Shah

02:16 PM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> Remove it from @CommonFrame@.
When @extends TitledElement@ is removed from @CommonFrame@, the ... Roger Borrello

01:47 PM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> Feel free to fix your bugs.
LOL... nowhere did I add a 4052 warning... I'll find and fix, whoeve... Roger Borrello

01:06 PM Feature #4393: add TITLE-DCOLOR attribute support

Remove it from @CommonFrame@.
> Proceeding with the testcase, with all getters/setters in both GenericFrame and Fr... Greg Shah

12:46 PM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> @GenericFrame@ should not implement @TitledElement@, only implement in @FrameWidget@. You can st... Roger Borrello

12:04 PM Feature #4393: add TITLE-DCOLOR attribute support

@GenericFrame@ should not implement @TitledElement@, only implement in @FrameWidget@. You can still leave the setter... Greg Shah

11:23 AM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> > Where are the frame-options methods?
>
> That is the @setTitleColor(ColorSpec)@ (which you k... Roger Borrello

11:11 AM Feature #4393: add TITLE-DCOLOR attribute support

> Where are the frame-options methods?
That is the @setTitleColor(ColorSpec)@ (which you keep complaining about) w... Greg Shah

11:01 AM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> I forgot that we emit attributes using the @FrameWidget@ instead of direct access to the @Generic... Roger Borrello

10:48 AM Feature #4393: add TITLE-DCOLOR attribute support

I forgot that we emit attributes using the @FrameWidget@ instead of direct access to the @GenericFrame@ instance. Th... Greg Shah

10:38 AM Feature #4393: add TITLE-DCOLOR attribute support

Using this to ef with @titleDColor@:... Roger Borrello

09:05 AM Feature #4393: add TITLE-DCOLOR attribute support

> In setTitleColor(ColorSpec titleColor) on line 6860? That wasn't me... I mentioned how odd that method looked. Did ... Greg Shah

08:55 AM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> Code Review Task Branch 4231b Revision 11395
>
> The changes are moving the right direction.
... Roger Borrello

07:29 AM Feature #4393: add TITLE-DCOLOR attribute support

Code Review Task Branch 4231b Revision 11395
The changes are moving the right direction.
1. You've messed with ... Greg Shah

06:53 AM Feature #4393: add TITLE-DCOLOR attribute support

> Does this mean titleDColor is stored outside GenericFrame?
Yes.
As I said in #4393-20:
> Thus, only @conf... Greg Shah

09:45 PM Feature #4393: add TITLE-DCOLOR attribute support

Updates are buildable in revision 11395. Right now @GenericFrame@ is the only update without stubs. Please review... ... Roger Borrello

09:34 PM Feature #4393: add TITLE-DCOLOR attribute support

The documentation for Attributes:Frames for writing/assignment says...
_in ChUI:_
_before realization, all ... Roger Borrello

10:17 AM Feature #4164: implement misc UI features (frame options, misc attributes)

Branch 4231b revision 11395 includes @GenericFrame@ changes that complete @next-tab-item@ and @prev-tab-item@ per #41... Greg Shah

03/25/2020

03:42 PM Bug #4589: menu case sensitivity and conversion

> The current changes were checked with testcases/uast/menu tests that passed except 4 known cases:
Roger is worki... Greg Shah

03:33 PM Bug #4589: menu case sensitivity and conversion

The current changes were checked with @testcases/uast/menu@ tests that passed except 4 known cases:... Sergey Ivanovskiy

03:21 PM Bug #4589: menu case sensitivity and conversion

If everything is working so far and there are no regressions, then I am OK with it. Greg Shah

02:32 PM Bug #4589: menu case sensitivity and conversion

Greg, I compared the previous version and the current changes and found that this map used @submenu = names.convert(... Sergey Ivanovskiy

10:36 AM Bug #4589: menu case sensitivity and conversion

Code Review Task Branch 4231b Revision 11387
It seems wrong to use the javaname (even a lowercased one) as a key t... Greg Shah

08:13 AM Bug #4589: menu case sensitivity and conversion

Greg Shah wrote:
> This is not a problem with menu conversion. The skeletons should never be converted. They are s... Hynek Cihlar

08:13 AM Bug #4589: menu case sensitivity and conversion

Hynek Cihlar wrote:
> Sergey, I think I stepped on another error during <large_app> conversion caused by the menu ch... Hynek Cihlar

08:04 AM Bug #4589: menu case sensitivity and conversion

This is not a problem with menu conversion. The skeletons should never be converted. They are supposed to be exclud... Greg Shah

07:53 AM Bug #4589: menu case sensitivity and conversion

Sergey, I think I stepped on another error during <large_app> conversion caused by the menu changes.... Hynek Cihlar

03:20 PM Feature #4393: add TITLE-DCOLOR attribute support

Consider the following facts:
* The reason we store state in a separate config class (e.g. @BaseConfig@) is to hav... Greg Shah

02:41 PM Feature #4393: add TITLE-DCOLOR attribute support

Are @titleBGgColor@ and @titleFgColor@ attributes to be kept with @titleDColor@ and @titleFont@ in @BaseConfig@? Roger Borrello

01:45 PM Feature #4393: add TITLE-DCOLOR attribute support

> Yes I did, but in that method, it is setting frame.config.titleDColor instead of frame.config.titleColor.
There ... Greg Shah

01:34 PM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> Did you see this part in #4393-6:
>
> > @GenericFrame@: Add @getTitleDColor()@, @setTitleDColo... Roger Borrello

12:59 PM Feature #4393: add TITLE-DCOLOR attribute support

Did you see this part in #4393-6:
> @GenericFrame@: Add @getTitleDColor()@, @setTitleDColor(int64 dcolor)@, @setTi... Greg Shah

12:52 PM Feature #4393: add TITLE-DCOLOR attribute support

@GenericFrame@ had a method that looks a little broken:... Roger Borrello

11:45 AM Feature #4393: add TITLE-DCOLOR attribute support

Just implement what is in @TitledElement@. The @ColorSpec@ stuff is not related. Greg Shah

11:44 AM Feature #4393: add TITLE-DCOLOR attribute support

Should there be setters for *FG*, *BG*, and *D* which receive @(ColorSpec color)@? Or just @int64@ and @long@? Roger Borrello

11:21 AM Feature #4393: add TITLE-DCOLOR attribute support

Greg Shah wrote:
> It is recommended anytime you are:
>
> * implementing a method from an interface (which you ar... Roger Borrello

11:17 AM Feature #4393: add TITLE-DCOLOR attribute support

It is recommended anytime you are:
* implementing a method from an interface (which you are doing, right?) OR
* o... Greg Shah

10:58 AM Feature #4393: add TITLE-DCOLOR attribute support

I'm in @BrowseWidget@ and I see some methods with @&#64;Override@ annotation, and some without. What determines wheth... Roger Borrello

09:49 PM Feature #4393: add TITLE-DCOLOR attribute support

Just re-enable the warnings suppression. Greg Shah

03/24/2020

05:48 PM Feature #4393: add TITLE-DCOLOR attribute support

Is the below caused by commented out @options { generateAmbigWarnings = false; }@ ? Was that just for debug?... Roger Borrello

05:06 PM Feature #4393: add TITLE-DCOLOR attribute support

Attached are my incomplete changes for this task. The intention is to *finish the implementation of all the options ... Greg Shah

07:18 AM Bug #4589: menu case sensitivity and conversion

Sergey Ivanovskiy wrote:
> The AST of menu item in the assignment expression #4589-41 missed @<annotation datatype="... Sergey Ivanovskiy

06:07 AM Bug #4589: menu case sensitivity and conversion

The AST of menu item in the assignment expression #4589-41 missed @<annotation datatype="java.lang.String" key="in_wi... Sergey Ivanovskiy

05:37 AM Bug #4589: menu case sensitivity and conversion

This code... Sergey Ivanovskiy

05:12 AM Bug #4589: menu case sensitivity and conversion

The code is still needed to fix ACCELERATOR assignments for menu. I added new test case @test-accelerator-1.p@ into @... Sergey Ivanovskiy

03/23/2020

10:06 AM Bug #4598: define shared menu statement is converted using new name to reference the shared menu

This issue can be reproduced with @./menu/shared_new.p@ and @./menu/shared_use.p@ from the old @testcases/uast@ proje... Sergey Ivanovskiy

09:54 AM Bug #4598 (New): define shared menu statement is converted using new name to reference the shared...

Sergey Ivanovskiy

09:03 AM Bug #4589: menu case sensitivity and conversion

As long as the regression in the other customer's application is also fixed, then any remaining issues can be moved i... Greg Shah

08:53 AM Bug #4589: menu case sensitivity and conversion

I saw the Redmine entries about the regression, and I am grateful you found a better solution. The customer code stil... Roger Borrello

07:51 AM Bug #4589: menu case sensitivity and conversion

Roger Borrello wrote:
> I converted/compiled the customer code with your @4589_6.patch@ updates. Excellent! Did you ... Sergey Ivanovskiy

05:19 AM Bug #4589: menu case sensitivity and conversion

Hynek Cihlar wrote:
> Sergey Ivanovskiy wrote:
> > where @SharedUseMmMenu.class@ should be @SharedNewMmMenu.class@.... Sergey Ivanovskiy

03:55 AM Bug #4589: menu case sensitivity and conversion

Sergey Ivanovskiy wrote:
> where @SharedUseMmMenu.class@ should be @SharedNewMmMenu.class@. @SharedUseMmMenu.java@ i... Hynek Cihlar

03/20/2020

05:17 PM Feature #4164: implement misc UI features (frame options, misc attributes)

Please read the 4GL docs to see what the @FRAME-COL@ function is meant to do. It is the counterpart of @FRAME-ROW@ a... Greg Shah

11:10 AM Feature #4164: implement misc UI features (frame options, misc attributes)

This test is promising:
!frame_loc_attributes_get.png!
Code below. What else can I do to validate?... Roger Borrello

09:01 AM Feature #4164: implement misc UI features (frame options, misc attributes)

Stanislav Lomany wrote:
> Review rev 11377: Roger, you forgot to remove the array need for @eventDrawingBracket@. Yo... Roger Borrello

04:32 AM Feature #4164: implement misc UI features (frame options, misc attributes)

Review rev 11377: Roger, you forgot to remove the array need for @eventDrawingBracket@. You can just write @return br... Stanislav Lomany

03:50 PM Bug #4589: menu case sensitivity and conversion

I converted/compiled the customer code with your @4589_6.patch@ updates. Excellent! Did you remove all your debug @pr... Roger Borrello

03:30 PM Bug #4589: menu case sensitivity and conversion

Roger Borrello wrote:
> Are you getting a conversion error or runtime?
This is the menu generation issue for shared... Sergey Ivanovskiy

03:21 PM Bug #4589: menu case sensitivity and conversion

Committed revision 11381 (4231b) fixed case-insensitive issue for menu identifiers but the shared menu and labels iss... Sergey Ivanovskiy

03:01 PM Bug #4589: menu case sensitivity and conversion

Are you getting a conversion error or runtime?
I've made a change to the @shared_new.p@ to make is easier to run o... Roger Borrello

02:09 PM Bug #4589: menu case sensitivity and conversion

This diff @4589_6.patch@ fixed these tests (see #4589-28)... Sergey Ivanovskiy

01:44 PM Bug #4589: menu case sensitivity and conversion

I encountered a strange annotation generated for menu item... Sergey Ivanovskiy

11:10 AM Bug #4589: menu case sensitivity and conversion

These files... Sergey Ivanovskiy

09:20 AM Bug #4589: menu case sensitivity and conversion

It seems like you are getting close. Keep up the good work. If you need a hand on anything, please post. Roger Borrello

07:01 AM Bug #4589: menu case sensitivity and conversion

The following 3 types of exceptions were encountered with the last attached diff.
These tests... Sergey Ivanovskiy

01:57 AM Bug #4589: menu case sensitivity and conversion

Roger, the last changes fixed your test case but these changes to use low case identifier are not correct. There is a... Sergey Ivanovskiy

03/19/2020

06:16 PM Feature #4164: implement misc UI features (frame options, misc attributes)

Updated per suggestions. Thanks!... Roger Borrello

05:27 PM Feature #4164: implement misc UI features (frame options, misc attributes)

I don't see any value in @BrowseWidget.getBrowseCacheSize()@ when @getNumEntries()@ can call @LogicalTerminal.getBrow... Greg Shah

05:22 PM Feature #4164: implement misc UI features (frame options, misc attributes)

> * @ClientExports: public int getBrowseCacheSize(int browseId);@
> * @BrowseWidget: private int getBrowseCacheSize(... Stanislav Lomany

05:04 PM Feature #4164: implement misc UI features (frame options, misc attributes)

Stanislav Lomany wrote:
> Consider you have a browse:
>
> [...]
>
> DOWN is 5, cache size is 2.
Thanks.
... Roger Borrello

03:32 PM Feature #4164: implement misc UI features (frame options, misc attributes)

Consider you have a browse:... Stanislav Lomany

03:28 PM Feature #4164: implement misc UI features (frame options, misc attributes)

Greg Shah wrote:
> Yes, except you will only have the @browseId@ parameter and you must return an @int@.
What's t... Roger Borrello

02:33 PM Feature #4164: implement misc UI features (frame options, misc attributes)

Yes, except you will only have the @browseId@ parameter and you must return an @int@. Greg Shah

02:24 PM Feature #4164: implement misc UI features (frame options, misc attributes)

Stanislav Lomany wrote:
> > Do we already have an API for this? If not, do we store this value on the client?
>
... Roger Borrello

09:33 AM Feature #4164: implement misc UI features (frame options, misc attributes)

As Stanislav noted above, FWD returning 11 is a BUG. So you don't need to analyze how 11 relates to anything.
> 4... Greg Shah

09:16 AM Feature #4164: implement misc UI features (frame options, misc attributes)

FWD returning 11 leads me to believe there are still 11 rows in the view. Evidence is the slider on the scrollbar has... Roger Borrello

08:39 AM Feature #4164: implement misc UI features (frame options, misc attributes)

Read the comments in @browse_num_entries.p@ and look carefully at the current FWD implementation/comments. Make sure... Greg Shah

05:38 PM Bug #4589: menu case sensitivity and conversion

Sergey Ivanovskiy wrote:
> It is interesting what tests from @testcases/uast/menu/@ should be passed with the trunc... Roger Borrello

04:03 PM Bug #4589: menu case sensitivity and conversion

It is interesting what tests from @testcases/uast/menu/@ should be passed with the trunc version now. I found that m... Sergey Ivanovskiy

09:22 AM Bug #4589: menu case sensitivity and conversion

* 4GL names are always case-insensitive. Java names are always case-sensitive. So we must handle this at conver... Greg Shah

09:02 AM Bug #4589: menu case sensitivity and conversion

Yes, I have similar questions.
I have only this way to collect all @"javaname"@ annotations created for menus, sub... Sergey Ivanovskiy

08:54 AM Bug #4589: menu case sensitivity and conversion

Sergey Ivanovskiy wrote:
> I am investigating logs from my changes in @rules/annotations/menu_scoping.rules@ because... Roger Borrello

05:25 AM Bug #4589: menu case sensitivity and conversion

I am investigating logs from my changes in @rules/annotations/menu_scoping.rules@ because the rules are not clear for... Sergey Ivanovskiy

03:04 AM Bug #4589: menu case sensitivity and conversion

It seems that the sub menu name from its definition should be here, but the converted name can use lower and upper ca... Sergey Ivanovskiy

03/18/2020

07:01 PM Feature #4164: implement misc UI features (frame options, misc attributes)

> What is @NUM-ENTRIES@ attribute hold? Is it the number of entries left?
It is the number of data rows in the vie... Stanislav Lomany

06:51 PM Feature #4164: implement misc UI features (frame options, misc attributes)

Greg Shah wrote:
> Stanislav Lomany wrote:
> > > Stanislav: Please review.
> >
> > The approach in @getNumEntrie... Roger Borrello

05:12 PM Bug #4589: menu case sensitivity and conversion

I updated the testcase to get the new failure:... Roger Borrello

04:46 PM Bug #4589: menu case sensitivity and conversion

I don't think it fixes the issue. The Javaname is now lowercase, but not found. Shouldn't it be a converted name?
... Roger Borrello

04:06 PM Bug #4589: menu case sensitivity and conversion

Is this a complete fix for all known issues seen in this task? Greg Shah

03:58 PM Bug #4589: menu case sensitivity and conversion

Sergey Ivanovskiy wrote:
> Thank you. Please use this patch. It seems that it fixed @menu-item@ according to my test... Roger Borrello

03:41 PM Bug #4589: menu case sensitivity and conversion

Thank you. Please use this patch. It seems that it fixed @menu-item@ according to my test. Sergey Ivanovskiy

03:39 PM Bug #4589: menu case sensitivity and conversion

Sergey Ivanovskiy wrote:
> Roger, I tested this diff with 4231b patched. It couldn't convert @menu-item quititem:SEN... Roger Borrello

03:21 PM Bug #4589: menu case sensitivity and conversion

Roger, I tested this diff with 4231b patched. It couldn't convert @menu-item quititem:SENSITIVE = false.@ properly bu... Sergey Ivanovskiy

11:49 AM Bug #4589: menu case sensitivity and conversion

Sergey Ivanovskiy wrote:
> I think that frame scoping is not connected with this issue because this example 4GL GUI ... Roger Borrello

11:41 AM Bug #4589: menu case sensitivity and conversion

I cannot think of ANY names in the 4GL that are case-sensitive. Our conversion needs to treat all menu, sub-menu and... Greg Shah

11:31 AM Bug #4589: menu case sensitivity and conversion

I think that frame scoping is not connected with this issue because this example 4GL GUI works similarly on the nativ... Sergey Ivanovskiy

11:27 AM Bug #4589: menu case sensitivity and conversion

There isn't a @frame@ scoping the menu in the new case. That is what the customer's code is doing, so we need to unde... Roger Borrello

11:23 AM Bug #4589: menu case sensitivity and conversion

According to this 4GL UI test:... Sergey Ivanovskiy

10:39 AM Bug #4589: menu case sensitivity and conversion

Thank you, I see it now. Sergey Ivanovskiy

10:16 AM Bug #4589: menu case sensitivity and conversion

If you change @thissubmenu:SENSITIVE@ to @ThisSUBMENU:SENSITIVE@ it will generate the correct results. Roger Borrello

10:13 AM Bug #4589: menu case sensitivity and conversion

Compiling of @testcases/uast/submenu_case-sensitive.p@ with 4231b (rev 11376) failed... Sergey Ivanovskiy

10:07 AM Bug #4589 (Closed): menu case sensitivity and conversion

Sergey Ivanovskiy

04:30 PM Feature #3880: enhanced browse 3rd phase of improvements

> Help me understand what you see as the advantage of separating things.
I think you're right that it is a single ... Stanislav Lomany

09:24 PM Feature #3880: enhanced browse 3rd phase of improvements

I assume that the layout parameters for use in both the screen/display and the reports.
Why do we want to keep thi... Greg Shah

03/17/2020

05:42 PM Feature #3880: enhanced browse 3rd phase of improvements

All kinds of colors and fonts, column width, order and visibility, row height. Stanislav Lomany

05:39 PM Feature #3880: enhanced browse 3rd phase of improvements

What are examples of "layout parameters"? Greg Shah

05:29 PM Feature #3880: enhanced browse 3rd phase of improvements

Greg, I have some conceptual questions about report-related enhanced parameters. Enhanced parameters can be divided i... Stanislav Lomany

03:44 PM Feature #2683: reverse proxy implementation

> > It is used only to generate mapping on the fly for new connected broker client.
>
> Is it used for anything ex... Sergey Ivanovskiy

03:31 PM Feature #2683: reverse proxy implementation

> It is used only to generate mapping on the fly for new connected broker client.
Is it used for anything external... Greg Shah

03:22 PM Feature #2683: reverse proxy implementation

I rechecked if Jetty implements the reverse proxy support for wss connections and found that https://webtide.com/http... Sergey Ivanovskiy

02:29 PM Feature #2683: reverse proxy implementation

This hosts file is related only to the network configuration. It is used only to generate mapping on the fly for new ... Sergey Ivanovskiy

01:45 PM Feature #2683: reverse proxy implementation

> Should I change it to use the directory?
Is there any advantage to leaving it as a separate file? Greg Shah

01:02 PM Feature #2683: reverse proxy implementation

Committed revision 11374 (4231b) fixed 1), 2), 4) and 5). The hosts file is used for mapping unique pairs of host and... Sergey Ivanovskiy

08:37 AM Feature #2683: reverse proxy implementation

Code Review Task Branch 4231b Revision 11372
Overall, it is a nicer implementation to have the @HostsManager@.
... Greg Shah

07:51 AM Feature #2683: reverse proxy implementation

Constantin Asofiei wrote:
> Sergey Ivanovskiy wrote:
> > Yes, it looks like the old issue documented in @WebClientP... Sergey Ivanovskiy

11:43 AM Bug #3690: BUTTON widget reaction on SPACEBAR and ENTER keys

It seems the changes in 4231b Revision 11372 are good. Sergey Ivanovskiy

11:11 AM Bug #3690: BUTTON widget reaction on SPACEBAR and ENTER keys

Code Review Task Branch 4231b Revision 11372
The changes seem good to me.
Sergey: Any objections? Greg Shah

10:54 AM Bug #3690: BUTTON widget reaction on SPACEBAR and ENTER keys

Committed fix in 4231b/rev. 11373. The changes were minimal - only the space-bar generates additional released events. Adrian Lungu

06:11 AM Bug #3690: BUTTON widget reaction on SPACEBAR and ENTER keys

After checking the @GUIKeyboardReader@, I see there is the following code which triggers that additional @keyups@:
<... Adrian Lungu

05:16 AM Bug #3690: BUTTON widget reaction on SPACEBAR and ENTER keys

Now I see that this behavior can't be reproduced on Chrome. Maybe it is a bug in Firefox regarding long key presses? Adrian Lungu

05:10 AM Bug #3690: BUTTON widget reaction on SPACEBAR and ENTER keys

Greg Shah wrote:
> I don't want to send a large number of additional events up to Java if we only need a very small ... Adrian Lungu

11:39 AM Feature #4174: implement calendar control/dtpicker OCX replacement

The custom format is implemented exactly according to this documentations https://docs.oracle.com/javase/tutorial/i18... Sergey Ivanovskiy

11:16 AM Feature #4174: implement calendar control/dtpicker OCX replacement

I think the @VALUE-CHANGED@ event and the custom beginning day of week will be needed.
> but the custom format the... Greg Shah

10:46 AM Feature #4174: implement calendar control/dtpicker OCX replacement

Working to improve the documentation of calendar. The current version of @Calendar@ can convert @DateTimeValue@ value... Sergey Ivanovskiy

09:29 AM Feature #4174: implement calendar control/dtpicker OCX replacement

I see that there is a good start on the documentation. Thank you!
I sent the link to some customers for review. ... Greg Shah

03/16/2020

01:15 PM Bug #3690: BUTTON widget reaction on SPACEBAR and ENTER keys

>> Even though the KEY_RELEASED event is not a progress event, it should be generated in order to properly reproduce ... Greg Shah

12:29 PM Bug #3690: BUTTON widget reaction on SPACEBAR and ENTER keys

Adrian Lungu wrote:
> Yet, I am not sure how the key events are handled in the web driver. But from this slim invest... Sergey Ivanovskiy

12:12 PM Bug #3690: BUTTON widget reaction on SPACEBAR and ENTER keys

It seems that @keyup@ browser's event means that a key is released. Is it correct to change this part of the js web c... Sergey Ivanovskiy

10:39 AM Bug #3690: BUTTON widget reaction on SPACEBAR and ENTER keys

Adrian Lungu wrote:
> I recently found a bug related to the issue here - regarding the web interface. The web driver... Sergey Ivanovskiy

10:32 AM Bug #3690: BUTTON widget reaction on SPACEBAR and ENTER keys

Whatever solution is created, please minimize the number of events forwarded to the FWD client (Java side). Optimall... Greg Shah

10:21 AM Bug #3690: BUTTON widget reaction on SPACEBAR and ENTER keys

I recently found a bug related to the issue here - regarding the web interface. The web driver doesn't emit release e... Adrian Lungu

03/13/2020

03:45 PM Feature #4164: implement misc UI features (frame options, misc attributes)

Task branch 4335a was merged to trunk as revision 11345. Greg Shah

03:43 PM Feature #4199: implement a data visualization widget which can be used for a wide range of data-b...

Task branch 4335a was merged to trunk as revision 11345. Greg Shah

03:42 PM Feature #4472 (Closed): Allow pause for a fraction of a second

Task branch 4335a was merged to trunk as revision 11345. Greg Shah

03:41 PM Feature #4170: add drag and drop support to treeview widget

Task branch 4335a was merged to trunk as revision 11345.
See #4170-21 for the remaining work. Greg Shah