Activity
From 03/09/2020 to 04/07/2020
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...
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.
- 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...
- 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 ... - 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... - 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... - 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... - 09:34 AM Feature #4164: implement misc UI features (frame options, misc attributes)
- I had a testcase:...
- 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...
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... - 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.
04/01/2020
- 06:54 PM Bug #4604 (New): DCOLOR on FRAME or BROWSE TITLE phrase results in title not being displayed
- 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...
- 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....
- 10:53 AM Feature #4393: add TITLE-DCOLOR attribute support
- h2. Browse TITLE Attribute Defaults (before/after realization)
|_. |_. ChUI (bef) |_. ChUI (aft) |_. GUI (... - 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.....
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...
- 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... - 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...
- 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... - 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... - 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... - 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'... - 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...
- 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...
- 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_... - 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? ... - 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... - 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...
- 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...
- 12:14 PM Feature #4393: add TITLE-DCOLOR attribute support
- Greg Shah wrote:
> What happens in trunk?... - 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...
- 11:55 AM Feature #4393: add TITLE-DCOLOR attribute support
- Compilation of the testcase:...
- 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... - 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...
- 11:31 AM Feature #4393: add TITLE-DCOLOR attribute support
- Code Review updates made:...
- 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... - 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...
- 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 ... - 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...
- 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... - 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... - 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*@... - 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... - 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@.
03/30/2020
- 04:14 PM Feature #4393: add TITLE-DCOLOR attribute support
- Committed updates:...
- 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 *... - 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*:
<... - 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...
- 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... - 02:04 PM Feature #4393: add TITLE-DCOLOR attribute support
- What do you mean by "build"? It parsed fine.
- 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...
- 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...
- 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 ...
- 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(... - 12:25 PM Feature #4393: add TITLE-DCOLOR attribute support
- I see the missing "new" in there.
- 12:20 PM Feature #4393: add TITLE-DCOLOR attribute support
- Greg Shah wrote:
> You did not reverse the logic.
Like this?... - 11:37 AM Feature #4393: add TITLE-DCOLOR attribute support
- You did not reverse the logic.
- 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... - 11:19 AM Feature #4393: add TITLE-DCOLOR attribute support
- Just reverse the logic. The @long@ version calls the wrapper version.
- 10:49 AM Feature #4393: add TITLE-DCOLOR attribute support
- The previous convention for the setters had the @int64@ versions calling the @long@ versions:...
- 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 ...
- 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 ...
- 09:20 AM Feature #4393: add TITLE-DCOLOR attribute support
- Yes.
- 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 ...
03/27/2020
- 06:53 PM Feature #4393: add TITLE-DCOLOR attribute support
- Cleaned up and checked in:...
- 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...
- 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...
- 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... - 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... - 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... - 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... - 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.... - 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....
- 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... - 03:41 PM Feature #4393: add TITLE-DCOLOR attribute support
- Updates for frames in *4231b-11400*.
- 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... - 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...
- 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... - 02:17 PM Feature #4393: add TITLE-DCOLOR attribute support
- > TitiledElement - leave alone
Yes
> CommonFrame - currently an inteface that extends TitledElement. No getters... - 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... - 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...
- 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... - 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,... - 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 ... - 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...
- 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... - 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@.
- 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...
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... - 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 ...
- 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 @"{&window-system}" ...
- 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,...
- 04:48 PM Feature #4393: add TITLE-DCOLOR attribute support
- Correct.
- 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...
- 03:52 PM Feature #4393: add TITLE-DCOLOR attribute support
- Nevermind. Found the sources to our @readOnlyError@.
- 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... - 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...
- 02:57 PM Feature #4393: add TITLE-DCOLOR attribute support
- No, I don't think so.
- 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... - 02:38 PM Feature #4393: add TITLE-DCOLOR attribute support
- Why would we want it to be @extends@?
- 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 @... - 02:21 PM Feature #4393: add TITLE-DCOLOR attribute support
- Did you add it to @FrameWidget@?
- 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 ... - 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... - 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... - 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... - 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...
- 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... - 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... - 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... - 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...
- 10:38 AM Feature #4393: add TITLE-DCOLOR attribute support
- Using this to ef with @titleDColor@:...
- 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 ...
- 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.
... - 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 ... - 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... - 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... ...
- 09:34 PM Feature #4393: add TITLE-DCOLOR attribute support
- The documentation for Attributes:Frames for writing/assignment says...
_in ChUI:_
_before realization, all ... - 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...
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... - 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:...
- 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.
- 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(...
- 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... - 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... - 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... - 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...
- 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....
- 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... - 02:41 PM Feature #4393: add TITLE-DCOLOR attribute support
- Are @titleBGgColor@ and @titleFgColor@ attributes to be kept with @titleDColor@ and @titleFont@ in @BaseConfig@?
- 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 ... - 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... - 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... - 12:52 PM Feature #4393: add TITLE-DCOLOR attribute support
- @GenericFrame@ had a method that looks a little broken:...
- 11:45 AM Feature #4393: add TITLE-DCOLOR attribute support
- Just implement what is in @TitledElement@. The @ColorSpec@ stuff is not related.
- 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@?
- 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... - 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... - 10:58 AM Feature #4393: add TITLE-DCOLOR attribute support
- I'm in @BrowseWidget@ and I see some methods with @@Override@ annotation, and some without. What determines wheth...
- 09:49 PM Feature #4393: add TITLE-DCOLOR attribute support
- Just re-enable the warnings suppression.
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?...
- 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 ...
- 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="... - 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...
- 05:37 AM Bug #4589: menu case sensitivity and conversion
- This code...
- 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 @...
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...
- 09:54 AM Bug #4598 (New): define shared menu statement is converted using new name to reference the shared...
- 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...
- 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...
- 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 ... - 05:19 AM Bug #4589: menu case sensitivity and conversion
- Hynek Cihlar wrote:
> Sergey Ivanovskiy wrote:
> > where @SharedUseMmMenu.class@ should be @SharedNewMmMenu.class@.... - 03:55 AM Bug #4589: menu case sensitivity and conversion
- Sergey Ivanovskiy wrote:
> where @SharedUseMmMenu.class@ should be @SharedNewMmMenu.class@. @SharedUseMmMenu.java@ i...
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...
- 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?... - 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... - 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...
- 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...
- 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... - 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...
- 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... - 02:09 PM Bug #4589: menu case sensitivity and conversion
- This diff @4589_6.patch@ fixed these tests (see #4589-28)...
- 01:44 PM Bug #4589: menu case sensitivity and conversion
- I encountered a strange annotation generated for menu item...
- 11:10 AM Bug #4589: menu case sensitivity and conversion
- These files...
- 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.
- 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... - 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...
03/19/2020
- 06:16 PM Feature #4164: implement misc UI features (frame options, misc attributes)
- Updated per suggestions. Thanks!...
- 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...
- 05:22 PM Feature #4164: implement misc UI features (frame options, misc attributes)
- > * @ClientExports: public int getBrowseCacheSize(int browseId);@
> * @BrowseWidget: private int getBrowseCacheSize(... - 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.
... - 03:32 PM Feature #4164: implement misc UI features (frame options, misc attributes)
- Consider you have a browse:...
- 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... - 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@.
- 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?
>
... - 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... - 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...
- 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...
- 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... - 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...
- 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...- 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... - 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... - 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...
- 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...
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... - 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... - 05:12 PM Bug #4589: menu case sensitivity and conversion
- I updated the testcase to get the new failure:...
- 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?
... - 04:06 PM Bug #4589: menu case sensitivity and conversion
- Is this a complete fix for all known issues seen in this task?
- 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... - 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.
- 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... - 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...
- 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 ... - 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...
- 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...
- 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...
- 11:23 AM Bug #4589: menu case sensitivity and conversion
- According to this 4GL UI test:...
- 10:39 AM Bug #4589: menu case sensitivity and conversion
- Thank you, I see it now.
- 10:16 AM Bug #4589: menu case sensitivity and conversion
- If you change @thissubmenu:SENSITIVE@ to @ThisSUBMENU:SENSITIVE@ it will generate the correct results.
- 10:13 AM Bug #4589: menu case sensitivity and conversion
- Compiling of @testcases/uast/submenu_case-sensitive.p@ with 4231b (rev 11376) failed...
- 10:07 AM Bug #4589 (Closed): menu case sensitivity and conversion
- 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 ... - 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...
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.
- 05:39 PM Feature #3880: enhanced browse 3rd phase of improvements
- What are examples of "layout parameters"?
- 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...
- 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... - 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... - 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...
- 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 ...
- 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? - 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...
- 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@.
... - 07:51 AM Feature #2683: reverse proxy implementation
- Constantin Asofiei wrote:
> Sergey Ivanovskiy wrote:
> > Yes, it looks like the old issue documented in @WebClientP... - 11:43 AM Bug #3690: BUTTON widget reaction on SPACEBAR and ENTER keys
- It seems the changes in 4231b Revision 11372 are good.
- 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? - 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.
- 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@:
<... - 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?
- 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 ... - 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...
- 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... - 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...
- 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. ...
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 ...
- 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... - 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...
- 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... - 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...
- 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...
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.
- 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.
- 03:42 PM Feature #4472 (Closed): Allow pause for a fraction of a second
- Task branch 4335a was merged to trunk as revision 11345.
- 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.
03/11/2020
- 04:05 PM Feature #4174: implement calendar control/dtpicker OCX replacement
- Sergey: Please write up the documentation for this widget in "CALENDAR Widget and OCX Replacement":/projects/p2j/wiki...
- 02:54 PM Feature #2683: reverse proxy implementation
- We don't have documentation yet. External widget support is only needed in special cases where the code cannot be pl...
- 02:25 PM Feature #2683: reverse proxy implementation
- Greg, please could you provide me with information about external widget support? Are there ready examples?
- 02:18 PM Feature #2683: reverse proxy implementation
- Although it should be safe, I prefer to avoid changes in 4335a. Please put your changes into 4231b after it is rebas...
03/10/2020
- 09:10 AM Feature #2683: reverse proxy implementation
- Sergey Ivanovskiy wrote:
> I can't use 4231b for #2683-73 changes because these code exist only in 4335a. These chan... - 08:42 AM Feature #2683: reverse proxy implementation
- I can't use 4231b for #2683-73 changes because these code exist only in 4335a. These changes are only to remove logs ...
- 04:34 AM Feature #4182: KERNEL32:SLEEP() emulation
- Roger Borrello wrote:
> Constantin Asofiei wrote:
> > Roger, I was missing the .ext-hints. After this, your sleep t... - 03:56 AM Feature #4170: add drag and drop support to treeview widget
- Roger Borrello wrote:
> I see many new keywords added related to @node@ and @nodes@. Were they a result of this issu...
03/09/2020
- 06:24 PM Feature #2683: reverse proxy implementation
- Sergey Ivanovskiy wrote:
> Yes, it looks like the old issue documented in @WebClientProtocol@
OK, please add javado... - 06:23 PM Feature #2683: reverse proxy implementation
- Greg Shah wrote:
> I don't have a problem with the patch, but Constantin should review.
I'm OK with the patch.
>... - 05:38 PM Feature #2683: reverse proxy implementation
- Yes, it looks like the old issue documented in @WebClientProtocol@...
- 04:59 PM Feature #2683: reverse proxy implementation
- It is interesting if we should remove logs here in @ThinClient.initializePost@ because it would block the web client?...
- 04:31 PM Feature #2683: reverse proxy implementation
- It seems that this hanging issue is reproduced only when I tried to connect @https://localhost/gui@ and the Apache pr...
- 04:04 PM Feature #2683: reverse proxy implementation
- The hanging issue has this thread context:
1) @KeyReader@ thread is in this state... - 03:41 PM Feature #2683: reverse proxy implementation
- No, the broker manager patch is related to the case when the broker manager is not used but clients to ports map is r...
- 03:28 PM Feature #2683: reverse proxy implementation
- I don't have a problem with the patch, but Constantin should review.
> It requires to set up a broker configuratio... - 03:14 PM Feature #2683: reverse proxy implementation
- I tested this Apache Reverse proxy configuration with this branch 4335a using Hotel Gui application. The current code...
- 05:15 PM Feature #4170: add drag and drop support to treeview widget
- I see many new keywords added related to @node@ and @nodes@. Were they a result of this issue? I noticed that @node-e...
- 03:49 PM Feature #4182: KERNEL32:SLEEP() emulation
- Constantin Asofiei wrote:
> Roger, I was missing the .ext-hints. After this, your sleep testcase in #4182-6 works fi... - 03:41 PM Feature #4182: KERNEL32:SLEEP() emulation
- Roger, I was missing the .ext-hints. After this, your sleep testcase in #4182-6 works fine.
Please check your clas... - 03:27 PM Feature #4182: KERNEL32:SLEEP() emulation
- Roger, sorry, I can't make anything to work (@gethostbyname@, @GetComputerNameA@). I don't recall how this was suppo...
- 02:52 PM Feature #4182: KERNEL32:SLEEP() emulation
- Eric Faulhaber wrote:
> Sorry, no. If this test case is one of mine, I don't remember it.
>
> Are you getting a p... - 02:48 PM Feature #4182: KERNEL32:SLEEP() emulation
- Sorry, no. If this test case is one of mine, I don't remember it.
Are you getting a particular error, or just no r... - 02:39 PM Feature #4182: KERNEL32:SLEEP() emulation
- Eric, did you ever test the @myconn1.p@ testcase under FWD and Windows? I'm trying to get my @Kernel32:Sleep@ functio...
- 02:35 PM Feature #4182: KERNEL32:SLEEP() emulation
- I made a simplified version of @uast/myconn1.p@:...
- 02:12 PM Feature #4182: KERNEL32:SLEEP() emulation
- I made sure my pathing was correct by validating that @directory.xml@ contains:...
- 01:26 PM Feature #4182: KERNEL32:SLEEP() emulation
- @./src/com/goldencode/testcases/name_map.xml@ looks good... although perhaps the pathing is misconfigured?...
- 01:21 PM Feature #4182: KERNEL32:SLEEP() emulation
- Roger Borrello wrote:
> When @getAbsoluteName(Object referent)@ is called, @jname="com.goldencode.testcases.windows_... - 01:16 PM Feature #4182: KERNEL32:SLEEP() emulation
- When @getAbsoluteName(Object referent)@ is called, @jname="com.goldencode.testcases.windows_emulation.Kernel32Sleep"@...
- 12:50 PM Feature #4182: KERNEL32:SLEEP() emulation
- Constantin Asofiei wrote:
> Roger Borrello wrote:
> > I hit a breakpoint in @Kernel32Sleep.lambda$execute$1@ so I'l... - 12:27 PM Feature #4182: KERNEL32:SLEEP() emulation
- Roger Borrello wrote:
> I hit a breakpoint in @Kernel32Sleep.lambda$execute$1@ so I'll try to find my way from here.... - 12:25 PM Feature #4182: KERNEL32:SLEEP() emulation
- Constantin Asofiei wrote:
> Roger Borrello wrote:
> > I was expecting to hit my breakpoint, but I must be missing s... - 12:19 PM Feature #4182: KERNEL32:SLEEP() emulation
- Roger Borrello wrote:
> I was expecting to hit my breakpoint, but I must be missing something.
You need to attach t... - 12:17 PM Feature #4182: KERNEL32:SLEEP() emulation
- I've been trying to catch this in the debugger, but since this is my first rodeo debugging client or server runtime, ...
Also available in: Atom