fix resize on rtl page

[kurkle]

Fixes: #6057
Demo

[benmccann]

Can you test an rtl language in the demo when switching to rtl mode? E.g. change the legend from "# of votes" to "عدد الاصوات"

[kurkle]

Can you test an rtl language in the demo when switching to rtl mode? E.g. change the legend from "# of votes" to "عدد الاصوات"

Don't see a difference, do you think there should be?

[simonbrunel]

@kurkle can you add a regression test by duplicating one of these tests.

[kurkle]

@kurkle can you add a regression test by duplicating one of these tests.

Done 🚀

[benmccann]

Can you test an rtl language in the demo when switching to rtl mode? E.g. change the legend from "# of votes" to "عدد الاصوات"

Don't see a difference, do you think there should be?

No, after looking into rtl a bit more it seems there should not be a difference.

I was worried that setting ltr might not do the right thing for users in Arabic or Hewbrew. But as far as I can tell this doesn't change anything. The tooltips look like they might not be aligned properly when the user is using an rtl language, but that's the case on master as well. At least this fixes the resize issue

[simonbrunel]

But as far as I can tell this doesn't change anything

direction: rtl is enforced only on the internal div (.chartjs-size-monitor-*) that are used to detect a resize, thus it doesn't impact the canvas or any parent of the canvas or any user DOM elements.

[simonbrunel]

Thanks @kurkle