chore(ratchet): verwijder overbodige #1117-basislijnverhogingen (superseded #1119) #1126
No reviewers
Labels
No labels
accepted
bug
declined
docs
duplicate
enhancement
good first issue
in-progress
needs-info
privacy
security
triage
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
LibreKAT/Ocideck!1126
Loading…
Reference in a new issue
No description provided.
Delete branch "fix/markdown-editor-gate-followup-1123"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Wat
Verwijdert overbodige basislijn-verhogingen die ik in #1117 aanbracht en die
door #1119 (#1118) achterhaald zijn.
De situatie: #1117 en #1118/#1119 losten dezelfde conventiepoort-drift (uit de
Markdown-editor-merges) parallel op. #1117 trok de basislijnen op naar de
gegroeide omvang; #1119 loste het beter op via extractie, waardoor de
bestanden weer ónder hun oorspronkelijke plafonds kwamen:
markdown_deck_editor.darttabs_provider.dartapp_shell.dartapp_shell_main_layout.dartMijn verhogingen werden daardoor loze, opgeblazen plafonds: ze verzwakken de
poort (een bestand kon terug groeien naar 1512 zonder te tripen) en gaven bij
elke run "shrank"-tips.
Hoe
Herstelt
tool/check_conventions.dartnaar de staat van #1119 (8a24e400) voorprecies de zes invoeren die ik had aangeraakt:
markdown_deck_editor.dart,tabs_provider.dart) — beide nu onder 1000, geen basislijn nodigapp_shell.dart841→840,app_shell_main_layout.dart1026→1023TabsNotifier2291→2243,_MainLayoutState1490→1457git checkout 8a24e400 -- tool/check_conventions.dartis exact het terugdraaienvan mijn #1117-commit (de enige die dat bestand na #1119 raakte). Alleen
tool/,geen gedrag.
Poort
make checkgroen op de tak: lagere basislijnen maken de poort alleen strenger,en
check-conventionsblijft groen (geen bestand/klasse boven het plafond; mijnzes invoeren geven geen "shrank"-tips meer omdat ze exact op #1119's maat staan).