From 30937f4ae3d6d8ebada4610bac72a02f7e1395d7 Mon Sep 17 00:00:00 2001 From: Audric Ackermann Date: Thu, 13 Aug 2020 15:31:51 +1000 Subject: [PATCH] make start new session screen closer to the android one --- _locales/en/messages.json | 17 +++++++++++++---- js/views/last_seen_indicator_view.js | 2 +- stylesheets/_session.scss | 2 +- stylesheets/_session_left_pane.scss | 1 + .../session/SessionClosableOverlay.tsx | 10 +++++----- ts/types/PhoneNumber.ts | 2 +- 6 files changed, 22 insertions(+), 12 deletions(-) diff --git a/_locales/en/messages.json b/_locales/en/messages.json index 24b8bee6e..46115a95b 100644 --- a/_locales/en/messages.json +++ b/_locales/en/messages.json @@ -234,8 +234,14 @@ "description": "Displayed when a user can't send a message because they have left the group" }, "unreadMessage": { - "message": "1 Unread Message", - "description": "Text for unread message separator, just one message" + "message": "$count$ Unread Message", + "description": "Text for unread message separator, just one message", + "placeholders": { + "count": { + "content": "$1", + "example": "5" + } + } }, "unreadMessages": { "message": "$count$ Unread Messages", @@ -1825,7 +1831,7 @@ "remove": { "message": "Remove" }, - "invalidHexId": { + "invalidSessionId": { "message": "Invalid Session ID", "description": "Error string shown when user types an invalid pubkey hex string" }, @@ -1992,8 +1998,11 @@ "enterSessionID": { "message": "Enter Session ID" }, + "enterSessionIDOfRecipient": { + "message": "Enter Session ID of recipient" + }, "usersCanShareTheir...": { - "message": "Users can share their Session ID from their account settings, or by sharing their QR code." + "message": "Users can share their Session ID by going into their account settings and tapping \"Share Session ID\", or by sharing their QR code." }, "message": { "message": "Message" diff --git a/js/views/last_seen_indicator_view.js b/js/views/last_seen_indicator_view.js index 12049d59e..de475aff5 100644 --- a/js/views/last_seen_indicator_view.js +++ b/js/views/last_seen_indicator_view.js @@ -25,7 +25,7 @@ render_attributes() { const unreadMessages = this.count === 1 - ? i18n('unreadMessage') + ? i18n('unreadMessage', [this.count]) : i18n('unreadMessages', [this.count]); return { diff --git a/stylesheets/_session.scss b/stylesheets/_session.scss index 5838f929d..c611aa0da 100644 --- a/stylesheets/_session.scss +++ b/stylesheets/_session.scss @@ -1353,7 +1353,7 @@ label { padding: 0px 5px 20px 5px; &.session-id-editable-textarea:placeholder-shown { - padding: 20px 5px 0px 5px; + padding: 10px 5px 0px 5px; } &.group-id-editable-textarea { diff --git a/stylesheets/_session_left_pane.scss b/stylesheets/_session_left_pane.scss index 71fb51a5e..c17d3e1f9 100644 --- a/stylesheets/_session_left_pane.scss +++ b/stylesheets/_session_left_pane.scss @@ -352,6 +352,7 @@ $session-compose-margin: 20px; .user-search-dropdown { width: 100%; + min-height: 34px; flex-grow: 1; overflow-y: auto; } diff --git a/ts/components/session/SessionClosableOverlay.tsx b/ts/components/session/SessionClosableOverlay.tsx index 2e6744b38..d69b539d0 100644 --- a/ts/components/session/SessionClosableOverlay.tsx +++ b/ts/components/session/SessionClosableOverlay.tsx @@ -134,14 +134,14 @@ export class SessionClosableOverlay extends React.Component { buttonText = window.i18n('next'); descriptionLong = window.i18n('usersCanShareTheir...'); subtitle = window.i18n('enterSessionID'); - placeholder = window.i18n('enterSessionID'); + placeholder = window.i18n('enterSessionIDOfRecipient'); break; case 'contact': title = window.i18n('addContact'); buttonText = window.i18n('next'); descriptionLong = window.i18n('usersCanShareTheir...'); subtitle = window.i18n('enterSessionID'); - placeholder = window.i18n('enterSessionID'); + placeholder = window.i18n('enterSessionIDOfRecipient'); break; case 'open-group': title = window.i18n('joinOpenGroup'); @@ -235,9 +235,9 @@ export class SessionClosableOverlay extends React.Component { {descriptionLong && (
{descriptionLong}
)} - {isMessageView &&

{window.i18n('or')}

} - - {isMessageView && ( + {isMessageView && false &&

{window.i18n('or')}

} + {/* FIXME enable back those two items when they are working */} + {isMessageView && false && (