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

Enhancement: Improve viewer integration for images #1624

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Koc
Copy link

@Koc Koc commented Feb 19, 2025

This PR adds viewer with lightbox for images that included into markdown.

image

Signed-off-by: Kostiantyn Miakshyn <molodchick@gmail.com>
@Koc Koc requested review from enjeck and blizzz as code owners February 19, 2025 22:56
@@ -263,3 +274,16 @@ export default {
padding-bottom: 4px; // to make it nice with the padding-top
}
</style>

<style lang="scss">
.preview [data-node-view-content] {
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's hide this for view only

@blizzz blizzz requested a review from a team February 20, 2025 11:47
@blizzz blizzz added 3. to review Waiting for reviews enhancement New feature or request labels Feb 20, 2025
Comment on lines +243 to +245
const filePath = $img.src.match(/\/dav\/files\/(.+?)\/(.+)/)[2]
OCA.Viewer.open({ path: filePath })
})
Copy link
Contributor

@enjeck enjeck Feb 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably a n00b question: After uploading an image to files, how do I get a filepath that matches this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants