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

fix empty block children is not clickable issue #3593

Merged
merged 2 commits into from Dec 27, 2021
Merged

fix empty block children is not clickable issue #3593

merged 2 commits into from Dec 27, 2021

Conversation

pengx17
Copy link
Contributor

@pengx17 pengx17 commented Dec 26, 2021

In #3559, block children are wrapped into a flex container, but it previously took all available width, after the change it only takes its children width. This issue will make the width of block children smaller. For empty block children, they are not clickable.
image

@andelf
Copy link
Collaborator

andelf commented Dec 27, 2021

I've been using a temporary fix in my custom.css

.block-children {
  width: 100%;
}

Not quite sure which fix is a little better

The same as w-full.

@pengx17
Copy link
Contributor Author

pengx17 commented Dec 27, 2021

Do we need a e2e case for it?

@andelf
Copy link
Collaborator

andelf commented Dec 27, 2021

I prefer not to test the geometric properties of an element. This is a buggy modification introduced by accident. CSS modifications should be carefully reviewed.

@tiensonqin tiensonqin self-requested a review December 27, 2021 02:31
@tiensonqin tiensonqin merged commit 3924f5c into logseq:master Dec 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants