Here's a screenshot of the current adjustment:

On further investigation, I found that add reference and add qualifier are the only modals using the "minimal" style. Since the changes to this style can be easily isolated, I've made a commit that sticks to the original scope of this ticket. The broader question about consistency for all modal headers can be handled later, if necessary.

hooray! moving it along now

In investigating this, I found that we have at least 2 ways of creating modal headers. The ones with too much padding ("add qualifier", "add reference", "add statement", "add value") are using the "header" slot in WikibaseWbui2025ModalOverlay.
The padding for these headers are defined as: padding: @spacing-250 @spacing-75 @spacing-150 @spacing-75; (resulting in 40px on top, and 24 px on the bottom)

Ahhh maybe not. A test failure on CI lead me to this commit in api-testing, which makes the config into a module object rather than a function.

Should we just wait and see what LibUp does in its next run then?

LibUp's config was updated yesterday, so now I'm confused about a couple things:

Of these options, 1 sounds the best to me. I agree that it makes sense to keep Wbui2025 logic out of random extensions.

The current version of api-testing locks supertest at "7.1.0", which I assume is why LibUp could not update it.
A patch to bump the dependency in api-testing is merged, but we'll need a new release of that before it is available.

There's an in-progress patch for this, with a css-only change. It works for musical notation, but is not ideal for long string values.

@ArthurTaylor mentioned in the parent task:

The task split looks pretty sensible, and we can go through in more detail when we do the task breakdown. The big concern I have is about the end-to-end testing - we would need a way to enable the Musical Notation datatype for Wikibase when the Score extension is loaded, and we would need there to be a live Shellbox for CI that has lilypond installed. I expect that both of these things will be hard to achieve with Quibble. We might have to make do with unit testing - still an improvement on the zero tests currently included in the Score extension.

makes sense! I'll add this context to T412141

spike notes:

• once the extension is set up correctly, existing statements with musical notation are displayed with some minor style issues for long values. Covered in T412128: [MEX] M4.1.10 Display statements with musical notation in view mode

oops, yeah, that should be attached. I'll fix the commit message

Notes on getting Kartographer working locally:

@Alice.moutinho all good! I haven't gotten very far with this yet, so it's fine to include it in this ticket.