plone / Products.CMFPlone

The core of the Plone content management system
https://plone.org
GNU General Public License v2.0
255 stars 191 forks source link

Toolbar: Wrong position of content #3236

Closed csenger closed 1 year ago

csenger commented 3 years ago

What I did:

Have a Window that is not as high as the content of the contentmenu in the toolbar. E.g. the Add...-Menu with a lot of types in it.

What I expect to happen:

The contentmenu has the full height of the window and a scrollbar and is fully visible

What actually happened:

The contentmenu has a negative margin-top. Only the lower part is visible.

image

The code in mockup should be limited to a minumum margin-top of 0:

https://github.com/plone/mockup/blob/5e71d149beacbebc97eef2cf8693b1a706431923/mockup/patterns/toolbar/pattern.js#L324-L331

What version of Plone/ Addons I am using:

Plone CMFPlone 5.2.2., mockup 3.2.2, plone.staticresources 1.3.2

jensens commented 1 year ago

AFAICT it works fine in Plone 6. Please reopen if I am wrong.