Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

md-menu has a problem top when inside a toolbar #6080

Closed
pierrybos opened this issue Dec 4, 2015 · 2 comments
Closed

md-menu has a problem top when inside a toolbar #6080

pierrybos opened this issue Dec 4, 2015 · 2 comments

Comments

@pierrybos
Copy link

Hi all, Someone has this problem?

http://codepen.io/anon/pen/jWOyGE

I identify on
material/src/components/menu/js/menuServiceProvider.js

392 . function calculateMenuPosition(el, opts) {

(...)

405.      var bounds = {
406.        left: boundryNodeRect.left + MENU_EDGE_MARGIN,
407.        top: Math.max(boundryNodeRect.top, 0) + MENU_EDGE_MARGIN,
408.        bottom: Math.max(boundryNodeRect.bottom, Math.max(boundryNodeRect.top, 0) + boundryNodeRect.height) - MENU_EDGE_MARGIN,
409.        right: boundryNodeRect.right - MENU_EDGE_MARGIN
410.      };

if change 407: to:

       top: Math.max(boundryNodeRect.top, 0) + MENU_EDGE_MARGIN,

from:

       top: Math.max(boundryNodeRect.top, 0), //  + MENU_EDGE_MARGIN,

This solve my problem, but isn't a graceful solution.
Anyway to identify is inside a toolbar and do other rule?

Only me with this problem or i can try help solving this?

@rudzikdawid
Copy link
Contributor

+1 duplicate of #6073 and #6049

@ThomasBurleson
Copy link
Contributor

Duplicate

rschmukler added a commit that referenced this issue Dec 8, 2015
fixes #6131, fixes #6109, fixes #6049, fixes #6073, references #6080,
fixes #6089
rschmukler added a commit that referenced this issue Dec 8, 2015
fixes #6131, fixes #6109, fixes #6049, fixes #6073, references #6080,
fixes #6089, fixes #6116
rschmukler added a commit that referenced this issue Dec 8, 2015
fixes #6131, fixes #6109, fixes #6049, fixes #6073, references #6080,
fixes #6089, fixes #6116
rschmukler added a commit that referenced this issue Dec 8, 2015
fixes #6131, fixes #6109, fixes #6049, fixes #6073, references #6080,
fixes #6089, fixes #6116
rschmukler added a commit that referenced this issue Dec 8, 2015
fixes #6131, fixes #6109, fixes #6049, fixes #6073, references #6080,
fixes #6089, fixes #6116
rschmukler added a commit that referenced this issue Dec 8, 2015
fixes #6131, fixes #6109, fixes #6049, fixes #6073, references #6080,
fixes #6089, fixes #6116
rschmukler added a commit that referenced this issue Dec 8, 2015
fixes #6131, fixes #6109, fixes #6049, fixes #6073, references #6080,
fixes #6089, fixes #6116
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants