Przemek Kuliga

Results 16 comments of Przemek Kuliga

> However I wonder if @Automattic/zap tinkering with the cookies in those PRs could be the issue, as the dates seem to match: https://github.com/Automattic/jetpack/pull/35849 https://github.com/Automattic/jetpack/pull/35852 Hi @millerf, not sure what...

There is a similar block in Gutenberg already: https://github.com/WordPress/gutenberg/tree/7fce392ac694f4dc66ba6300f7b3356e9cdc2123/packages/block-library/src/loginout We could take some ideas from it.

> BTW Block name can be sentence-case; "Subscriber login". Not sure about this, all blocks I checked are title-cased. I prefer to be consistent with the rest of the platform...

I merged the first iteration of it and added some TODOs for the further iterations (need to scroll to the top to see it). Feel free to modify it if...

> One feature that is important to add is ability to add the block to the navigation block: Added it to TODOS, thanks!

Hi folks, before releasing it would be nice to have some testing. To do that you need to have access to the Beta blocks. You can enable it like this...

Closing this one since the block has already been released. Further work will be done under separate issues.

> * other toggle copy ends in `.`, other doesn't. Which way should it be? > * Starting in lower capital feels a bit weird for translators; it might be...

@crisbusquets adding a main toggle introduces too much complexity. Let's do it when we have more toggles.

@crisbusquets where should be the "Let visitors subscribe to new posts and comments via email" toggle in the new structure? AFAIK, this toggle works now as a "main toggle" to...