Align cresc and dim in palette#32963
Conversation
Palette elements are drawn centered on the cell by their bounding box, so they are not aligned at the baseline. See for instance also mp and mf. Here dim has a taller bounding box because of the d and i ascenders, so it's expected that the baseline doesn't match with cresc. I don't know how they were aligned before. We could do fancy stuff using the font x-height etc but it's frankly not worth it, so I've just adjusted the offset.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis change modifies the 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
@mike-spa Tested and approved on Ubuntu 24.04.4 LTS. |
Palette elements are drawn centered on the cell by their bounding box, so they are not aligned at the baseline. See for instance also mp and mf.
Here dim has a taller bounding box because of the d and i ascenders, so it's expected that the baseline doesn't match with cresc. I don't know how they were aligned before. We could do fancy stuff using the font x-height etc but it's frankly not worth it, so I've just adjusted the offset.
Resolves: #32909