From 1e963a6c3ae59f401fa4e440b128c6fa2ea8ada9 Mon Sep 17 00:00:00 2001 From: Matthew Hodgson Date: Fri, 15 Jul 2016 16:54:27 +0100 Subject: [PATCH] Revert "Presentation for inline link" --- src/skins/vector/css/common.css | 8 --- .../views/rooms/EventTile.css | 10 ++-- .../views/rooms/LinkPreviewWidget.css | 54 ++++++++++--------- src/skins/vector/img/icon-link.svg | 15 ------ 4 files changed, 36 insertions(+), 51 deletions(-) delete mode 100644 src/skins/vector/img/icon-link.svg diff --git a/src/skins/vector/css/common.css b/src/skins/vector/css/common.css index 4b590a56..72648446 100644 --- a/src/skins/vector/css/common.css +++ b/src/skins/vector/css/common.css @@ -46,14 +46,6 @@ h2 { margin-bottom: 16px; } -a { - text-decoration: none; -} - -a:hover { - text-decoration: underline; -} - a:hover, a:link, a:visited { diff --git a/src/skins/vector/css/matrix-react-sdk/views/rooms/EventTile.css b/src/skins/vector/css/matrix-react-sdk/views/rooms/EventTile.css index e1b4e521..5c93f9df 100644 --- a/src/skins/vector/css/matrix-react-sdk/views/rooms/EventTile.css +++ b/src/skins/vector/css/matrix-react-sdk/views/rooms/EventTile.css @@ -64,12 +64,16 @@ limitations under the License. margin-right: 95px; } -/* Removed hidden overflow in order to have hanging elements - in the message stream gutters. This may re-introduce +/* all the overflow-y: hidden; are to trap Zalgos - + but they introduce an implicit overflow-x: auto. + so make that explicitly hidden too to avoid random + horizontal scrollbars occasionally appearing, like in https://github.com/vector-im/vector-web/issues/1154 -*/ + */ .mx_EventTile_content { display: block; + overflow-y: hidden; + overflow-x: hidden; } /* De-zalgoing */ diff --git a/src/skins/vector/css/matrix-react-sdk/views/rooms/LinkPreviewWidget.css b/src/skins/vector/css/matrix-react-sdk/views/rooms/LinkPreviewWidget.css index 01676609..ed470c00 100644 --- a/src/skins/vector/css/matrix-react-sdk/views/rooms/LinkPreviewWidget.css +++ b/src/skins/vector/css/matrix-react-sdk/views/rooms/LinkPreviewWidget.css @@ -15,47 +15,51 @@ limitations under the License. */ .mx_LinkPreviewWidget { - position: relative; margin-top: 15px; + margin-right: 15px; + margin-bottom: 15px; + display: -webkit-flex; + display: flex; + border-left: 4px solid #ddd; + color: #888; } -.mx_LinkPreviewWidget:before { - position: absolute; - z-index: 1; - content: '\0020'; - top: 0; - left: -15px; - width: 1px; - height: 100%; - background-color: #dddddd; -} - -.mx_LinkPreviewWidget_indicator { - position: absolute; - z-index: 1; - top: 2px; - left: -40px; +.mx_LinkPreviewWidget_image { + -webkit-flex: 0 0 100px; + flex: 0 0 100px; + margin-left: 15px; + text-align: center; + cursor: pointer; } .mx_LinkPreviewWidget_caption { - margin-bottom: 10px; + margin-left: 15px; + -webkit-flex: 1 1 auto; + flex: 1 1 auto; } .mx_LinkPreviewWidget_title { + display: inline; font-weight: bold; } .mx_LinkPreviewWidget_siteName { - opacity: 0.5; - font-size: 13px; - line-height: 18px; + display: inline; } -.mx_LinkPreviewWidget_image { - margin-top: 10px; - margin-bottom: 10px; +.mx_LinkPreviewWidget_description { + margin-top: 8px; + white-space: normal; + word-break: break-word; } -.mx_LinkPreviewWidget_image img { +.mx_LinkPreviewWidget_cancel { + visibility: hidden; cursor: pointer; + -webkit-flex: 0 0 40px; + flex: 0 0 40px; +} + +.mx_LinkPreviewWidget:hover .mx_LinkPreviewWidget_cancel { + visibility: visible; } diff --git a/src/skins/vector/img/icon-link.svg b/src/skins/vector/img/icon-link.svg deleted file mode 100644 index 68329978..00000000 --- a/src/skins/vector/img/icon-link.svg +++ /dev/null @@ -1,15 +0,0 @@ - - - - F06866DA-05C9-4E9B-93BD-F25D93FF5A0B - Created with sketchtool. - - - - - - - - - - \ No newline at end of file