search

data-for-change / anyway-newsflash-infographics

Development branch deployed automatically to website:
https://anyway-infographics-staging.web.app/
MIT License
12 stars 23 forks source link

yaxis added #1082

Closed SejoB closed 4 months ago

SejoB commented 4 months ago

image

atalyaalon commented 4 months ago

@MichalOren @galraij what do you think of this change?

MichalOren commented 4 months ago

Thanks @SejoB This is a great job (: I would like to understand regarding the Reference Line, that according to the specification, should appear in the background of the graph (#1050). Is there a problem implementing them ?? Their importance is reflected in two use cases:

  1. When you want to download the graph as an image and then there is no tip tool (no number at all)
  2. In case you choose an 8-year display, then the information is dense and hard to read, the reference line can make it more readable.
atalyaalon commented 4 months ago

@MichalOren we can add a Grid, no problem in that, however @SejoB and I thought it would not look good (so news sites won't necessarily want to use it) If you and @galraij want to add a grid it's no brainer, we can add it

MichalOren commented 4 months ago

Hi @SejoB @atalyaalon @galraij
Can you recomend on any other indication that will give the user more info? According to my check N12 are using Reference Line. For example: image I think this is clear and nice display Would you like to share with us an example

atalyaalon commented 4 months ago

@SejoB can you please take a look at the reference line @MichalOren added in the graph above? Can we add such line (not grid like the one here but a line like the image above)?