Project

General

Profile

Activity

From 09/08/2025 to 10/07/2025

10/04/2025

05:19 PM Bug #10654: TreeList MultiSelect Mode Issues
h3. Issue 4: Selection Processing on Mouse Up vs. Mouse Down
In OE, a selection change is processed on mouse relea...
Vladimir Tsichevski

10/02/2025

01:25 PM Bug #8809: Incorrect next enabled widget calculation
Can this be closed? Vladimir Tsichevski
01:24 PM Bug #9929 (Closed): FILL-IN: Incorrect APPLY statement behavior
Vladimir Tsichevski wrote:
> Can this be closed?
Yes.
Hynek Cihlar
01:23 PM Bug #9929: FILL-IN: Incorrect APPLY statement behavior
Can this be closed? Vladimir Tsichevski
04:41 AM Bug #10654: TreeList MultiSelect Mode Issues
By assertion failure, do you mean that our code is intentionally abending (exiting the client session)? This is some... Greg Shah

10/01/2025

02:34 PM Bug #10654: TreeList MultiSelect Mode Issues
h3. Issue 1: Incorrect Node Selection in MultiSelect Mode
Steps to reproduce:
# Expand a node.
# Set @MultiSel...
Vladimir Tsichevski
02:32 PM Bug #10654 (New): TreeList MultiSelect Mode Issues
Vladimir Tsichevski

09/30/2025

08:29 AM Bug #10485 (Test): App freeze due to lack of synchronization
10485a was merged into trunk as rev 16188 and archived. Paula Păstrăguș
08:01 AM Bug #10485 (Merge Pending): App freeze due to lack of synchronization
Please merge 10485a to trunk now. Hynek Cihlar
07:06 AM Bug #10485: App freeze due to lack of synchronization
I believe Hotel Gui and the large GUI application provide sufficient coverage. If there's anything else you'd like me... Paula Păstrăguș
05:42 AM Bug #10485 (Internal Test): App freeze due to lack of synchronization
Code review 10485a. The changes look good.
Is there any more testing needed?
Hynek Cihlar
08:03 AM Bug #10236: Side label for a widget isn't displayed when parent frame is defined using n DOWN, ...
This is how the testcase looks in FWD with trunk:
GUI: !1GUI.png! --------- ChUI: !1ChUI.png!
This is how the...
Delia Mitric
07:34 AM Bug #10236: Side label for a widget isn't displayed when parent frame is defined using n DOWN, ...
How is this working in ChUI in FWD? Constantin Asofiei
07:32 AM Bug #10236: Side label for a widget isn't displayed when parent frame is defined using n DOWN, ...
*Constantin*, I've noticed that you worked on that method. Can you please explain why it is necessary to check this: ... Delia Mitric
07:08 AM Bug #10236 (WIP): Side label for a widget isn't displayed when parent frame is defined using n ...
I've minimized the testcase:... Delia Mitric
07:58 AM Feature #6583 (Closed): Implement support for TOP-ONLY and ALWAYS-ON-TOP in Web GUI driver
@TOP-ONLY@ and @ALWAYS-ON-TOP@ had been implemented for Web GUI, closing the issue. Hynek Cihlar

09/29/2025

09:32 AM Bug #10602 (Test): Fix scrolling in browse on Swing after switching to accumulated scrolling
10602a has been merged into trunk with rev. 16186 and archived. Șerban Bursuc
09:23 AM Bug #10602 (Merge Pending): Fix scrolling in browse on Swing after switching to accumulated scrol...
Please merge to trunk now. Alexandru Lungu
08:58 AM Bug #10602: Fix scrolling in browse on Swing after switching to accumulated scrolling
Alexandru Lungu wrote:
> Serban, is it ready to be merged?
It is now.
For clarity: somehow a set of changes sl...
Șerban Bursuc
08:45 AM Bug #10602: Fix scrolling in browse on Swing after switching to accumulated scrolling
Serban, is it ready to be merged? Alexandru Lungu
05:28 AM Bug #10628 (Test): embedded mode loading indicator issue
Branch 10628a was merged into trunk as rev. 16185 and archived. Sergey Ivanovskiy
05:20 AM Bug #10628: embedded mode loading indicator issue
OK. For my current version of embedded hotel gui the indicator page looks differently. Sergey Ivanovskiy
04:59 AM Bug #10628 (Merge Pending): embedded mode loading indicator issue
Please merge now. Constantin Asofiei
04:56 AM Bug #10628: embedded mode loading indicator issue
10628a created with rev 16185. Open Regressions in FWD v4 list was updated with rev 16177 regression.
These changes ...
Sergey Ivanovskiy
02:05 AM Bug #10628: embedded mode loading indicator issue
Constantin Asofiei wrote:
> Greg, the fix would be this:
> [...]
>
>
> Just a check in p2j.socket.js for embed...
Sergey Ivanovskiy
01:59 AM Bug #10628: embedded mode loading indicator issue
OK. This issue looks unexpected because I definitely checked this issue and even checked the web:client:embedded init... Sergey Ivanovskiy

09/28/2025

06:32 AM Bug #10628 (Review): embedded mode loading indicator issue
That does indeed fix the issue.
Sergey: Please get this into a branch, reviewed, tested and merged ASAP. Also: Pl...
Greg Shah

09/26/2025

04:28 PM Bug #10628: embedded mode loading indicator issue
Greg, the fix would be this:... Constantin Asofiei
03:47 PM Bug #10628: embedded mode loading indicator issue
After 20 minutes, you can see the indicator increase.
!hotel_gui_embedded_mode_loading_indicator_does_increase_202...
Greg Shah
03:25 PM Bug #10628: embedded mode loading indicator issue
Weirdly, the window underneath is actually functional and responds to input:
!hotel_gui_embedded_mode_loading_indi...
Greg Shah
03:22 PM Bug #10628: embedded mode loading indicator issue
Using trunk rev 16184, Hotel GUI embedded mode does not initialize properly. The iframe has a loading indicator that... Greg Shah
03:18 PM Bug #10628 (Test): embedded mode loading indicator issue
Greg Shah

09/25/2025

09:49 AM Bug #10593: SPREADSHEET: Keikai backend sends huge amount of data to the frontend
Today I had a remote session with Potix support. I demoed the issue, they spent some time on gathering data and analy... Hynek Cihlar
08:10 AM Feature #6583: Implement support for TOP-ONLY and ALWAYS-ON-TOP in Web GUI driver
There is an issue now: https://proj.goldencode.com/issues/10617
TOP-ONLY works fine though. We only seem to be mi...
Jurjen Dijkstra
04:45 AM Bug #10602 (Internal Test): Fix scrolling in browse on Swing after switching to accumulated scrol...
Șerban Bursuc
04:43 AM Bug #10602: Fix scrolling in browse on Swing after switching to accumulated scrolling
> 10602a rev 16185.
The update is good.
Stanislav Lomany
04:32 AM Bug #10602: Fix scrolling in browse on Swing after switching to accumulated scrolling
Stanislav Lomany wrote:
> I'm OK with the update, but please hide the calculations unnecessary for the Swing case un...
Șerban Bursuc

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
 

Also available in: Atom