Project

General

Profile

Actions

Bug #1858

closed

Queued/pending browser events can cause display errors, e.g. tabs in bootstrap-themed WTabWidget.

Added by Bruce Toll over 11 years ago. Updated over 11 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Start date:
04/19/2013
Due date:
% Done:

0%

Estimated time:

Description

When pending events are queued on the browser (due to communication or server delay, etc.), they are processed in a batch on the server and this can lead to display problems --- under some circumstances. This report describes one way this can happen (using github version 3.3.0-7-g53cd144), along with a test program and some captured communication output. The issue is not believed to be browser-specific, although it does require javascript.

The attached test program (tabwidget_bug_20130417a.cc) displays a TabWidget with three tabs. To drive the problem:

  1. Run the program on a server and navigate to the application page in a browser.
  2. Temporarily disable communication with the server.
  3. Click on tabs in the following sequence: 1 - 2 - 3 - 2 - 1.
  4. Enable communication with the server.
  5. You should note that the contents of the first tab are displayed, but not the tab itself. That is, no tab is selected.

The test program can be invoked with a URL parameter of "?polished" to test with the "polished" theme. Unlike with the "bootstrap" theme, the first tab should appear selected after this sequence.

The request/response pairs along with headers are provided in a second attached file: queue_style_issue_log_20130419c.txt

This issue can also currently be observed on the Wt site's tab widget example at: http://www.webtoolkit.eu/widgets/navigation/tab-widget

In this case, just click the first three tabs in sequence: "First" - "Preload" - "Style" - "Preload" - "First", before re-enabling communication.

A potential fix for this issue will be posted shortly on github and I will update this report with a link.


Files

Actions #1

Updated by Bruce Toll over 11 years ago

A pull request with a proposed patch is now available here: https://github.com/kdeforche/wt/pull/27

Actions #2

Updated by Koen Deforche over 11 years ago

  • Status changed from New to InProgress
  • Assignee set to Koen Deforche
  • Target version set to 3.3.1
Actions #3

Updated by Koen Deforche over 11 years ago

  • Status changed from InProgress to Resolved

Hey,

it turned out it had to do with multiple additions of the same style class to the addedStyleClasses and removedStyleClasses vectors... Good we caught it.

Regards,

koen

Actions #4

Updated by Koen Deforche over 11 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF