From dcc64f44f9f81d32c8071c6bdac86546eaeb57a0 Mon Sep 17 00:00:00 2001 From: Maxime Quandalle Date: Wed, 27 May 2015 17:17:00 +0200 Subject: UI improvements * Implement visibility choice on board creation; * Rework the board header bar. Remove links to un-implemented features; * Implement a board star counter (visible if the board have >2 stars); * Define a new icon (a thin cross) to close elements; * Remove $(document).on('mouseover') event handlers that were basically fired hundreds of times for nothing, we now define a proper Tracker dependency to execute jquery-ui plugin initialization only when something has changed; * Bug fixes related to list scrolling. --- client/components/forms/forms.styl | 23 +++-------------------- client/components/forms/inlinedform.js | 7 ++++--- 2 files changed, 7 insertions(+), 23 deletions(-) (limited to 'client/components/forms') diff --git a/client/components/forms/forms.styl b/client/components/forms/forms.styl index 1084a4a6..c572863d 100644 --- a/client/components/forms/forms.styl +++ b/client/components/forms/forms.styl @@ -229,26 +229,9 @@ textarea padding-top: 5px padding-bottom: 5px - i.fa.fa-times - font-size: 20px - - .option - border-color: transparent - border-radius: 3px - color: #8c8c8c - display: block - float: right - height: 30px - line-height: 30px - padding: 0 8px - margin: 0 2px - - &:hover - background-color: #dbdbdb - color: #4d4d4d - - &:active - background-color: #ccc + .fa-times-thin + font-size: 26px + margin: 3px 4px .button-link background: #fff diff --git a/client/components/forms/inlinedform.js b/client/components/forms/inlinedform.js index b84952d2..f2774084 100644 --- a/client/components/forms/inlinedform.js +++ b/client/components/forms/inlinedform.js @@ -30,11 +30,12 @@ BlazeComponent.extendComponent({ this.isOpen = new ReactiveVar(false); }, + onDestroyed: function() { + currentlyOpenedForm.set(null); + }, + open: function() { // Close currently opened form, if any - // if (currentlyOpenedForm.get() !== null) { - // currentlyOpenedForm.get().close(); - // } EscapeActions.executeLowerThan('inlinedForm'); this.isOpen.set(true); currentlyOpenedForm.set(this); -- cgit v1.2.3-1-g7c22