Project

General

Profile

Actions

Bug #5807

open

WMediaPlayer custom time text widgets working

Added by Alex Ignatov over 7 years ago. Updated almost 5 years ago.

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

0%

Estimated time:

Description

I'm using wt-3.3.7 but I checked the same applies to the later RC versions.

I simply quote the code from the render() function in WMediaPlayer.C (545):

const char *displaySelectors[] =
{ "currentTime", "duration" };
for (unsigned i = CurrentTime; i < Duration; ++i) {
if (control_[i]) {
if (!first)
ss << ", ";
ss << const_cast<char *>(displaySelectors[i]) << ":\"#"
<< display_[i]->id() << "\"";
first = false;
}
}

First of all the for loop covers only one control (CurrentTime).

And in the loop the control_ pointer is checked to be not null, not the display_.

I do not understand why these text fields work in the default control widget implementation though.

Actions #1

Updated by Alex Ignatov over 7 years ago

NOT working of course :)

Regards

Alex

Actions #2

Updated by Alex Ignatov over 7 years ago

Same error higher in the code - while looping over control_ array the for loop does not include the last "RepeatOff" control:

for (unsigned i = VideoPlay; i < RepeatOff; ++i)

Actions #3

Updated by Alex Ignatov almost 7 years ago

I upgraded my development to the current release (4.0.2) and the bug is still there. Custom controls widget will not work properly out of the box.

Actions #4

Updated by Alex Ignatov almost 5 years ago

Both bugs are still there in the master :) Has no one ever used custom controls for the player?

Actions

Also available in: Atom PDF