839318 - context-menu overflows depending on the total number of menu items regardless of whether they are hidden or not

Status: RESOLVED FIXED

(Add-on SDK Graveyard :: General, defect, P1)

Description

[Dave Townsend [:mossop]]

If add-ons add 20 context menu items but only one is visible we should not overflow, but we do.

Comment 1

[Dave Townsend [:mossop]]

Attachment: Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/776

Pointer to Github pull-request

Comment 2

[[github robot]]

Commits pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/83b1d8fc326829d71a23c06ed4e0a51b0c72502c
Bug 839318: context-menu should overflow based on the number of visible menu items.

https://github.com/mozilla/addon-sdk/commit/991638d83e13df8f353f3bf473e723df7964745f
Merge pull request #776 from Mossop/bug839318

Fix Bug 839318: context-menu should overflow based on the number of visible menu items r=@erikvold

Comment 3

[[github robot]]

Commit pushed to stabilization at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/056da994cc2802fdc2b23964e4080cb95bfecb1e
Bug 839318: context-menu should overflow based on the number of visible menu items.
(cherry picked from commit 83b1d8fc326829d71a23c06ed4e0a51b0c72502c)

Comment 4

[[github robot]]

Commit pushed to release at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/96f521489255504569233720f96811fa00c17c0e
Bug 839318: context-menu should overflow based on the number of visible menu items.
(cherry picked from commit 83b1d8fc326829d71a23c06ed4e0a51b0c72502c)
(cherry picked from commit 056da994cc2802fdc2b23964e4080cb95bfecb1e)

Comment 5

[[github robot]]

Commit pushed to stabilization at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/96f521489255504569233720f96811fa00c17c0e
Bug 839318: context-menu should overflow based on the number of visible menu items.