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

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

Reply via email to