map hover properties and design

noamoss commented 4 years ago

blocking #306

available data points:

city/town name
city/town population size
number of reports - this week
number of report - total
symptoms_ratio,
symptoms_ratio_confidence
num_reports_weighted
symptoms_ratio_weighted
symptoms_ratio_confidence_weighted

"הסיכוי בעיר/שכונה לתוצאה חיובית בבדיקת אבחון לנגיף הקורונה, על-פי מודל מכון ויצמן לשקלול תסמינים"

notice: city/town name and other labels should fit the user locale follow the map places names rules and fallbacks

Design:

In English we would still use Sunday-Saturday layout as this is how we live in Israel
In the first milliseconds the calendar would load to indicate the time context:

The hover effect would manipulate the border of the polygon:

noamoss commented 4 years ago

@mushon I am willing to give up on my original thought to include here not only the research result but also a weekly trend of reports, took it out of the list.

mushon commented 4 years ago

Check proposed design: https://zpl.io/aXvMd4E

noamoss commented 4 years ago

@mushon

users won't understand dots are calendar (I am certain, as one potential user actually asked me what are they).
Also, I think we should consider to show the connection between the value (13) to the level. Maybe even moving the number next to the label -

maybe something like

רמת תסמינים
גבעת שמואל
ש ו  ה ד ג  ב א
o o o o o o o
o o o o o o o
o o o o O
      בינונית
       17

or just adding the number, once again, in the lower textual label
or a little scala next to the border to remind the meaning of the color

noamoss commented 4 years ago

@mushon also

please clarify if X reports this week means starting Sunday, or during the last 7 days.
enable long city/town - maybe leave 2 lines for that

noamoss commented 4 years ago

@mushon last point (hopefully): some polygons are very narrow or very tight. Please show the polygon hover color/border effect as well.

mushon commented 4 years ago

I made some updates to the design. Mainly the days would appear to complete the week block for the start and the end of the weeks:

Reports starting on a Sunday would have the previous week as labels:

In the first milliseconds the calendar would load to indicate the time context:

In English we would still use Sunday-Saturday layout as this is how we live in Israel:

The hover effect would manipulate the border of the polygon:

I think it's good to go… https://zpl.io/aXvMd4E

noamoss commented 4 years ago

@mushon

Mainly the days would appear to complete the week block for the start and the end of the weeks:

I think it is overcomplicated for the standard user: you assume they will catch the relations between the first row and the last row of the calendar. Why not keep it simple, like in this example?

(accepted and updated the description with all the other suggestions)

Also, you skipped these two comments:

please clarify if X reports this week means starting Sunday, or during the last 7 days.

I would change to בשבוע האחרון

enable long city/town - maybe leave 2 lines for that

akariv commented 4 years ago

אפשר להחליף את "אמינות ניתוח" ל'וודאות"

On Tue, May 12, 2020, 21:13 noamoss notifications@github.com wrote:

@mushon https://github.com/mushon

Mainly the days would appear to complete the week block for the start and the end of the weeks:

I think it is overcomplicated for the standard user: you assume they will catch the relations between the first row and the last row of the calendar. Why not keep it simple, like in this example?

[image: image] https://user-images.githubusercontent.com/3581741/81729528-ca666400-9494-11ea-803e-46ca58ab7cd8.png

(accepted and updated the description with all the other suggestions)

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/hasadna/avid-covider/issues/309#issuecomment-627507522, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACAY5KZ22424ZWGM25Z3LTRRGGVBANCNFSM4M4BXYIQ .

noamoss commented 4 years ago

@akariv

אפשר להחליף את "אמינות ניתוח" ל'וודאות"

אני חושש ש"ודאות נמוכה" או "ודאות בינונית" אלה לא ביטויים ברורים. מה הבעיה שאתה רואה ב"אמינות ניתוח..."?

mushon commented 4 years ago

It’s possible, but it would cost us another line. Maybe that’s ok.

mushon commented 4 years ago

אפשר להחליף את "אמינות ניתוח" ל'וודאות

בהחלט

akariv commented 4 years ago

״אמינות ניתוח״ זה מושג שבוודאות אנשים לא משתמשים בו ביום יום. הוא גם בוודאות לא נכון - הניתוח אמין בכל מקרה, השאלה כמה החוקרים בטוח שהמספר מייצג את המציאות - האם זה וודאי או שלא כל כך. אני חושב שיש וודאות נמוכה שאנשים יבינו מה זה אומר ווודאות גבוהה שמרי תקבל הרבה שאלות אם נשתמש בניסוח הנוכחי.

On Tue, May 12, 2020 at 9:52 PM Mushon Zer-Aviv notifications@github.com wrote:

אפשר להחליף את "אמינות ניתוח" ל'וודאות

בהחלט

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/hasadna/avid-covider/issues/309#issuecomment-627527151, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACAY5NEQ64YY3IFL476F23RRGLE7ANCNFSM4M4BXYIQ .

noamoss commented 4 years ago

נדמה לי ששניכם נוטים להצעה של אדם, נלך איתה.

@mushon בבקשה אל תפספס את זה: https://github.com/hasadna/avid-covider/issues/309#issuecomment-627507522

hasadna / avid-covider

map hover properties and design #309

available data points: