Luxon: Edge 16, 17, toFormat ZZZZ and ZZZZZ returns a string with \u200e characters and whitespace

Created on 29 Nov 2018  路  2Comments  路  Source: moment/luxon

I produced this on the Luxon documentation site (https://moment.github.io/luxon/) through Edge 16 and 17 (via Browserstack, though this was found in the wild through user testing) and checked that the locale was en-US.

From within the console:

luxon.DateTime.local().toFormat('ZZZZ');
luxon.DateTime.local().toFormat('ZZZZZ');
encodeURIComponent(luxon.DateTime.local().toFormat('ZZZZ'));

Couldn't copy so I grabbed a screenshot.
luxon

bug

Most helpful comment

Fixed in 7fdd27c, released in 1.8.2.

Thanks for the report!

All 2 comments

Yeah, I think I see the problem. If I'm right, it's an easy fix. It should also not reproduce in Edge 18

Fixed in 7fdd27c, released in 1.8.2.

Thanks for the report!

Was this page helpful?
0 / 5 - 0 ratings