FWD » Core Development » User Interface

09/24/2025

03:52 PM Bug #10602: Fix scrolling in browse on Swing after switching to accumulated scrolling

I'm OK with the update, but please hide the calculations unnecessary for the Swing case under appropriate condition:
... Stanislav Lomany

08:08 AM Bug #10602 (Review): Fix scrolling in browse on Swing after switching to accumulated scrolling

Șerban Bursuc

08:08 AM Bug #10602: Fix scrolling in browse on Swing after switching to accumulated scrolling

Swing gives the same scroll amounts for both smooth and non smooth scrolling unlike JavaScript, which makes the accum... Șerban Bursuc

06:53 AM Bug #9738 (Test): Infinite loop/memory leak caused by ENTRY trigger on current-window

Branch 9738a was merged to trunk rev 16183 and archived. Razvan-Nicolae Chichirau

06:39 AM Bug #9738 (Merge Pending): Infinite loop/memory leak caused by ENTRY trigger on current-window

Razvan-Nicolae Chichirau wrote:
> The testing plan passed. Can we merge this?
Yes, indeed. You may merge 9738a now.
Radu Apetrii

06:37 AM Bug #9738: Infinite loop/memory leak caused by ENTRY trigger on current-window

Razvan-Nicolae Chichirau wrote:
> I'll do:
> * Smoke-tests on 2 customer + demo apps on web
> * check some UI rela... Razvan-Nicolae Chichirau

03:59 AM Bug #10485: App freeze due to lack of synchronization

Hotel GUI smoke testing also passed with the last rev of 10485a. Paula Păstrăguș

03:19 AM Bug #10485 (Review): App freeze due to lack of synchronization

The testing phase uncovered a small issue. When the connection was interrupted during the upload test, the behavior w... Paula Păstrăguș

09/23/2025

01:51 PM Bug #10485 (Internal Test): App freeze due to lack of synchronization

Code review 10485a. The changes look good. Hynek Cihlar

08:58 AM Bug #10485: App freeze due to lack of synchronization

Addressed review as rev *16140*. Paula Păstrăguș

05:56 AM Bug #10485: App freeze due to lack of synchronization

Paula Păstrăguș wrote:
> Okay, so I can take the status code (per https://datatracker.ietf.org/doc/html/rfc6455#sect... Hynek Cihlar

04:03 AM Bug #10485: App freeze due to lack of synchronization

Okay, so I can take the status code (per https://datatracker.ietf.org/doc/html/rfc6455#section-7.4.1) and map it to a... Paula Păstrăguș

06:55 AM Bug #9738: Infinite loop/memory leak caused by ENTRY trigger on current-window

I'll do:
* Smoke-tests on 2 customer + demo apps on web
* check some UI related tests
* verify the unit tests from... Razvan-Nicolae Chichirau

06:47 AM Bug #9738 (Internal Test): Infinite loop/memory leak caused by ENTRY trigger on current-window

Code review 9738a. The changes look good. Surprising we didn't notice this earlier. Please go ahead and regression test. Hynek Cihlar

03:38 AM Bug #9738 (Review): Infinite loop/memory leak caused by ENTRY trigger on current-window

Commited on *9738a/rev. 16177*.
Hynek: Please review. Razvan-Nicolae Chichirau

09/22/2025

09:45 AM Bug #10485: App freeze due to lack of synchronization

Paula Păstrăguș wrote:
> I think @Test failed@ on its own is enough, no need to mention @connection error@, since th... Hynek Cihlar

09:22 AM Bug #10485: App freeze due to lack of synchronization

I’ve attached the patch. Let me know if there’s anything else I should adjust before I commit everything together. Th... Paula Păstrăguș

09:16 AM Bug #10485: App freeze due to lack of synchronization

I noticed that I was assigning null to the handlers *too early*. What happened is that the cleanup ran right after ca... Paula Păstrăguș

09:14 AM Bug #10485: App freeze due to lack of synchronization

I think @Test failed@ on its own is enough, no need to mention @connection error@, since the exact reason isn’t that ... Paula Păstrăguș

09:08 AM Bug #10485: App freeze due to lack of synchronization

Hynek Cihlar wrote:
> Paula Păstrăguș wrote:
> > Hynek Cihlar wrote:
> > > The connection error is output to the t... Hynek Cihlar

09:07 AM Bug #10485: App freeze due to lack of synchronization

Paula Păstrăguș wrote:
> Hynek Cihlar wrote:
> > The connection error is output to the test results from the websoc... Hynek Cihlar

08:44 AM Bug #10485: App freeze due to lack of synchronization

Also, no error is raised when the internet connection is interrupted and then restored. I’ll double-check, but as I r... Paula Păstrăguș

08:41 AM Bug #10485: App freeze due to lack of synchronization

Hynek Cihlar wrote:
> The connection error is output to the test results from the websocket's close handler. Should ... Paula Păstrăguș

08:37 AM Bug #10485: App freeze due to lack of synchronization

Code review.
When logging the websocket close/error event, do not log just the @reason@ property, but the whole ob... Hynek Cihlar

01:38 AM Bug #10485: App freeze due to lack of synchronization

Committed as rev *16139*.
Hynek, please review. Paula Păstrăguș

02:56 AM Bug #9738 (WIP): Infinite loop/memory leak caused by ENTRY trigger on current-window

Web GUI is stuck in @p2j.canvas_renderer.js -> replay()@ due to using the same variable for looping:... Razvan-Nicolae Chichirau

09/19/2025

02:42 PM Bug #10485: App freeze due to lack of synchronization

Paula Păstrăguș wrote:
> I’d like your input: should we leave the behavior as it is, or also display the @Test abort... Hynek Cihlar

09:59 AM Bug #10485: App freeze due to lack of synchronization

Hynek, I’ve attached two videos to illustrate the network test behavior when the internet connection is interrupted a... Paula Păstrăguș

04:30 AM Bug #10485: App freeze due to lack of synchronization

Hynek Cihlar wrote:
> Please test the case when the network connection is briefly interrupted. It is OK for the test... Paula Păstrăguș

08:22 AM Bug #10602: Fix scrolling in browse on Swing after switching to accumulated scrolling

From #9986 we discovered smooth scrolling, a feature available for certain mice hardware that can be enabled through ... Șerban Bursuc

08:18 AM Bug #10602 (Test): Fix scrolling in browse on Swing after switching to accumulated scrolling

Șerban Bursuc

09/18/2025

01:13 PM Bug #10485: App freeze due to lack of synchronization

Paula Păstrăguș wrote:
> Addressed points 2 and 3 in rev *16138*.
>
> I double checked the first point, the sock... Hynek Cihlar

04:01 AM Bug #9985: Incorrect text width calculation when metrics are not cached

I’ve been thinking about improving the way we handle text width measurements.
Right now we rely on Swing metrics p... Paula Păstrăguș

09/17/2025

03:40 PM Bug #10593: SPREADSHEET: Keikai backend sends huge amount of data to the frontend

This issue was first identified in #10210. Keikai transmits an excessive amount of data to the frontend, even for min... Hynek Cihlar

03:37 PM Bug #10593 (New): SPREADSHEET: Keikai backend sends huge amount of data to the frontend

Hynek Cihlar

03:26 AM Bug #10485 (Review): App freeze due to lack of synchronization

Addressed points 2 and 3 in rev *16138*.
I double checked the first point, the socket is already closed at the en... Paula Păstrăguș

09/16/2025

02:46 PM Bug #10485 (WIP): App freeze due to lack of synchronization

Code review 10485a.
Overall the changes look fine. Just a few points.
* The socket should be closed after the t... Hynek Cihlar

10:46 AM Bug #10485: App freeze due to lack of synchronization

Here's an image with the results from my machine:
!netResults.png! Paula Păstrăguș

10:28 AM Bug #10485 (Review): App freeze due to lack of synchronization

Committed as rev *16137*: Added dedicated socket endpoint for network performance tests.
Hynek, Sergey please review. Paula Păstrăguș

09/11/2025

06:43 AM Bug #10485: App freeze due to lack of synchronization

The network test is only a diagnostic tool. Real users will almost never use it. The websocket should only ever be ... Greg Shah

03:51 AM Bug #10485 (WIP): App freeze due to lack of synchronization

Below is how I plan to approach this, along with a few questions to clarify some key points:
* I’ve created a new en... Paula Păstrăguș

09/10/2025

07:16 AM Bug #10323 (Closed): FILL-IN in GUI Mode: Issues with Displaying Empty Screen Value

Hynek Cihlar

06:42 AM Bug #10323: FILL-IN in GUI Mode: Issues with Displaying Empty Screen Value

Hynek Cihlar wrote:
> Please merge 10323a to trunk.
Done at rev. 16157. Vladimir Tsichevski

04:56 AM Bug #10323 (Merge Pending): FILL-IN in GUI Mode: Issues with Displaying Empty Screen Value

Please merge 10323a to trunk. Hynek Cihlar

07:16 AM Bug #10512 (Closed): REGRESSION: In ChUI, all RADIO-SET items are positioned at the same location

Hynek Cihlar

07:13 AM Bug #10512: REGRESSION: In ChUI, all RADIO-SET items are positioned at the same location

Hynek Cihlar wrote:
> Please merge 10512a after 10323a.
Done at rev. 16158. Vladimir Tsichevski

04:57 AM Bug #10512 (Merge Pending): REGRESSION: In ChUI, all RADIO-SET items are positioned at the same l...

Please merge 10512a after 10323a. Hynek Cihlar

09/09/2025

07:20 AM Bug #10485: App freeze due to lack of synchronization

Yes, I agree. Sergey Ivanovskiy

09/08/2025

11:03 AM Bug #10485: App freeze due to lack of synchronization

Sergey raises a valid point. Adding synchronization logic to cover up a flawed design is not a good approach.
The ... Hynek Cihlar

10:31 AM Bug #10485: App freeze due to lack of synchronization

Let me explain my point of view. Please look at the usage of @MSG_GET_WEB_BASIC_PALETTE@ of @WebClientProtocol@.
@Gu... Sergey Ivanovskiy

09:36 AM Bug #10485: App freeze due to lack of synchronization

Paula Păstrăguș wrote:
> Committed as rev *16136*.
>
> Please review.
The changes look good to me. Sergey, wha... Hynek Cihlar

08:40 AM Bug #10485: App freeze due to lack of synchronization

Committed as rev *16136*.
Please review. Paula Păstrăguș

08:13 AM Bug #10485: App freeze due to lack of synchronization

Paula Păstrăguș wrote:
> Yes, sendMessage is only called from the push worker thread, so in practice it doesn’t real... Greg Shah

07:55 AM Bug #10485: App freeze due to lack of synchronization

Yes, sendMessage is only called from the push worker thread, so in practice it doesn’t really make sense to also guar... Paula Păstrăguș

07:48 AM Bug #10485: App freeze due to lack of synchronization

Paula Păstrăguș wrote:
> I’ve attached a patch that switches the implementation to use a @ReentrantLock@. If this ap... Sergey Ivanovskiy

07:46 AM Bug #10485: App freeze due to lack of synchronization

Hynek Cihlar wrote:
> Shouldn't @sendString@ be protected, too?
Indeed, it should have been protected too. I ov... Paula Păstrăguș

07:08 AM Bug #10485: App freeze due to lack of synchronization

Paula Păstrăguș wrote:
> I’ve attached a patch that switches the implementation to use a @ReentrantLock@. If this ap... Hynek Cihlar

06:35 AM Bug #10485 (Review): App freeze due to lack of synchronization

I’ve attached a patch that switches the implementation to use a @ReentrantLock@. If this approach looks good, I’ll co... Paula Păstrăguș

07:08 AM Bug #10530: Move cursor change logic from JS client to the FWD client

Please move this task to User Interface if possible... Șerban Bursuc

07:07 AM Bug #10530 (WIP): Move cursor change logic from JS client to the FWD client

Since #8088 it was decided to remove any cursor changes from the JS client to prevent unwanted cursor behavior.
Mo... Șerban Bursuc

07:02 AM Bug #10530 (WIP): Move cursor change logic from JS client to the FWD client

Șerban Bursuc

09/05/2025

04:03 AM Bug #9916 (Closed): FILL-IN: date format: incorrect condition for triggering normalization of thi...

Hynek Cihlar

09/04/2025

01:38 PM Bug #9916: FILL-IN: date format: incorrect condition for triggering normalization of third component

Hynek Cihlar wrote:
> Are there any related customer test scenarios?
Nothing I can think of.
> Should the issue b... Vladimir Tsichevski

12:15 PM Bug #10323: FILL-IN in GUI Mode: Issues with Displaying Empty Screen Value

Hynek Cihlar wrote:
> ChUI tests passed.
We can proceed with the merge now. Vladimir Tsichevski

06:29 AM Bug #10323: FILL-IN in GUI Mode: Issues with Displaying Empty Screen Value

ChUI tests passed. Hynek Cihlar

12:09 PM Bug #10266: Incorrect Input Handling in Grouped CHARACTER FILL-INs

h2. ChUI Rendering Issue
Run this program in CHARACTER (ChUI) mode:... Vladimir Tsichevski

10:49 AM Bug #10266: Incorrect Input Handling in Grouped CHARACTER FILL-INs

**Issue**
# Run the program in GUI mode.
# Enter text into all @FILL-IN@ widgets within the group.
# In any grou... Vladimir Tsichevski

10:13 AM Bug #10512: REGRESSION: In ChUI, all RADIO-SET items are positioned at the same location

Vladimir Tsichevski wrote:
> I will register another task for #10512-7.
Done. The issue #10521.
I think, we ca... Vladimir Tsichevski

07:39 AM Bug #10512: REGRESSION: In ChUI, all RADIO-SET items are positioned at the same location

Hynek Cihlar wrote:
> ChUI regression tests passed. Vladimir, do you plan some more regression testing?
No. I wil... Vladimir Tsichevski

02:36 AM Bug #10512: REGRESSION: In ChUI, all RADIO-SET items are positioned at the same location

ChUI regression tests passed. Vladimir, do you plan some more regression testing? Hynek Cihlar

10:12 AM Bug #10521: RADIO-SET Rendering in ChUI Issues

h2. Scenario... Vladimir Tsichevski

08:11 AM Bug #10521 (Test): RADIO-SET Rendering in ChUI Issues

Vladimir Tsichevski

09/03/2025

05:16 PM Bug #10323: FILL-IN in GUI Mode: Issues with Displaying Empty Screen Value

Vladimir Tsichevski wrote:
> Hynek Cihlar wrote:
> > Vladimir Tsichevski wrote:
> > > Hynek Cihlar wrote:
> > > >... Hynek Cihlar

04:08 PM Bug #10323: FILL-IN in GUI Mode: Issues with Displaying Empty Screen Value

Hynek Cihlar wrote:
> Vladimir Tsichevski wrote:
> > Hynek Cihlar wrote:
> > > Code review 10323a. The changes loo... Vladimir Tsichevski

05:25 AM Bug #10512: REGRESSION: In ChUI, all RADIO-SET items are positioned at the same location

Hynek Cihlar wrote:
> Code review 10512a. The changes are good. I'm running ChUI regression tests. Should the issue ... Vladimir Tsichevski

03:52 AM Bug #10512 (Internal Test): REGRESSION: In ChUI, all RADIO-SET items are positioned at the same l...

Code review 10512a. The changes are good. I'm running ChUI regression tests. Should the issue be set to 100% done? Hynek Cihlar

09/02/2025

03:20 PM Bug #9916 (Test): FILL-IN: date format: incorrect condition for triggering normalization of third...

Are there any related customer test scenarios? Should the issue be moved to Test state? Hynek Cihlar

02:23 PM Bug #9916: FILL-IN: date format: incorrect condition for triggering normalization of third component

Hynek Cihlar wrote:
> Vladimir Tsichevski wrote:
> > I rebased the 9916a to trunk rev. 16141. Can we merge now?
> ... Vladimir Tsichevski

01:57 PM Bug #9916 (Merge Pending): FILL-IN: date format: incorrect condition for triggering normalization...

Vladimir Tsichevski wrote:
> I rebased the 9916a to trunk rev. 16141. Can we merge now?
Please 9916a merge to trunk. Hynek Cihlar

12:28 PM Bug #9916: FILL-IN: date format: incorrect condition for triggering normalization of third component

I rebased the 9916a to trunk rev. 16141. Can we merge now? Vladimir Tsichevski

01:58 PM Bug #10512: REGRESSION: In ChUI, all RADIO-SET items are positioned at the same location

Hynek Cihlar wrote:
> Vladimir Tsichevski wrote:
> > Hynek, this is your code. What potential issues might arise if... Vladimir Tsichevski

01:53 PM Bug #10512: REGRESSION: In ChUI, all RADIO-SET items are positioned at the same location

Vladimir Tsichevski wrote:
> Hynek, this is your code. What potential issues might arise if we remove this code?
... Hynek Cihlar

12:05 PM Bug #10512: REGRESSION: In ChUI, all RADIO-SET items are positioned at the same location

Vladimir Tsichevski wrote:
> When the selection is shifted to the left item:
> # the inverted caret appears to th... Vladimir Tsichevski

12:01 PM Bug #10512 (Review): REGRESSION: In ChUI, all RADIO-SET items are positioned at the same location

Regression resolved in branch 10512a revision 16145. The fix involved removing the code block referenced in #10512-5.... Vladimir Tsichevski

11:42 AM Bug #10512: REGRESSION: In ChUI, all RADIO-SET items are positioned at the same location

Greg Shah wrote:
> Please update [[regression:Open Regressions in FWD v4]] with an entry about the regression.
Done. Vladimir Tsichevski

11:26 AM Bug #10512: REGRESSION: In ChUI, all RADIO-SET items are positioned at the same location

The issue occurs when this condition is satisfied, causing the loop to exit prematurely, which prevents the positioni... Vladimir Tsichevski

10:25 AM Bug #10512: REGRESSION: In ChUI, all RADIO-SET items are positioned at the same location

Please update [[regression:Open Regressions in FWD v4]] with an entry about the regression. Greg Shah

09:45 AM Bug #10512 (WIP): REGRESSION: In ChUI, all RADIO-SET items are positioned at the same location

In ChUI, all items are rendered at the same screen location. This regression stems from the merge of revision 8933a i... Vladimir Tsichevski

09:39 AM Bug #10512 (Closed): REGRESSION: In ChUI, all RADIO-SET items are positioned at the same location

Vladimir Tsichevski

09/01/2025

11:27 AM Bug #10266: Incorrect Input Handling in Grouped CHARACTER FILL-INs

Similar situation when I am entering two spaces into an empty @txt1@ @FILL-IN@, the character is not added due to:
... Vladimir Tsichevski

08/29/2025

09:58 AM Bug #10500: Frame incorrectly stays enabled

Consider this testcase:... Paula Păstrăguș

09:45 AM Bug #10500 (New): Frame incorrectly stays enabled

Paula Păstrăguș

09:36 AM Bug #10499: LEAVE event missing for top-level window

Consider this tetscase:... Paula Păstrăguș

09:29 AM Bug #10499 (New): LEAVE event missing for top-level window

Paula Păstrăguș

04:06 AM Bug #10485: App freeze due to lack of synchronization

Sergey Ivanovskiy wrote:
> Is this issue is really related to #10428?
Not exactly, but since the main discussion ... Paula Păstrăguș

04:05 AM Bug #10485: App freeze due to lack of synchronization

Paula Păstrăguș wrote:
> Hynek / Sergey, I don’t think the solution I pushed is really viable. We’re synchronizing o... Sergey Ivanovskiy

04:00 AM Bug #10485: App freeze due to lack of synchronization

The race condition can only occur during a network test when a PS timer is active and its callback interacts with the... Paula Păstrăguș

03:55 AM Bug #10485: App freeze due to lack of synchronization

Hynek / Sergey, I don’t think the solution I pushed is really viable. We’re synchronizing on the same lock that’s als... Paula Păstrăguș

03:30 AM Bug #10485: App freeze due to lack of synchronization

... Sergey Ivanovskiy

01:47 AM Bug #10485: App freeze due to lack of synchronization

Paula Păstrăguș wrote:
> Hynek Cihlar wrote:
> > Code review 10485a.
> >
> > Please remove @PushMessagesWorker.s... Hynek Cihlar

08/28/2025

01:04 PM Bug #10485: App freeze due to lack of synchronization

Hynek Cihlar wrote:
> Code review 10485a.
>
> Please remove @PushMessagesWorker.session@ and the related methods.... Sergey Ivanovskiy

08:49 AM Bug #10485: App freeze due to lack of synchronization

Hynek Cihlar wrote:
> Code review 10485a.
>
> Please remove @PushMessagesWorker.session@
If we remove the sess... Paula Păstrăguș

07:46 AM Bug #10485 (WIP): App freeze due to lack of synchronization

Code review 10485a.
Please remove @PushMessagesWorker.session@ and the related methods.... Hynek Cihlar

03:18 AM Bug #10485 (Review): App freeze due to lack of synchronization

Created task branch 10485a.
Committed solution as rev *16135*.
Hynek / Sergey please review. Paula Păstrăguș

02:37 AM Bug #10485: App freeze due to lack of synchronization

For additional details, please refer to entries #10428-24 through #10428-29 Paula Păstrăguș

02:35 AM Bug #10485 (WIP): App freeze due to lack of synchronization

The application freeze occurs due to missing synchronization between @PushMessagesWorker.sendMessage@ and the network... Paula Păstrăguș

02:31 AM Bug #10485 (Test): App freeze due to lack of synchronization

Paula Păstrăguș

12:58 PM Bug #10075: Number format validation issues

@NumberFormatValidationTest@ and @NumberFormattingTest@ classes added to the testcases at rev. 1802. Vladimir Tsichevski

09:28 AM Bug #10494: Overlapping widgets are no displayed depending of when and how they're set visible

This issue started from searching for a testcase for #8015.
Found that the issue from #8015 is a particular one an... Delia Mitric

09:21 AM Bug #10494 (New): Overlapping widgets are no displayed depending of when and how they're set visible

Delia Mitric

08/27/2025

07:13 AM Bug #10075: Number format validation issues

Greg Shah wrote:
> > Therefore, I’ve decided to place @NumberFormatValidationTest.cls@ in the same directory as @Tes... Vladimir Tsichevski

06:14 AM Bug #10075: Number format validation issues

> Therefore, I’ve decided to place @NumberFormatValidationTest.cls@ in the same directory as @TestString.cls@
Thi... Greg Shah

08/26/2025

05:54 PM Bug #10075: Number format validation issues

Greg Shah wrote:
> I mean in that set of tests. It probably won't be in that same class. Over time we will probabl... Vladimir Tsichevski

03:44 PM Bug #10075: Number format validation issues

I mean in that set of tests. It probably won't be in that same class. Over time we will probably refactor the exist... Greg Shah

03:06 PM Bug #10075: Number format validation issues

Greg Shah wrote:
> It is still the place to put the test code. I expect many more tests to be added over time.
D... Vladimir Tsichevski

03:02 PM Bug #10075: Number format validation issues

It is still the place to put the test code. I expect many more tests to be added over time. Greg Shah

01:52 PM Bug #10075: Number format validation issues

Greg Shah wrote:
> See [[p2j:Format String Tests]].
This page points to a single test class designed to evaluate ... Vladimir Tsichevski

01:23 PM Bug #10075: Number format validation issues

See [[p2j:Format String Tests]]. Greg Shah

12:58 PM Bug #10075: Number format validation issues

Greg Shah wrote:
> Please commit all tests into the testcases project.
1. I'm uncertain about the appropriate loc... Vladimir Tsichevski

07:17 AM Bug #10075: Number format validation issues

Please commit all tests into the testcases project. Greg Shah