From 5e01fe402505ece6c37cd3989b15c8547fec66c4 Mon Sep 17 00:00:00 2001 From: David Jay Date: Thu, 22 Dec 2016 11:01:59 -0800 Subject: [PATCH] Fixing bug in text box display. --- client/coral-embed-stream/style/default.css | 4 ++-- client/coral-plugin-flags/FlagBio.js | 2 +- client/coral-plugin-flags/FlagButton.js | 15 +++++---------- client/coral-plugin-flags/FlagComment.js | 2 +- 4 files changed, 9 insertions(+), 14 deletions(-) diff --git a/client/coral-embed-stream/style/default.css b/client/coral-embed-stream/style/default.css index 20a5e02ac..28b2066e6 100644 --- a/client/coral-embed-stream/style/default.css +++ b/client/coral-embed-stream/style/default.css @@ -240,7 +240,7 @@ hr { .coral-plugin-flags-popup-radio-label { margin:5px; font-weight: 700; - font-size: 1.16rem; + font-size: .9rem; } .coral-plugin-flags-popup-counter { @@ -254,7 +254,7 @@ hr { margin-top: 10px; } -.coral-plugin-flags-other-text { +.coral-plugin-flags-reason-text { margin-left: 20px; margin-top: 5px; width: 75%; diff --git a/client/coral-plugin-flags/FlagBio.js b/client/coral-plugin-flags/FlagBio.js index 90cc11a7a..edc0a5286 100644 --- a/client/coral-plugin-flags/FlagBio.js +++ b/client/coral-plugin-flags/FlagBio.js @@ -18,7 +18,7 @@ const getPopupMenu = [ {val: 'other', text: lang.t('other')} ], button: lang.t('continue'), - sets: 'detail' + sets: 'reason' }; }, () => { diff --git a/client/coral-plugin-flags/FlagButton.js b/client/coral-plugin-flags/FlagButton.js index 80a00a2d9..097ac9d5b 100644 --- a/client/coral-plugin-flags/FlagButton.js +++ b/client/coral-plugin-flags/FlagButton.js @@ -40,7 +40,7 @@ class FlagButton extends Component { // If itemType and reason are both set, post the action if (itemType && reason && !posted) { - + // Set the text from the "other" field if it exists. let item_id; switch(itemType) { @@ -71,11 +71,6 @@ class FlagButton extends Component { onPopupOptionClick = (sets) => (e) => { - // If the "other" option is clicked, show the other textbox - if(sets === 'reason' && e.target.value === 'other') { - this.setState({showOther: true}); - } - // If flagging a user, indicate that this is referencing the username rather than the bio if(sets === 'itemType' && e.target.value === 'user') { this.setState({field: 'username'}); @@ -93,7 +88,7 @@ class FlagButton extends Component { this.setState({[sets]: e.target.value}); } - onOtherTextChange = (e) => { + onNoteTextChange = (e) => { this.setState({note: e.target.value}); } @@ -147,11 +142,11 @@ class FlagButton extends Component {
-