mathjazz / pontoon

In-place localization tool
https://pontoon.mozilla.org/
BSD 3-Clause "New" or "Revised" License
3 stars 1 forks source link

Ability to select multiple filters/entities should be more intuitive #1084

Open mathjazz opened 7 years ago

mathjazz commented 7 years ago

This issue was created automatically by a script.

Bug 1377887

Bug Reporter: @flodolo CC: @adngdb, @mathjazz, monikamaheshwari1996@gmail.com

Currently, when hovering one of the filters (e.g. "Has Suggestions"), the text changes color but there's no indication that the icon can work as a checkbox.

When hovering the icon, it transforms into a check mark.

I think it would be more intuitive to:

Strings already have an icon that resembles a checkbox, maybe it should turn into a more empty checkbox on hover (i.e. only colored border, not full rectangle).

mathjazz commented 5 years ago

Comment Author: @mathjazz

Let's indeed turn filter icons into regular squares when hovering the text of the filter item.

mathjazz commented 5 years ago

Comment Author: justine <vee27mm@gmail.com>

(In reply to Matjaz Horvat [:mathjazz] from comment #1)

Let's indeed turn filter icons into regular squares when hovering the text of the filter item.

i was trying to find the filter button in pontoon i seem that i cant locate it can you help with that

mathjazz commented 5 years ago

Comment Author: @mathjazz

The filter icon is the "funnel" located in front of the "Search in All" placeholder in the translate view, e.g.: http://localhost:8000/projects/tutorial/playground/

You can read more about how filters work in the docs: https://mozilla-l10n.github.io/localizer-documentation/tools/pontoon/search_filters.html#filters

The idea of the bug is to change the icon into a square one you hover over filter item, making it more obvious that the icons act as checkboxes for selecting multiple filters.

mathjazz commented 5 years ago

Comment Author: justine <vee27mm@gmail.com>

Thank you now i understand, in which component is the icon located? as in which file in the source code?

mathjazz commented 5 years ago

Comment Author: @mathjazz

FiltersPanel: https://github.com/mozilla/pontoon/blob/master/frontend/src/modules/search/components/FiltersPanel.js

You might be able to fix the bug with CSS only: https://github.com/mozilla/pontoon/blob/master/frontend/src/modules/search/components/FiltersPanel.css

mathjazz commented 4 years ago

Comment Author: Monika Maheshwari [:MonikaMaheshwari] <monikamaheshwari1996@gmail.com>

I would like to work on this issue :)

mathjazz commented 4 years ago

Comment Author: @adngdb

Hi Justine, because this has been stalled for a while now, I'm assigning it to Monika. If you want to continue working on Pontoon, please let us know and we'll find you another bug.

mathjazz commented 4 years ago

Comment Author: GitHub Bugzilla PR Linker <pulgasaur@mozilla.bugs>

Created attachment 9104852 Link to GitHub pull-request: https://github.com/mozilla/pontoon/pull/1462

Attached file: file_333531271.txt (text/x-github-pull-request, 44 bytes) Description: Link to GitHub pull-request: https://github.com/mozilla/pontoon/pull/1462

mathjazz commented 4 years ago

Comment Author: @mathjazz

As commented in the closed PR, we need to address some open questions before continuing.