From b15ea6e42ed75884835c65f302d000330ee3c9bf Mon Sep 17 00:00:00 2001 From: Beatrice Olivera <31216493+beatriceo@users.noreply.github.com> Date: Thu, 6 Sep 2018 01:59:16 +0100 Subject: [PATCH] Revert "Revert "Chat css"" --- .gitignore | 1 - app/assets/stylesheets/components/_hand.scss | 2 +- app/assets/stylesheets/pages/_home.scss | 8 +++ app/assets/stylesheets/pages/_index.scss | 1 + app/assets/stylesheets/pages/_messages.scss | 65 ++++++++++++++++++++ app/controllers/pages_controller.rb | 11 +++- app/javascript/packs/chatrooms.js | 63 ++++++++++++++----- app/javascript/packs/scroll.js | 10 +++ app/views/chat_rooms/show.html.erb | 57 +++++++++-------- translation-credentials.json | 14 +++++ 10 files changed, 188 insertions(+), 44 deletions(-) create mode 100644 app/assets/stylesheets/pages/_messages.scss create mode 100644 app/javascript/packs/scroll.js create mode 100644 translation-credentials.json diff --git a/.gitignore b/.gitignore index 127293d..2a08fef 100644 --- a/.gitignore +++ b/.gitignore @@ -14,4 +14,3 @@ yarn-error.log .byebug_history .env* .env* -translation-credentials.json diff --git a/app/assets/stylesheets/components/_hand.scss b/app/assets/stylesheets/components/_hand.scss index 49312e2..74a3a5f 100644 --- a/app/assets/stylesheets/components/_hand.scss +++ b/app/assets/stylesheets/components/_hand.scss @@ -2,7 +2,7 @@ position: absolute; bottom: 20px; z-index: 10000000000; - left: 48%; + left: 10%; font-size: 26px; cursor: pointer; } diff --git a/app/assets/stylesheets/pages/_home.scss b/app/assets/stylesheets/pages/_home.scss index a3fa38e..ee6d1a9 100755 --- a/app/assets/stylesheets/pages/_home.scss +++ b/app/assets/stylesheets/pages/_home.scss @@ -6,6 +6,14 @@ video { #remote-video-container { float: left; + i { + text-align: center; + background-color: $primary; + border-radius: 50%; + padding: 15px; + z-index: 999999; + color: white; + } } #local-video { diff --git a/app/assets/stylesheets/pages/_index.scss b/app/assets/stylesheets/pages/_index.scss index 52316dd..96bb455 100644 --- a/app/assets/stylesheets/pages/_index.scss +++ b/app/assets/stylesheets/pages/_index.scss @@ -2,6 +2,7 @@ @import "home"; @import "contacts"; @import "settings"; +@import "messages"; .accept-call-button { border-style: none; diff --git a/app/assets/stylesheets/pages/_messages.scss b/app/assets/stylesheets/pages/_messages.scss new file mode 100644 index 0000000..8c2b0b7 --- /dev/null +++ b/app/assets/stylesheets/pages/_messages.scss @@ -0,0 +1,65 @@ +.messages-container-container { + background-color: $light-gray; + width: 100%; + height: 100vh; + position: relative; + .chat { + position: absolute; + bottom: 0px; + } +} + +#messages-container { + overflow: scroll; + height: 95%; + padding: 10px; +} + +select:not([multiple]) { + -webkit-appearance: none; + -moz-appearance: none; + background-position: right 50%; + background-repeat: no-repeat; + background-image: url(data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAA4AAAAMCAYAAABSgIzaAAAAGXRFWHRTb2Z0d2FyZQBBZG9iZSBJbWFnZVJlYWR5ccllPAAAAyJpVFh0WE1MOmNvbS5hZG9iZS54bXAAAAAAADw/eHBhY2tldCBiZWdpbj0i77u/IiBpZD0iVzVNME1wQ2VoaUh6cmVTek5UY3prYzlkIj8+IDx4OnhtcG1ldGEgeG1sbnM6eD0iYWRvYmU6bnM6bWV0YS8iIHg6eG1wdGs9IkFkb2JlIFhNUCBDb3JlIDUuMC1jMDYwIDYxLjEzNDc3NywgMjAxMC8wMi8xMi0xNzozMjowMCAgICAgICAgIj4gPHJkZjpSREYgeG1sbnM6cmRmPSJodHRwOi8vd3d3LnczLm9yZy8xOTk5LzAyLzIyLXJkZi1zeW50YXgtbnMjIj4gPHJkZjpEZXNjcmlwdGlvbiByZGY6YWJvdXQ9IiIgeG1sbnM6eG1wPSJodHRwOi8vbnMuYWRvYmUuY29tL3hhcC8xLjAvIiB4bWxuczp4bXBNTT0iaHR0cDovL25zLmFkb2JlLmNvbS94YXAvMS4wL21tLyIgeG1sbnM6c3RSZWY9Imh0dHA6Ly9ucy5hZG9iZS5jb20veGFwLzEuMC9zVHlwZS9SZXNvdXJjZVJlZiMiIHhtcDpDcmVhdG9yVG9vbD0iQWRvYmUgUGhvdG9zaG9wIENTNSBNYWNpbnRvc2giIHhtcE1NOkluc3RhbmNlSUQ9InhtcC5paWQ6NDZFNDEwNjlGNzFEMTFFMkJEQ0VDRTM1N0RCMzMyMkIiIHhtcE1NOkRvY3VtZW50SUQ9InhtcC5kaWQ6NDZFNDEwNkFGNzFEMTFFMkJEQ0VDRTM1N0RCMzMyMkIiPiA8eG1wTU06RGVyaXZlZEZyb20gc3RSZWY6aW5zdGFuY2VJRD0ieG1wLmlpZDo0NkU0MTA2N0Y3MUQxMUUyQkRDRUNFMzU3REIzMzIyQiIgc3RSZWY6ZG9jdW1lbnRJRD0ieG1wLmRpZDo0NkU0MTA2OEY3MUQxMUUyQkRDRUNFMzU3REIzMzIyQiIvPiA8L3JkZjpEZXNjcmlwdGlvbj4gPC9yZGY6UkRGPiA8L3g6eG1wbWV0YT4gPD94cGFja2V0IGVuZD0iciI/PuGsgwQAAAA5SURBVHjaYvz//z8DOYCJgUxAf42MQIzTk0D/M+KzkRGPoQSdykiKJrBGpOhgJFYTWNEIiEeAAAMAzNENEOH+do8AAAAASUVORK5CYII=); + padding: .5em; + padding-right: 1.5em; +} + +#language-1 { + border-radius: 0; +} + +.avatar-sm { + width: 35px; + height: 35px; +} + +.messageDiv { + display: flex; + align-items: center; + padding-top: 10px; + padding-bottom: 10px; + + p { + padding-top: 2px; + padding-bottom: 2px; + padding-left: 8px; + padding-right: 8px; + background-color: #d4d5d6; + color: $navbar-background; + border-radius: 2px; + } + + &.right { + flex-direction: row-reverse; + p { + background-color: $primary; + color: white !important; + } + } + +} + +.margin-left { + margin-left: 10px; +} diff --git a/app/controllers/pages_controller.rb b/app/controllers/pages_controller.rb index 9577412..1ad249d 100644 --- a/app/controllers/pages_controller.rb +++ b/app/controllers/pages_controller.rb @@ -50,7 +50,9 @@ class PagesController < ApplicationController chat_message: { message: 'test', user_info: user_info, - time_stamp: Time.now } + time_stamp: Time.now, + photo_url: photo_url + } }) head :ok end @@ -80,7 +82,8 @@ class PagesController < ApplicationController ActionCable.server.broadcast "chat_room_#{params[:chat_room_id]}", { translated_message: translated_message, - userId: userId + userId: userId, + photo_url: params[:photo_url] } end @@ -90,6 +93,7 @@ class PagesController < ApplicationController chatroom = 'chat_room_' + params[:chat_room_id] puts params user_info = {} + photo_url = current_user.photo.url || "https://upload.wikimedia.org/wikipedia/commons/8/89/Portrait_Placeholder.png" if current_user.first_name.nil? || current_user.last_name.nil? user_info[:name] = current_user.email @@ -102,7 +106,8 @@ class PagesController < ApplicationController message: params[:message], user_info: user_info, time_stamp: Time.now.strftime("%H:%M"), - userId: current_user.id + userId: current_user.id, + photo_url: photo_url } }) head :ok diff --git a/app/javascript/packs/chatrooms.js b/app/javascript/packs/chatrooms.js index c5e1267..8853946 100644 --- a/app/javascript/packs/chatrooms.js +++ b/app/javascript/packs/chatrooms.js @@ -1,5 +1,5 @@ import ActionCable from 'actioncable' - +// import { scrollLastMessageIntoView } from './chatrooms' // create App object with key cable == new cosumer (function() { window.App || (window.App = {}); @@ -24,14 +24,35 @@ App['chatroom' + chatroomId] = App.cable.subscriptions.create({ const chatMessage = data["chat_message"] const message = `${chatMessage["message"]}` const messagesContainer = document.getElementById('messages-container') + + // Div Settings + const messageDiv = document.createElement("div") + messageDiv.classList.add("messageDiv") + messageDiv.classList.add("right") + + // Photo Settings + const photo = document.createElement("img") + photo.src = chatMessage["photo_url"] + photo.classList.add("img-circle") + photo.classList.add("avatar-sm") + photo.classList.add("margin-left") + + // Text Settings const messageElement = document.createElement("p") + messageElement.classList.add("message") + messageElement.classList.add("no-margin") messageElement.innerText = message - messagesContainer.appendChild(messageElement) - } - else if (data["chat_message"] && data["chat_message"]["userId"] != userId) { + + // Add message and photo to div + messageDiv.appendChild(photo) + messageDiv.appendChild(messageElement) + + // Add message to container + messagesContainer.appendChild(messageDiv) + + } else if (data["chat_message"] && data["chat_message"]["userId"] != userId) { const chatMessage = data["chat_message"] const target = document.getElementById('language-1').value - const message = `${chatMessage["message"]}` fetch(`/chat_rooms/${chatroomId}/translate_message` , { @@ -39,21 +60,33 @@ App['chatroom' + chatroomId] = App.cable.subscriptions.create({ body: JSON.stringify({ message: message, target: target, - userId: userId + userId: userId, + photo_url: chatMessage["photo_url"] }), headers: { "content-type": "application/json", "X-CSRF-Token": document.querySelector('meta[name=csrf-token]').content } }) - } else if (data["translation"] && data["userId"] == userId) { - if (data["input"] == 1) { - document.getElementById('language-2-input').value = data["translation"].text - } else { - document.getElementById('language-1-input').value = data["translation"].text - } } else if (data["translated_message"] && data["userId"] == userId) { const messagesContainer = document.getElementById('messages-container') - const message = document.createElement("p") - message.innerText = data["translated_message"] - messagesContainer.appendChild(message) + + const messageDiv = document.createElement("div") + messageDiv.classList.add("messageDiv") + + const photo = document.createElement("img") + photo.src = data["photo_url"] + photo.classList.add("img-circle") + photo.classList.add("avatar-sm") + photo.classList.add("margin-right") + + const messageElement = document.createElement("p") + messageElement.classList.add("message") + messageElement.classList.add("message") + messageElement.classList.add("no-margin") + messageElement.innerText = data["translated_message"] + + messageDiv.appendChild(photo) + messageDiv.appendChild(messageElement) + + messagesContainer.appendChild(messageDiv) } else { // console.log(data) } diff --git a/app/javascript/packs/scroll.js b/app/javascript/packs/scroll.js new file mode 100644 index 0000000..9a8f982 --- /dev/null +++ b/app/javascript/packs/scroll.js @@ -0,0 +1,10 @@ +function scrollLastMessageIntoView() { + const messages = document.querySelectorAll('.message'); + const lastMessage = messages[messages.length - 1]; + + if (lastMessage !== undefined) { + lastMessage.scrollIntoView(); + } +} + +export { scrollLastMessageIntoView } diff --git a/app/views/chat_rooms/show.html.erb b/app/views/chat_rooms/show.html.erb index 304a277..d75555e 100644 --- a/app/views/chat_rooms/show.html.erb +++ b/app/views/chat_rooms/show.html.erb @@ -4,36 +4,45 @@
-