fix: improve publishing information spacing

[TahaKhan998]

Description

Changes include:

• Adding left margin to the publishing info grid for better alignment
• Introducing vertical spacing between sections on small screens
• Allowing long strings (e.g. identifiers or long titles) to wrap properly to prevent overflow

These adjustments improve readability and prevent layout issues when fields contain very long values.

Related PR

Depends on:
inveniosoftware/invenio-rdm-records#2268
inveniosoftware/invenio-app-rdm#3362

[zubeydecivelek]

Just a small question: is there any reason this styling is added in cds-rdm instead of invenio-app-rdm?

[TahaKhan998]

Just a small question: is there any reason this styling is added in cds-rdm instead of invenio-app-rdm?

I added it in cds-rdm since this is a CDS-specific styling adjustment (spacing and text wrapping) applied thourgh theme override and does not modify any shared component or template in invenio-app-rdm.

[palkerecsenyi]

Looks good! Regarding @zubeydecivelek's comment and your reply, just to clarify, does the new layout work okay without this CSS? Thanks!

[palkerecsenyi]

Nit: isn't it normally 768px?

[TahaKhan998]

yes, it should be 768px. I'll update it.

[TahaKhan998]

Looks good! Regarding @zubeydecivelek's comment and your reply, just to clarify, does the new layout work okay without this CSS? Thanks!

Without the additional CSS the long metadata values overflow the container and break the layout on smaller screens.

Without the CSS fix:

With the CSS fix applied:

The added CSS ensures long strings wrap correctly using overflow-wrap / word-break, which prevents the layout from breaking.

[palkerecsenyi]

Looks good! Regarding @zubeydecivelek's comment and your reply, just to clarify, does the new layout work okay without this CSS? Thanks!

Without the additional CSS the long metadata values overflow the container and break the layout on smaller screens.

Without the CSS fix:

With the CSS fix applied:

The added CSS ensures long strings wrap correctly using overflow-wrap / word-break, which prevents the layout from breaking.

Thanks, so does this mean it would be useful to add them on the InvenioRDM level instead of the CDS level as @zubeydecivelek suggested?

[TahaKhan998]

Looks good! Regarding @zubeydecivelek's comment and your reply, just to clarify, does the new layout work okay without this CSS? Thanks!

Without the additional CSS the long metadata values overflow the container and break the layout on smaller screens.
Without the CSS fix:

With the CSS fix applied:

The added CSS ensures long strings wrap correctly using overflow-wrap / word-break, which prevents the layout from breaking.

Thanks, so does this mean it would be useful to add them on the InvenioRDM level instead of the CDS level as @zubeydecivelek suggested?

Thanks for the feedback!

I’ve created a new PR with the changes applied at the InvenioRDM level: inveniosoftware/invenio-app-rdm#3371

Shall we review that one instead and close this PR?

[zubeydecivelek]

I think you can close this PR, thanks!