You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "github-actions[bot] (via GitHub)" <gi...@apache.org> on 2026/09/23 20:35:02 UTC

[PR] [v3-3-test] UI: Fix copied SQL from Rendered Templates breaking at every token (#73633) [airflow]

github-actions[bot] opened a new pull request, #73640:
URL: https://github.com/apache/airflow/pull/73640

   Combining `wrapLongLines` with `showLineNumbers` makes react-syntax-highlighter
   lay each line out as a flex container, so every highlighted token becomes a
   block-level box and the browser emits a newline after each one when it
   serializes a selection. Selecting a rendered SQL field and copying it therefore
   produced something that no longer parsed. The copy button kept working because
   it copies the raw string rather than the DOM, which is why this went unnoticed.
   
   The prism theme also hard-codes `white-space: pre` on the code tag and the
   library merges that over its own `pre-wrap`, so `wrapLongLines` has never
   actually wrapped anything.
   
   The copy button was pulled back over the code block with a fixed negative
   margin, so where it landed depended on the block's height.
   (cherry picked from commit ffa19c31a67f86dadf23709eb10cfaacd2669e63)
   
   Co-authored-by: Brent Bovenzi <br...@astronomer.io>
   Closes: #73537


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [v3-3-test] UI: Fix copied SQL from Rendered Templates breaking at every token (#73633) [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #73640:
URL: https://github.com/apache/airflow/pull/73640


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [v3-3-test] UI: Fix copied SQL from Rendered Templates breaking at every token (#73633) [airflow]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #73640:
URL: https://github.com/apache/airflow/pull/73640#issuecomment-5872594943

   Hi maintainer, this PR was merged without a milestone set.
   We've automatically set the milestone to **[Airflow 3.3.3](https://github.com/apache/airflow/milestone/152)** based on: bug fix merged to version branch
   If this milestone is not correct, please update it to the appropriate milestone.
   
   > This comment was generated by [Milestone Tag Assistant](https://github.com/apache/airflow/blob/main/.github/workflows/milestone-tag-assistant.yml).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org