Skip to content

Conversation

@linusha
Copy link
Collaborator

@linusha linusha commented Jan 11, 2026

Groups

image

Countries

image

Embed Section

image
  1. I personally would have preferred to have the country chart rotated as I find that easier to read than a very long list, but the stacked column chart works very different than the stacked bar chart and I did not find a way to rotate that one.
  2. Having flags in the chart would have also been cool, but that required having emoji codes for the flags in the data. Maybe some other time...
  3. The code for the DataWrapperLinkList is for sure not the cleanest solution to this. I would appreciate an opinion on what you say would be the best implementation.

@linusha linusha requested a review from tillprochaska January 11, 2026 18:27
@linusha linusha changed the title More datawrapper embeds Datawrapper Embeds for Groups and Countries Jan 11, 2026
@linusha linusha force-pushed the more-datawrapper-embeds branch from 3ad34d8 to 4c8ef96 Compare January 11, 2026 18:29
@tillprochaska
Copy link
Collaborator

I personally would have preferred to have the country chart rotated as I find that easier to read than a very long list, but the stacked column chart works very different than the stacked bar chart and I did not find a way to rotate that one.

Another argument for a bar chart is that it works much better on narrow/mobile screens.

The code for the DataWrapperLinkList is for sure not the cleanest solution to this. I would appreciate an opinion on what you say would be the best implementation.

I’ll take a look!

@linusha
Copy link
Collaborator Author

linusha commented Jan 11, 2026

Another argument for a bar chart is that it works much better on narrow/mobile screens.

True!

Regarding the texts - I'll change the wordings everywhere to your suggestions.

@linusha linusha force-pushed the more-datawrapper-embeds branch from 2e29c27 to b44667e Compare January 11, 2026 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants