[Webkit-unassigned] [Bug 117008] [WK2][GTK] add support for subtitles on webkit2GTK

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Aug 18 07:25:56 PDT 2013


https://bugs.webkit.org/show_bug.cgi?id=117008





--- Comment #18 from Danilo Cesar Lemes de Paula <danilo.cesar at collabora.co.uk>  2013-08-18 07:25:26 PST ---
(In reply to comment #17)
> (From update of attachment 209030 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=209030&action=review
> 
> > Source/WebCore/css/mediaControlsGtk.css:227
> > +    background-image: url('data:image/svg+xml,<svg id="svg7384" xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#" xmlns="http://www.w3.org/2000/svg" height="16" width="16" version="1.1" xmlns:cc="http://creativecommons.org/ns#" xmlns:dc="http://purl.org/dc/elements/1.1/"><metadata id="metadata90"><rdf:RDF><cc:Work rdf:about=""><dc:format>image/svg+xml</dc:format><dc:type rdf:resource="http://purl.org/dc/dcmitype/StillImage"/><dc:title>Gnome Symbolic Icon Theme</dc:title></cc:Work></rdf:RDF></metadata><g id="layer9" transform="translate(-101,-357)"><path id="path12148" style="block-progression:tb;color:#bebebe;direction:ltr;text-indent:0;text-align:start;enable-background:accumulate;text-transform:none;" fill="#bebebe" d="m104.75,357.06c-2.0602,0-3.75,1.6898-3.75,3.75v4.4375c0,2.0602,1.6898,3.75,3.75,3.75h4.9375l3.75,2.6562,1.5938,1.125v-1.9688l-0.0313-2.5c1.1106-0.59715,1.9688-1.6526,1.9688-3.0625v-4.4375c0-2.0602-1.6898-3.75-3.75-3.75h-8.4688zm0,2,8.4688,0c0.9868,0,1.75,0.7632,1.75,1.75v4.4375c0,0.86273-0.63508,1.541-1.125,1.625l-0.84,0.12v0.84375,1.0312l-2.4062-1.6875-0.25-0.1875h-0.3125-5.2812c-0.9868,0-1.75-0.7632-1.75-1.75v-4.4375c0-0.9868,0.7632-1.75,1.75-1.75z"/></g></svg>');
> 
> Are other ports putting the icon in the CSS itself?

As far as I know only apple is doing that.
I would love to use a Gtk themed image, but there's no gtk images related to subtitling. The SVG I'm using is a stripped version of a chat image inside Gtk code.

> 
> > Source/WebCore/css/mediaControlsGtk.css:232
> > +    position: absolute;
> 
> Absolute positioning?
Well, yes. We're using the same approach for the volume slider.

> 
> > Source/WebKit2/UIProcess/API/gtk/WebKitSettings.cpp:1155
> > +    WebKitSettings* settings = WEBKIT_SETTINGS(g_object_new(WEBKIT_TYPE_SETTINGS, NULL));
> > +    settings->priv->preferences->setShouldDisplaySubtitles(TRUE);
> > +    return settings;
> 
> It probably makes sense to do this in webKitSettingsConstructed instead.

I will push a new patch fixing that.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list