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

Add popup preview #48

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

rockmanvnx6
Copy link

@rockmanvnx6 rockmanvnx6 commented Oct 3, 2022

Add multi-layer preview popup as suggested from #42 suggested by @Yilunma1

The behavior is similar to obsidian publish (can view a preview within a preview)

Supports both light and dark theme. It only works on /docs/ link, used a hashmap to prevent continuously fetching.

image

Update: added preview based on block id
image

live demo: https://swe.auspham.dev/docs/aws-certified-developer-dva-c01/aws-security-and-encryption/aws-kms-limitations/

Please lemme know if it looks good or anything that needs to be changed

@rockmanvnx6 rockmanvnx6 changed the title feat: preview popup Add popup preview Oct 3, 2022
@DoNotResuscitate
Copy link

@rockmanvnx6 how do you have your subsections properly nested? For example when minimizing the AWS Certified Developer section all of the sub-folders get hidden. Peters example sites doesn't do that.

@DoNotResuscitate
Copy link

I answered my own question - I just checked out out your fork :P

@Yarden-zamir
Copy link

Can this be done the same for the notes in the sidebar?

@rockmanvnx6
Copy link
Author

@Yarden-zmr Should be able to without much changes. I'm not planning to add it in however since it sounds to me this might be dependent on one's preferences. Please feel free to modify the code for your usage.

@Yilunma1
Copy link

Hello ! thanks for the update and nice work ! Can't wait to get it pulled !

janeilagan added a commit to janeilagan/obsidian-zola that referenced this pull request Aug 24, 2023
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

4 participants