Project

General

Profile

Bug #5860

Wt 4 : Possibly rendering bug in WTable

Added by Hyeoni Hwasoo over 5 years ago. Updated about 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Target version:
Start date:
08/07/2017
Due date:
% Done:

0%

Estimated time:

Description

Hi. I had a bug in a WTable rendering (rows and lines not completely generated at the DOM).

But when I add a line (in the attached patch), my bug is fixed.


Files

wt-4.0.0-rc2-2017-08-07-bug-dom.diff (345 Bytes) wt-4.0.0-rc2-2017-08-07-bug-dom.diff Hyeoni Hwasoo, 08/07/2017 03:57 AM
main.cxx (1.93 KB) main.cxx Source code to reproduce Hyeoni Hwasoo, 08/09/2017 02:39 AM
wtable.png (13.4 KB) wtable.png Content wrongly displayed Hyeoni Hwasoo, 08/09/2017 02:39 AM
#1

Updated by Roel Standaert over 5 years ago

  • Status changed from New to Feedback

Hi, do you have some example code where it goes wrong? I can't manage to reproduce your problem, and the patch you propose is unusual enough (triggering a repaint from within the rendering code) that I'm not comfortable just applying it.

#2

Updated by Hyeoni Hwasoo over 5 years ago

Yes I wasn't sure my patch was right

But finally I isolated the behaviour ! (not the bug)

I joined two files :

  • main.cxx a small programm to reproduce it
  • wtable.png a screenshot to see the result

Steps to reproduce :

  1. Load the web page
  2. Click on the "Push Me" button -> The first time, the table is correctly displayed
  3. Reload the page with F5 (Firefox or Chromium)
  4. Click on the "Push Me" button -> The table is often wrongly displayed (lack of cell)
#3

Updated by Hyeoni Hwasoo over 5 years ago

Hello ! This issue appears to be fixed at this time ( https://github.com/emweb/wt/commit/99df4a973d63a53676d76db6232a22fd64e898f0 ) ! I didn't try the commit https://github.com/emweb/wt/commit/91b04306b47eadb743cb929d848e9f6ae22c4450

So this issue can be marked as resolved.

Thank you !

Hyeoni

#4

Updated by Wim Dumon over 5 years ago

  • Status changed from Feedback to Resolved
#5

Updated by Roel Standaert about 5 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF