Chromium Code Reviews| 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 #saved-passwords-list .list-inline-button { | 5 #saved-passwords-list .list-inline-button { |
| 6 -webkit-transition: opacity 150ms; | 6 -webkit-transition: opacity 150ms; |
| 7 background: rgb(138, 170, 237); | 7 background: rgb(138, 170, 237); |
| 8 font-size: 0.9em; | 8 font-size: 0.9em; |
| 9 height: 18px; | 9 height: 18px; |
| 10 margin-left: 2px; | 10 margin-left: 2px; |
| 11 margin-right: 2px; | 11 margin-right: 2px; |
| 12 padding: 0 2px; | 12 padding: 0 2px; |
| 13 position: absolute; | 13 position: absolute; |
| 14 top: 3px; | 14 top: 3px; |
| 15 } | 15 } |
| 16 | 16 |
| 17 html[dir='ltr'] #saved-passwords-list .list-inline-button { | 17 html[dir='ltr'] #saved-passwords-list .list-inline-button { |
| 18 right: 2px; | 18 right: 2px; |
| 19 } | 19 } |
| 20 | 20 |
| 21 html[dir='rtl'] #saved-passwords-list .list-inline-button { | 21 html[dir='rtl'] #saved-passwords-list .list-inline-button { |
| 22 left: 2px; | 22 left: 2px; |
| 23 } | 23 } |
| 24 | 24 |
| 25 input[type='password'].inactive-password { | 25 input[type='password'].inactive-item, |
| 26 input[type='text'].inactive-item { | |
| 26 background: transparent; | 27 background: transparent; |
| 27 border: none; | 28 border: none; |
| 28 } | 29 } |
| 29 | 30 |
| 30 #saved-passwords-list .url { | 31 #saved-passwords-list .url { |
| 31 box-sizing: border-box; | 32 box-sizing: border-box; |
| 32 width: 40%; | 33 width: 40%; |
| 33 } | 34 } |
| 34 | 35 |
| 35 #saved-passwords-list .name { | 36 #saved-passwords-list .name { |
| 36 -webkit-box-flex: 1; | 37 -webkit-box-flex: 1; |
| 37 width: 30%; | 38 width: 30%; |
| 38 } | 39 } |
| 39 | 40 |
| 41 #saved-passwords-list .url, | |
| 42 #saved-passwords-list .name { | |
| 43 -webkit-user-select: text; | |
| 44 } | |
| 45 | |
| 40 #saved-passwords-list .password, | 46 #saved-passwords-list .password, |
| 41 #saved-passwords-list .federation { | 47 #saved-passwords-list .federation { |
| 42 -webkit-box-flex: 1; | 48 -webkit-box-flex: 1; |
| 43 position: relative; | 49 position: relative; |
| 44 width: 30%; | 50 width: 30%; |
| 45 } | 51 } |
| 46 | 52 |
| 47 #saved-passwords-list .password input[type='password'], | 53 #saved-passwords-list .password input[type='password'], |
| 48 #saved-passwords-list .password input[type='text'] { | 54 #saved-passwords-list .password input[type='text'] { |
| 49 box-sizing: border-box; | 55 box-sizing: border-box; |
| 50 width: 100%; | 56 width: 100%; |
| 51 } | 57 } |
| 52 | 58 |
| 53 #password-exceptions-list .url { | 59 #password-exceptions-list .url { |
| 54 -webkit-box-flex: 1; | 60 -webkit-box-flex: 1; |
| 55 } | 61 } |
| 56 | 62 |
| 57 #saved-passwords-list .url, | 63 #saved-passwords-list .url, |
| 58 #saved-passwords-list .name, | 64 #saved-passwords-list .name, |
| 59 #saved-passwords-list .federation, | 65 #saved-passwords-list .federation, |
| 60 #password-exceptions-list .url { | 66 #password-exceptions-list .url { |
| 61 overflow: hidden; | 67 overflow: hidden; |
| 62 text-overflow: ellipsis; | 68 text-overflow: ellipsis; |
| 63 } | 69 } |
| 70 | |
| 71 #saved-passwords-list .left-elided-url, | |
|
Evan Stade
2015/09/29 18:06:52
can you just do .left-elided-url{...}?
kolos1
2015/09/29 18:29:01
Done.
| |
| 72 #password-exceptions-list .left-elided-url { | |
| 73 -webkit-margin-start: 0; | |
| 74 -webkit-padding-start: 3px; | |
| 75 direction: rtl; | |
|
Evan Stade
2015/09/29 18:06:52
I must be totally misunderstanding this class beca
kolos1
2015/09/29 18:29:01
This class elide urls (without scheme and path) fr
Evan Stade
2015/09/29 18:53:07
I see. So this is a big hack and needs a lot of do
kolos1
2015/10/05 18:43:22
Yes, I tested with RTL and LTR locales. Now it wor
| |
| 76 margin-left: 7px; | |
| 77 padding-left: 26px; | |
|
Evan Stade
2015/09/29 18:06:53
why do you have padding-left as well as -webkit-pa
kolos1
2015/10/05 18:43:22
For LTR: since the direction is changed, the icon
| |
| 78 text-align: left; | |
|
Evan Stade
2015/09/29 18:06:52
why do you want to left align for RTL? the omnibox
Evan Stade
2015/09/29 18:53:07
I still suspect this is the wrong text-align for R
kolos1
2015/10/05 18:43:23
Right. It should be only for LTR. Fixed.
| |
| 79 unicode-bidi: bidi-override; | |
| 80 } | |
| OLD | NEW |