Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert Clickthrough #16888

Closed
wants to merge 1 commit into from
Closed

Revert Clickthrough #16888

wants to merge 1 commit into from

Conversation

youknowriad
Copy link
Contributor

This reverts commit c5e8784.

Really sad to push that PR but I think the "clickthrough" experiment is not that succesfull especially when editing complex templates.

I had a number of conflicts when reverting, so I'd appreciate a sanity check for regressions.

@youknowriad youknowriad added the [Feature] Nested / Inner Blocks Anything related to the experience of nested/inner blocks inside a larger container, like Group or P label Aug 2, 2019
@youknowriad youknowriad self-assigned this Aug 2, 2019
@kjellr
Copy link
Contributor

kjellr commented Aug 2, 2019

@youknowriad I believe the clickthrough PR may have included something to speed up the addition/removal of the has-child-selected class. @gziolo originally helped us implement a temporary hack to get around that in #14961, but were able to remove it after clickthrough was merged.

Trying this PR out now, I'm seeing those delays return:

select

swissspidy
swissspidy previously approved these changes Aug 16, 2019
@youknowriad
Copy link
Contributor Author

superseded by #17239

@youknowriad youknowriad deleted the revert/clickthrough branch September 11, 2019 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Nested / Inner Blocks Anything related to the experience of nested/inner blocks inside a larger container, like Group or P
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants