Reviewed & tested by the community
Project:
Drupal core
Version:
main
Component:
javascript
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
30 Jun 2026 at 15:08 UTC
Updated:
2 Jul 2026 at 17:11 UTC
Jump to comment: Most recent
Show commands
changes
Comments
Comment #3
longwaveUpgraded prettier, fixed an incorrect path in .prettierignore, and ran yarn prettier to reformat all code.
We should likely backport this to 11.x for easier cherry-picking of default_admin changes.
Comment #4
godotislateUpdate to prettier has some good but also strange preferences on how to break lines, but this generally looks good.
There's one file that got unminified, so asked a question on the MR about that.
Comment #5
longwaveI wonder where that file came from originally. I think eslint ignores the migrated JS in default_admin, unsure if we should do the same here.
Regarding the unminification it at least makes it editable if we need to in the future, and JS aggregation will re-minify it again via Peast.
Comment #6
godotislateSounds good. It probably makes sense to look into that minified JS file in a follow up issue, probably a part of the default_admin clean up effort, if it doesn't already exist. I also noticed there are still a lot of gin references. But all that doesn't affect anything here, and the MR lgtm.
Comment #7
longwaveActually it's not that readable so I'm going to skip it via .prettierignore, the default_admin cleanup work can sort it out.
Comment #8
longwaveReverted scroll_sync.js.