feat: Add fullscreen toggle to code viewer#68044
Conversation
choo121600
requested review from
bbovenzi,
guan404ming,
pierrejeambrun,
ryanahamilton,
shubhamraj-git and
vatsrahul1001
as code owners
boring-cyborg
Bot
added
area:translations
area:UI
bbovenzi
left a comment
bbovenzi
left a comment
There was a problem hiding this comment.
We already do a lot of this to see logs in full screen. Let's reuse approaches, including translation keys.
pierrejeambrun
left a comment
pierrejeambrun
left a comment
There was a problem hiding this comment.
As Brent mentioned it would be more consistent to mirror what we do for logs.
Refactor a bit to reuse dag:logs.fullscreen.tooltip translation keys and similar and use a fullscreen Dialog component.
choo121600
force-pushed
the
feat/fullscreen-toggle-code-view
branch
from
735ce31 to
d975bf3
Compare
choo121600
force-pushed
the
feat/fullscreen-toggle-code-view
branch
from
d975bf3 to
1dd8a55
Compare
|
Thanks, I've updated it to mirror the logs view implementation :) Screen.Recording.2026-06-11.at.11.23.47.PM.movWhile doing that, I noticed a few inconsistencies across views around things like icons, tooltips, and button patterns. |
Backport successfully created: v3-3-testNote: As of Merging PRs targeted for Airflow 3.X In matter of doubt please ask in #release-management Slack channel.
|
4 participants
Why
The Dag code view became a small, fixed-height window in Airflow 3.2, showing only a few lines.
This restores it to fill the available height and adds a fullscreen toggle (button, hotkey
f,Escto exit) for reading longer files.Screen.Recording.2026-06-11.at.11.23.47.PM.mov
related: #67359
Was generative AI tooling used to co-author this PR?
claude opus 4.7
{pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.