Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1182)

Side by Side Diff: remoting/webapp/main.css

Issue 269793002: Fix <video> positioning for the MediaSource-based rendering (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « remoting/webapp/html/client_plugin.html ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 /* Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 * Use of this source code is governed by a BSD-style license that can be 2 * Use of this source code is governed by a BSD-style license that can be
3 * found in the LICENSE file. 3 * found in the LICENSE file.
4 */ 4 */
5 5
6 html, body, div, span, applet, object, iframe, h1, h2, h3, h4, h5, h6, p, 6 html, body, div, span, applet, object, iframe, h1, h2, h3, h4, h5, h6, p,
7 blockquote, pre, a, abbr, acronym, address, big, cite, code, del, dfn, em, 7 blockquote, pre, a, abbr, acronym, address, big, cite, code, del, dfn, em,
8 font, img, ins, kbd, q, s, samp, small, strike, strong, sub, sup, tt, var, dl, 8 font, img, ins, kbd, q, s, samp, small, strike, strong, sub, sup, tt, var, dl,
9 dt, dd, ol, ul, li, fieldset, form, label, legend, table, caption, tbody, 9 dt, dd, ol, ul, li, fieldset, form, label, legend, table, caption, tbody,
10 tfoot, thead, tr, th, td, button { 10 tfoot, thead, tr, th, td, button {
(...skipping 689 matching lines...) Expand 10 before | Expand all | Expand 10 after
700 } 700 }
701 701
702 /* 702 /*
703 * Setting hidden on elements that match some rule overriding 'display' doesn't 703 * Setting hidden on elements that match some rule overriding 'display' doesn't
704 * do what you would expect unless this is made explicit (and !important). 704 * do what you would expect unless this is made explicit (and !important).
705 */ 705 */
706 [hidden] { 706 [hidden] {
707 display: none !important; 707 display: none !important;
708 } 708 }
709 709
710 .full-width {
711 width: 100%;
712 }
713
710 .full-height { 714 .full-height {
711 height: 100%; 715 height: 100%;
712 } 716 }
713 717
714 .horizontally-centered { 718 .horizontally-centered {
715 display: flex; 719 display: flex;
716 align-items: center; 720 align-items: center;
717 } 721 }
718 722
719 .vertically-centered { 723 .vertically-centered {
(...skipping 18 matching lines...) Expand all
738 .full-screen #session-mode { 742 .full-screen #session-mode {
739 position: fixed; 743 position: fixed;
740 top: 0; 744 top: 0;
741 left: 0; 745 left: 0;
742 width: 100%; 746 width: 100%;
743 } 747 }
744 748
745 /* video-container needs relative position so that mediasource-video-output can 749 /* video-container needs relative position so that mediasource-video-output can
746 * be positioned relative to the parent with position:absolute. */ 750 * be positioned relative to the parent with position:absolute. */
747 #video-container { 751 #video-container {
748 width: 100%;
749 position: relative; 752 position: relative;
750 } 753 }
751 754
752 /* mediasource-video-output is hidden by default. */ 755 /* mediasource-video-output is hidden by default. */
753 #mediasource-video-output { 756 #mediasource-video-output {
754 display: none; 757 display: none;
755 } 758 }
756 759
757 /* Use absolute positioning for mediasource-video-output so that it's rendered 760 /* Use absolute positioning for mediasource-video-output so that it's rendered
758 * at the same position as the plugin. */ 761 * at the same position as the plugin. */
759 #video-container.mediasource-rendering #mediasource-video-output { 762 #video-container.mediasource-rendering #mediasource-video-output {
760 display: block; 763 display: block;
761 position: absolute; 764 position: absolute;
762 left: 0; 765 left: 0;
763 top: 0; 766 top: 0;
764 box-shadow: 0 0 8px 0 black; 767 box-shadow: 0 0 8px 0 black;
765 } 768 }
766 769
767 /* 770 /*
768 * With MediaSource-based rendering the plugin is transparent and is placed on 771 * With MediaSource-based rendering the plugin is transparent and is placed on
769 * top of the <video> element so that it can still receive mouse events. 772 * top of the <video> element so that it can still receive mouse events.
770 * 773 *
771 * TODO(sergeyu): This is temporary solution. Ideally mouse and keyboard events 774 * TODO(sergeyu): This is temporary solution. Ideally mouse and keyboard events
772 * should be captured on JS level and passed to the plugin. 775 * should be captured on JS level and passed to the plugin.
773 */ 776 */
774 #video-container.mediasource-rendering #client-plugin-container { 777 #video-container.mediasource-rendering #client-plugin-container {
775 opacity: 0; 778 opacity: 0;
776 } 779 }
OLDNEW
« no previous file with comments | « remoting/webapp/html/client_plugin.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698