Project

General

Profile

Actions
Copy link

Bug #11008

closed

StdGridLayoutImpl2.js: figure out what's up with the undefined i in setChildSize

Added by Roel Standaert about 2 years ago. Updated about 2 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Romain Mardulyn
Target version:
4.11.1
Start date:
10/14/2022
Due date:
% Done:

100%

Estimated time:

Description

While working on issue #10967 I noticed that i is always undefined in setChildSize in StdGridLayoutImpl2.js.

This is almost certainly a bug, so we should get to the bottom of it (or if we retire JS-based layouts, we can also just ignore this).

Related issues 1 (0 open — 1 closed)

Related to Improvements #10967: Add linting to Wt's JavaScript and address warnings coming from the linterClosedRoel Standaert09/22/2022

Actions
Actions
Copy link
 #1

Updated by Roel Standaert about 2 years ago

  • Related to Improvements #10967: Add linting to Wt's JavaScript and address warnings coming from the linter added
Actions
Copy link
 #2

Updated by Matthias Van Ceulebroeck 4 months ago

  • Assignee set to Romain Mardulyn
  • Target version changed from future to 4.11.2
Actions
Copy link
 #3

Updated by Romain Mardulyn 4 months ago

  • Status changed from New to InProgress
Actions
Copy link
 #4

Updated by Romain Mardulyn 4 months ago

  • Status changed from InProgress to Review
  • Assignee deleted (Romain Mardulyn)
Actions
Copy link
 #5

Updated by Matthias Van Ceulebroeck 3 months ago

  • Assignee set to Matthias Van Ceulebroeck
Actions
Copy link
 #6

Updated by Matthias Van Ceulebroeck 2 months ago

  • Target version changed from 4.11.2 to 4.11.1
Actions
Copy link
 #7

Updated by Matthias Van Ceulebroeck 2 months ago

  • Status changed from Review to Implemented @Emweb
  • Assignee changed from Matthias Van Ceulebroeck to Romain Mardulyn
  • % Done changed from 0 to 100
Actions
Copy link
 #8

Updated by Matthias Van Ceulebroeck 2 months ago

  • Status changed from Implemented @Emweb to Implemented @Test
Actions
Copy link
 #9

Updated by Matthias Van Ceulebroeck about 2 months ago

  • Status changed from Implemented @Test to Closed
Actions
Copy link

Also available in: Atom PDF