Project

General

Profile

Bug #5807

WMediaPlayer custom time text widgets working

Added by Alex Ignatov about 5 years ago. Updated over 2 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.

#1

Updated by Alex Ignatov about 5 years ago

NOT working of course :)

Regards

Alex

#2

Updated by Alex Ignatov about 5 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)

#3

Updated by Alex Ignatov over 4 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.

#4

Updated by Alex Ignatov over 2 years ago

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

Also available in: Atom PDF