Chromium Code Reviews Chromium Code Reviews
Issue 478033004:
Fix a couple of scroll-related issues: (Closed)
Base URL: svn://svn.chromium.org/chrome/trunk/src| OLD | NEW |
|---|---|
| 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 601 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 612 #daemon-plugin-container { | 612 #daemon-plugin-container { |
| 613 width: 0; | 613 width: 0; |
| 614 height: 0; | 614 height: 0; |
| 615 } | 615 } |
| 616 | 616 |
| 617 .dialog-container { | 617 .dialog-container { |
| 618 position: absolute; | 618 position: absolute; |
| 619 top: 200px; | 619 top: 200px; |
| 620 left: 0; | 620 left: 0; |
| 621 width: 100%; | 621 width: 100%; |
| 622 z-index: 2; | |
| 622 } | 623 } |
| 623 | 624 |
| 624 .dialog-screen { | 625 .dialog-screen { |
| 625 position: absolute; | 626 position: absolute; |
| 626 top: 0; | 627 top: 1px; |
| 627 left: 0; | 628 left: 1px; |
| 628 width: 100%; | 629 width: calc(100% - 2px); |
| 629 height: 100%; | 630 height: calc(100% - 2px); |
|
Jamie
2014/08/23 00:12:00
Allow 2px for the apps v2 window border (which is
garykac1
2014/08/23 00:21:40
Doesn't this affect the v1 app border as well?
Jamie
2014/08/23 00:32:57
For the v1 app, there is no HTML window border, so
| |
| 630 background-color: #fff; | 631 background-color: #fff; |
| 631 opacity: 0.75; | 632 opacity: 0.75; |
| 633 z-index: 1; | |
| 632 } | 634 } |
| 633 | 635 |
| 634 /* TODO(jamiewalch): crbug.com/252796: Remove these once crbug.com/240772 | 636 /* TODO(jamiewalch): crbug.com/252796: Remove these once crbug.com/240772 |
| 635 * is fixed. */ | 637 * is fixed. */ |
| 636 .no-horizontal-scroll { | 638 .no-horizontal-scroll { |
| 637 overflow-x: hidden !important; | 639 overflow-x: hidden !important; |
| 638 } | 640 } |
| 639 | 641 |
| 640 .no-vertical-scroll { | 642 .no-vertical-scroll { |
| 641 overflow-y: hidden !important; | 643 overflow-y: hidden !important; |
| (...skipping 98 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 740 * no scroll-bars, so the advantages of flex-box layout to achieve centering | 742 * no scroll-bars, so the advantages of flex-box layout to achieve centering |
| 741 * (ie, the DOM is easier to measure to determine when scroll-bars are needed) | 743 * (ie, the DOM is easier to measure to determine when scroll-bars are needed) |
| 742 * don't apply */ | 744 * don't apply */ |
| 743 .full-screen #session-mode { | 745 .full-screen #session-mode { |
| 744 position: fixed; | 746 position: fixed; |
| 745 top: 0; | 747 top: 0; |
| 746 left: 0; | 748 left: 0; |
| 747 width: 100%; | 749 width: 100%; |
| 748 } | 750 } |
| 749 | 751 |
| 752 /* Override full-height class until connected, otherwise it takes up 100% of | |
| 753 * the window height in addition to the home-screen UI. */ | |
| 754 body:not(.connected) #session-mode { | |
| 755 height: auto; | |
| 756 } | |
| 757 | |
| 750 /* video-container needs relative position so that mediasource-video-output can | 758 /* video-container needs relative position so that mediasource-video-output can |
| 751 * be positioned relative to the parent with position:absolute. */ | 759 * be positioned relative to the parent with position:absolute. */ |
| 752 #video-container { | 760 #video-container { |
| 753 position: relative; | 761 position: relative; |
| 754 } | 762 } |
| 755 | 763 |
| 756 /* mediasource-video-output is hidden by default. */ | 764 /* mediasource-video-output is hidden by default. */ |
| 757 #video-container video { | 765 #video-container video { |
| 758 display: none; | 766 display: none; |
| 759 } | 767 } |
| (...skipping 20 matching lines...) Expand all Loading... | |
| 780 } | 788 } |
| 781 | 789 |
| 782 .mouse-cursor-overlay { | 790 .mouse-cursor-overlay { |
| 783 position: absolute; | 791 position: absolute; |
| 784 pointer-events: none; | 792 pointer-events: none; |
| 785 } | 793 } |
| 786 | 794 |
| 787 body.hangout-remote-desktop .home-screen { | 795 body.hangout-remote-desktop .home-screen { |
| 788 display: none; | 796 display: none; |
| 789 } | 797 } |
| OLD | NEW |
