Bug 52474 – Volume slider for audio element does not fully display in Qt

• Status: RESOLVED
• Resolution: DUPLICATE
• Component: Media

[Keith Rosenblatt]

Overview: 

    The volume slider, upon hovering over the mute button of an audio element, is not fully visible.  The portion that displays beyond the controls panel is hidden.

Steps to Reproduce: 

    1) View html page with an audio element in QtTestBrowser
    2) move cursor over mute button
    3) view volume bar that appears

Actual Results: 

    Only very bottom of volume bar is visible

Expected Results: 

    Entire volume slider should be visible.

Build Date & Platform: 

    Build 2011-01-14 on Linux with Qt 4.7.0

[Keith Rosenblatt]

Created attachment 79301 [details]
Fix to Qt media control CSS

[Keith Rosenblatt]

Created attachment 79446 [details]
Fix to Qt media control CSS

Drop unnecessary pixel test

[Antonio Gomes]

Comment on attachment 79446 [details]
Fix to Qt media control CSS

The test lacks expected results for platforms other than Qt.

[Laszlo Gombos]

Comment on attachment 79446 [details]
Fix to Qt media control CSS

r- to fix expected results for other ports. An entry in the Skipped lists with a comment will do it.

[Antonio Gomes]

(In reply to comment #4)
> (From update of attachment 79446 [details])
> r- to fix expected results for other ports. An entry in the Skipped lists with a comment will do it.

I am afraid skipping the test for other can not be ideal, since this is changing default .css files.

-    overflow: hidden;
+    overflow: visible;
     height: 100%;

I would suggest at least also checking in Mac expected files...

[Antonio Gomes]

I am afraid skipping the test for other can not be ideal, since this is changing default .css files.
> I would suggest at least also checking in Mac expected files...

Correcting myself, the file being changed is Qt-only, so no worries.

Another option is adding the test to LayoutTest/platform/qt/xxx/yyy , so it will only run on QtWebKit, and does not need to be skipped to others.

Up to you ...

[Keith Rosenblatt]

Comment on attachment 79446 [details]
Fix to Qt media control CSS

Obsoleted by https://bugs.webkit.org/show_bug.cgi?id=52822

[Keith Rosenblatt]

*** This bug has been marked as a duplicate of bug 52822 ***