namanjain24-sudo opened a new pull request, #73590:
URL: https://github.com/apache/airflow/pull/73590

   Manually selecting and copying SQL (or other code) from the Rendered 
Templates tab pasted back with a line break after almost every word, breaking 
the syntax. `react-syntax-highlighter` defaults every line to `display: flex` 
whenever both `wrapLongLines` and `showLineNumbers` are set (as they are here); 
flex children get CSS-blockified, so each syntax-highlighted token becomes its 
own block-level element, and browsers insert a newline boundary around each one 
when serializing a manual text selection to the clipboard.
   
   A near-identical issue was fixed in #66221 by passing `lineProps` to 
override that per-line style. The Rendered Templates view was substantially 
rewritten afterwards (new SQL-aware language detection, a different 
`SyntaxHighlighter` wrapper), and that override did not carry over, so the 
underlying `display: flex` regressed.
   
   This reapplies the same `lineProps` override to the current component, and 
adds a regression test that mocks the highlighter and asserts the prop is 
passed — verified it fails without the fix and passes with it.
   
   closes: #73537
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Sonnet 5)
   
   Generated-by: Claude Code (Sonnet 5) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to