Fix overlapping buttons in header

[silverkszlo]

• Target version: stable33

Summary

The bug was probably introduced in #5496

Fixes this button overlap:

Checklist

• [ x] Code is properly formatted
• [ x] Sign-off message is added to all commits
• [ x] Code is AI-generated
• Documentation (manuals or wiki) has been updated or is not required

[elzody]

This problem also affects the main branch; maybe you could make the PR against that and we can then back-port it, unless there is some specific reason not to?

Also, at least one of the failing Cypress tests (open.spec.js) seem to be related to this change:

richdocuments/cypress/e2e/open.spec.js

Lines 41 to 44 in 3494cca

	cy.get('#viewer .modal-header')
	.should('exist')
	.and('not.be.visible')
	.and('have.css', 'display', 'none')

[silverkszlo]

This problem also affects the main branch; maybe you could make the PR against that and we can then back-port it, unless there is some specific reason not to?

Also, at least one of the failing Cypress tests (open.spec.js) seem to be related to this change:

richdocuments/cypress/e2e/open.spec.js

Lines 41 to 44 in 3494cca

	cy.get('#viewer .modal-header')
	.should('exist')
	.and('not.be.visible')
	.and('have.css', 'display', 'none')

You're right! I couldn't reproduce it on main at first, but now for some magical reason I can. I'll open a new PR against main, close this one and take a look at the tests. Thank you!

[silverkszlo]

New PR against main created here: #5599

Closing this one.