From 4ac3483f1a5f77ea6d220247f8c376cb7daa4245 Mon Sep 17 00:00:00 2001 From: Karina Kwiatek Date: Fri, 6 Jan 2023 17:35:49 +0100 Subject: [PATCH] Make sharing options more obvious --- .../retrospring/features/inbox/entry/options.ts | 11 ++++++++++- app/views/inbox/_entry.html.haml | 4 ++-- config/locales/views.en.yml | 2 +- 3 files changed, 13 insertions(+), 4 deletions(-) diff --git a/app/javascript/retrospring/features/inbox/entry/options.ts b/app/javascript/retrospring/features/inbox/entry/options.ts index 133c23fa..00a50905 100644 --- a/app/javascript/retrospring/features/inbox/entry/options.ts +++ b/app/javascript/retrospring/features/inbox/entry/options.ts @@ -4,4 +4,13 @@ export default function optionsEntryHandler(event: Event): void { const options = document.querySelector(`#ib-options-${inboxId}`); options.classList.toggle('d-none'); -} \ No newline at end of file + + const buttonIcon = button.getElementsByTagName('i')[0]; + if (buttonIcon.classList.contains('fa-chevron-down')) { + buttonIcon.classList.remove('fa-chevron-down'); + buttonIcon.classList.add('fa-chevron-up'); + } else { + buttonIcon.classList.remove('fa-chevron-up'); + buttonIcon.classList.add('fa-chevron-down'); + } +} diff --git a/app/views/inbox/_entry.html.haml b/app/views/inbox/_entry.html.haml index 09513f1e..12409502 100644 --- a/app/views/inbox/_entry.html.haml +++ b/app/views/inbox/_entry.html.haml @@ -30,8 +30,8 @@ %button.btn.btn-danger{ name: "ib-destroy", data: { ib_id: i.id } } = t("voc.delete") %button.btn.btn-default{ name: "ib-options", data: { ib_id: i.id, state: :hidden } } - %i.fa.fa-cog - %span.visually-hidden= t(".options") + %i.fa.fa-chevron-down + %span.pe-none= t(".options") .card-footer.d-none{ id: "ib-options-#{i.id}" } %h4= t(".sharing.heading") - if current_user.services.count.positive? diff --git a/config/locales/views.en.yml b/config/locales/views.en.yml index a631a7c3..efe4373a 100644 --- a/config/locales/views.en.yml +++ b/config/locales/views.en.yml @@ -211,7 +211,7 @@ en: answers: one: "1 answer" other: "%{count} answers" - options: "Options" + options: "Sharing Options" placeholder: "Write your answer hereā€¦" sharing: heading: "Sharing"