Skip to content

Conversation

@mejo-
Copy link
Member

@mejo- mejo- commented Oct 7, 2025

We want line wrapping in code blocks in plaintext txt documents only.

Fixes: #7601

πŸ–ΌοΈ Screenshots

🏚️ Before 🏑 After
image image
image image
image image

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits

@mejo- mejo- self-assigned this Oct 7, 2025
@mejo- mejo- requested a review from max-nextcloud as a code owner October 7, 2025 12:37
@mejo- mejo- added bug Something isn't working feature: formatting Features related to text formatting and node types 3. to review regression labels Oct 7, 2025
@codecov
Copy link

codecov bot commented Oct 7, 2025

Codecov Report

βœ… All modified and coverable lines are covered by tests.
βœ… Project coverage is 59.34%. Comparing base (59ad9f2) to head (5a4a5fc).
⚠️ Report is 23 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7733   +/-   ##
=======================================
  Coverage   59.34%   59.34%           
=======================================
  Files         503      503           
  Lines       39210    39210           
  Branches     1142     1142           
=======================================
  Hits        23268    23268           
  Misses      15834    15834           
  Partials      108      108           

β˜” View full report in Codecov by Sentry.
πŸ“’ Have feedback on the report? Share it here.

πŸš€ New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • πŸ“¦ JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mejo- mejo- force-pushed the fix/codeblock_no_wrap branch from b95fbd3 to 5a4a5fc Compare October 7, 2025 13:49
@mejo- mejo- changed the title fix(code): wrap lines in code blocks, only in plaintext documents fix(code): don't wrap lines in code blocks, only in plaintext documents Oct 7, 2025
@juliusknorr juliusknorr merged commit a691728 into main Oct 8, 2025
66 of 70 checks passed
@juliusknorr juliusknorr deleted the fix/codeblock_no_wrap branch October 8, 2025 19:48
@mejo-
Copy link
Member Author

mejo- commented Oct 9, 2025

/backport to stable32

@mejo-
Copy link
Member Author

mejo- commented Oct 9, 2025

/backport to stable31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review bug Something isn't working feature: formatting Features related to text formatting and node types regression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

No horizontal scroll bar for code fences - regression?

3 participants