Chromium

@Reporter: Could you please elaborate on the same by providing expected/actual behavior along with the clear reproducible steps. And provide sample file/url which you are facing the issue. If possible provide a screencast for better understanding/triaging the issue. Also confirm OS and chrome version details.

Thanks.

Previously it was possible to disable pretty print in "Experements". This option has now been removed in the latest Google Chrome update (chrome 121).

Thank you for providing more feedback. Adding the requester to the cc list.

Able to reproduce the issue on  reported chrome version  #121.0.6099.160 using Linux Debian as per steps in comment#0.
Note: Issue reproducible on Windows 10 and Mac 14.3.1

Reproducible:
===========
Canary- 123.0.6298.0
Dev - 123.0.6286.0
Beta - 122.0.6261.29
Stable - 121.0.6167.160

Bisect Information:
---------------------------------
Good Build: 121.0.6133.0
Bad Build:  121.0.6134.0

Bisect Script: python3 bisect_builds.py -a linux64 -g 1225867 -b 1226072

ChangeLog URL: https://chromium.googlesource.com/chromium/src/+log/581b20ae0be2897f92a05382267a730916ad7b5a..0e085d1f0d828270b4b7cfce3ed58d9df7f7ebc2

Suspect: https://chromium.googlesource.com/devtools/devtools-frontend.git/+/c25d9ada1503fd232214c520a9ebd574380a6d89
Change ID: Ib5a2746e6ee6d9a2b62f5cb71ddc5c23f9228c5f
Reviewed on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/5040490

@bmeurer: Please help us in re-assigning if this is not related to your change.

Thanks..!

Thanks for the answer! Please make a checkbox where I can turn off the automatically pretty print.
Because for me (html coder), and not only for me, this is bad ux.

Hello! Yes, I coding (css+html) the site in a browser and it is very convenient.
Firstly, I have to constantly switch to the default type of css code, because I write a css string and this is my writing style, which I want to maintain in the future.
Secondly, when using the toolbar, the pretty print object property is applied automatically and destroys the entire CSS code. I recorded a video.
https://drive.google.com/file/d/1f0UI78v0Fr-8qIX1D3Tk0s438zOeQ0vx/view?usp=sharing
https://drive.google.com/file/d/1ELF5wjg9yKRELWw0cdbHDSXXS79O-VrE/view?usp=sharing

Thank you for providing more feedback. Adding the requester to the CC list.

The Found In field may only contain numeric values.
Some values couldn't be corrected but were removed, please verify that any important data wasn't lost.
You can see the changes by toggling full history on the issue.

Project: devtools/devtools-frontend
Branch: main

commit ef23858f4bbc7b5cecce5d3db472d617482a46a1
Author: Benedikt Meurer <bmeurer@chromium.org>
Date:   Tue Mar 19 08:22:21 2024

    [sources] Introduce a setting to disable automatic pretty-printing.
   
    This partially reverts https://crrev.com/c/5040490 and reintroduces the
    experiment as a proper setting, as there are apparently valid use cases
    for disabling the automatic pretty-printing in DevTools.
   
    Screenshot: https://i.imgur.com/u821qOv.png
    Fixed: b:324314570
    Bug: chromium:1382752, chromium:1503031
    Change-Id: Ide7e8d5214140acf9385d58db3a27ddce8518d39
    Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/5372061
    Commit-Queue: Philip Pfaffe <pfaffe@chromium.org>
    Reviewed-by: Philip Pfaffe <pfaffe@chromium.org>
    Auto-Submit: Benedikt Meurer <bmeurer@chromium.org>

M       front_end/panels/sources/sources-meta.ts
M       front_end/testing/EnvironmentHelpers.ts
M       front_end/ui/legacy/components/source_frame/SourceFrame.ts
M       test/e2e/sources/can-pretty-print-sourcecode_test.ts

https://chromium-review.googlesource.com/5372061

Merge review required: no relevant commits could be automatically detected (via Git Watcher comments), sending to merge review for manual evaluation. If you have not already manually listed the relevant commits to be merged via a comment above, please do so ASAP.

Please answer the following questions so that we can safely process your merge request:
1. Why does your merge fit within the merge criteria for these milestones?
- Chrome Browser: https://chromiumdash.appspot.com/branches
- Chrome OS: https://goto.google.com/cros-release-branch-merge-guidelines
2. What changes specifically would you like to merge? Please link to Gerrit.
3. Have the changes been released and tested on canary?
4. Is this a new feature? If yes, is it behind a Finch flag and are experiments active in any release channels?
5. [Chrome OS only]: Was the change reviewed and approved by the Eng Prod Representative? https://goto.google.com/cros-engprodcomponents
6. If this merge addresses a major issue in the stable channel, does it require manual verification by the test team? If so, please describe required testing.

Please contact the milestone owner if you have questions.
Owners: eakpobaro (Android), eakpobaro (iOS), obenedict (ChromeOS), danielyip (Desktop)

Project: devtools/devtools-frontend
Branch: chromium/6367

commit 985d7d73c01f90b38d28ea2260e8c9ad616cdd9b
Author: Benedikt Meurer <bmeurer@chromium.org>
Date:   Tue Mar 19 08:22:21 2024

    [sources] Introduce a setting to disable automatic pretty-printing.
   
    This partially reverts https://crrev.com/c/5040490 and reintroduces the
    experiment as a proper setting, as there are apparently valid use cases
    for disabling the automatic pretty-printing in DevTools.
   
    Screenshot: https://i.imgur.com/u821qOv.png
    Fixed: b:324314570
    Bug: chromium:1382752, chromium:1503031
    Change-Id: Ide7e8d5214140acf9385d58db3a27ddce8518d39
    Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/5372061
    Commit-Queue: Philip Pfaffe <pfaffe@chromium.org>
    Reviewed-by: Philip Pfaffe <pfaffe@chromium.org>
    Auto-Submit: Benedikt Meurer <bmeurer@chromium.org>
    (cherry picked from commit ef23858f4bbc7b5cecce5d3db472d617482a46a1)
    Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/5385649
    Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
    Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>

M       front_end/panels/sources/sources-meta.ts
M       front_end/testing/EnvironmentHelpers.ts
M       front_end/ui/legacy/components/source_frame/SourceFrame.ts
M       test/e2e/sources/can-pretty-print-sourcecode_test.ts

https://chromium-review.googlesource.com/5385649

Hello, the problem UX of them being mistaken has started to reoccur again.