shmruin opened a new pull request, #4986: URL: https://github.com/apache/zeppelin/pull/4986
### What is this PR for? Fix an issue where the new Angular UI (zeppelin-web-angular) does not correctly handle carriage return (`\r`) characters. This causes outputs like progress bars(`#`) or status updates that use `\r` to overwrite lines to appear ugly and unnecessarily long. This implementation is a direct port of the previous logic from the classic UI (zeppelin-web), specifically based on the original implementation in [result.controller.js](https://github.com/apache/zeppelin/blob/8ca6e57f27324b1e74fc78bea5cabd176b607408/zeppelin-web/src/app/notebook/paragraph/result/result.controller.js#L527) and [result.js](https://github.com/apache/zeppelin/blob/8ca6e57f27324b1e74fc78bea5cabd176b607408/zeppelin-web/src/app/notebook/paragraph/result/result.js#L23). ### What type of PR is it? Bug Fix ### Todos ### What is the Jira issue? [ZEPPELIN-6214] ### How should this be tested? * Any zeppelin notebook paragraph that generating progress bar (`#`) is the easiest to test this change. * Or you can reproduce the Jira Issue ticket's situation by running the code. Installing conda, set your own conda env, and set path in zeppelin is required. ### Screenshots (if appropriate) * Current Version <img width="2423" height="1021" alt="CR_Current" src="https://github.com/user-attachments/assets/e368c49d-2519-4497-ab25-f0ddfdd19a8a"; /> * Fixed Version <img width="2497" height="309" alt="CR_Fixed" src="https://github.com/user-attachments/assets/d6d40ce3-3d96-4fcf-a50a-224fb384f160"; /> ### Questions: * Does the license files need to update? N * Is there breaking changes for older versions? N * Does this needs documentation? N -- 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]
