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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(blocks): links made clickable, z-index clean up #772 #778

Merged

Conversation

arkRedM
Copy link
Contributor

@arkRedM arkRedM commented Mar 9, 2021

https://www.loom.com/share/3b3d566401eb45a38e8ea63e74c309e2

This issue forced to fix a root issue about how z-index is handled in too many places. This was well buried inside the hierarchy. In the loom, I tried to explain the logic for z-index, textarea and block interactions and can probably act as a reference. Cut down a good amount of hacky code with this 馃槍

@tangjeff0
Copy link
Collaborator

tangjeff0 commented Mar 9, 2021

Doesn't seem to work for Linked Refs (or TODOs inside Linked Ref).

image

@arkRedM arkRedM force-pushed the fix/block-embed-z-index-issue branch from 7af6c92 to 84978fc Compare March 11, 2021 00:43
@arkRedM
Copy link
Contributor Author

arkRedM commented Mar 11, 2021

Doesn't seem to work for Linked Refs

Hey @tangjeff0 can you check again? I am unable to reproduce this issue.
One theory is stylefy doesn't refresh completely even when changed(some caching at lib level)
Try and reload(CMD+R) when you switch to the branch or open a new window altogether

@tangjeff0 tangjeff0 merged commit 8cabc5f into athensresearch:master Mar 11, 2021
@tangjeff0
Copy link
Collaborator

CACHE IS KING

@tangjeff0
Copy link
Collaborator

I don't think clicking TODO/DONE works though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants