Heidi Kasemir comments

Results 5 comments of


                                            Heidi Kasemir

Change named arguments syntax to $name = value

This one leaves me a touch confused, and ultimately I prefer the old syntax, but this might just be because it parallels javascript so well. my reactions: - I like...

Change named arguments syntax to $name = value

cool, sounds good! Those are good points, and I think after just a little mental re-framing, it's easy to see the benefits of this simplification.

Remove the requirement for superfluous equal sign after an identifier

I think I agree with this one. To me, it feels cleaner to have ``` email-input .placeholder = Email Address ``` rather than ``` email-input = .placeholder = Email Address...

invalid relative reference to formik

mm, not a good one. I just recreated the package locally.

i18next-pseudo is not compatible with i18next-browser-languagedetector, pseudo doesnt work when .use(languageDetector) is used, if i remove that then it works nicely

yup - initializing it as ``` new Pseudo({ enabled: true, languageToPseudo: 'en-US', }) ``` fixed it for me