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

POC for opening an app inside of a modal #1499

Closed
wants to merge 1 commit into from
Closed

Conversation

ajnart
Copy link
Owner

@ajnart ajnart commented Oct 25, 2023

This is a POC, is it is not meant to be merged

This allows opening apps inside of an iframe that's roughly the size of the window

Need to be addressed

  • Some websites can't be opened in iframes
  • The size of the window should be changed
  • The animation should "pop" slowly out of the app where it's coming from
  • Changing apps quickly shows the content of the previous modal
  • The modal should not be unmounted, allowing the user to quickly jump between apps
  • We should change the behavior settings to be a toggle between "open in same window" / "open in new window" / "open in iframe"

From discord https://www.w3schools.com/tags/att_iframe_sandbox.asp

@github-actions
Copy link

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 17.79% 4540 / 25508
🔵 Statements 17.79% 4540 / 25508
🔵 Functions 6.71% 28 / 417
🔵 Branches 38.98% 115 / 295
File Coverage
File Stmts % Branch % Funcs % Lines Uncovered Lines
Changed Files
src/components/Dashboard/Tiles/Apps/AppTile.tsx 17.46% 100% 0% 17.46% 19-130, 133-157
Generated in workflow #4772

@Meierschlumpf Meierschlumpf marked this pull request as draft October 25, 2023 11:10
@vishalkadam47
Copy link

have a look a this is useful for iframe, extra feature
https://github.com/verlok/vanilla-lazyload

@ajnart
Copy link
Owner Author

ajnart commented Nov 7, 2023

have a look a this is useful for iframe, extra feature verlok/vanilla-lazyload

Thanks for the link but I'm not sure how it can apply. I think the issues I was facing were more related to some kind of ID management with the modals, the iFrames are not loaded when the page opens but on the click of the apps

@vishalkadam47
Copy link

even I'm not sure how it works, instead of modals a full screen iframe within the homarr dashboard on a click of the apps might work, try if that is possible. I'll look for some other links if I find I'll share

@vishalkadam47
Copy link

not sure but look for X-Frame-Options SAMEORIGIN

image

@manuel-rw
Copy link
Collaborator

Closing this due to #1993

@manuel-rw manuel-rw closed this Oct 7, 2024
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.

3 participants