Skip to content

Proposal: WIP: fullscreen cell and output#2801

Closed
cocoa-xu wants to merge 2 commits intolivebook-dev:mainfrom
cocoa-xu:cx-fullscreen
Closed

Proposal: WIP: fullscreen cell and output#2801
cocoa-xu wants to merge 2 commits intolivebook-dev:mainfrom
cocoa-xu:cx-fullscreen

Conversation

@cocoa-xu
Copy link
Copy Markdown
Contributor

It would be nice if we could fullscreen cell and output. Although I haven't quite figured out a way to move/display these iframes in the output (and input if it's a smartcell) in the fullscreen view.

@josevalim
Copy link
Copy Markdown
Contributor

Hi @cocoa-xu! You can already amplify outputs. Is that not enough? Do you have a use case where full screen is necessary?

@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 29, 2024

Uffizzi Preview deployment-56725 was deleted.

@cocoa-xu
Copy link
Copy Markdown
Contributor Author

cocoa-xu commented Sep 29, 2024

Hi @cocoa-xu! You can already amplify outputs. Is that not enough? Do you have a use case where full screen is necessary?

I think it can open new possibilities for livebook, for example, using Kino to do Keynote in fullscreen mode (like 1, 2), and one step further, that would be creating simple fullscreen apps with Kino (or other smart cells).

@josevalim
Copy link
Copy Markdown
Contributor

Given this is specific to some use cases, perhaps the Kino should be the one allow to go full screen instead? Maybe if we add some permissions to the iframe that would be enough?

@cocoa-xu
Copy link
Copy Markdown
Contributor Author

Given this is specific to some use cases, perhaps the Kino should be the one allow to go full screen instead? Maybe if we add some permissions to the iframe that would be enough?

Yeah for use cases like this I totally agree it would be the libraries' responsibility to do it.

Yet sometimes I feel the more or less the same way as @madlep mentioned in #2588, that occasionally there's a need make use of the screen real estate more effectively.

And of course, this PR is far from perfect, maybe I should switch it into an issue or we can just track it in #2588. I'll close this one ;)

@cocoa-xu cocoa-xu closed this Sep 29, 2024
@jonatanklosko
Copy link
Copy Markdown
Member

We already allow iframes to enter fullscreen (#1982). It wouldn't work for kinos composed of frame/grid. Though in the slides example, the slides are specified using HTML, so it could as well be a single Kino.JS with a fullscreen action.

@hugobarauna
Copy link
Copy Markdown
Member

It wouldn't work for kinos composed of frame/grid

@jonatanklosko Out of curiosity, why would it not work for kinos composed of frame/grid?

@jonatanklosko
Copy link
Copy Markdown
Member

why would it not work for kinos composed of frame/grid?

The fullscreen mode is something an iframe can request, so it could be done with JavaScript from within Kino.JS. Frame and grid are built-in outputs and don't use iframes.

@cocoa-xu cocoa-xu deleted the cx-fullscreen branch February 11, 2025 23:10
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.

4 participants