Bug 1749998 Comment 37 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Hm, I don't think that is the right place for the new menu item. All the items in that first section are referring to the download panel menu button, not the panel. 

Gijs, I know moving things around gets tricky. See below... possible to add another separator? 


Pin to overflow menu
Hide button when empty
Remove from toolbar
----------------------- 
Show panel when download begins  
----------------------- 
Bookmarks Toolbar >
-----------------------
Customize Toolbar...

Note: string does not change with state change. We are using the checkmark pattern to communicate state.
Hm, I don't think that is the right place for the new menu item. All the items in that first section are referring to the download panel menu button, not the panel. 

Gijs, I know moving things around gets tricky. See below... possible to add another separator? 

```
Pin to overflow menu
Hide button when empty
Remove from toolbar
----------------------- 
Show panel when download begins  
----------------------- 
Bookmarks Toolbar >
-----------------------
Customize Toolbar...
```

Note: string does not change with state change. We are using the checkmark pattern to communicate state.
Hm, I don't think that is the right place for the new menu item. All the items in that first section are referring to the download panel menu button, not the panel. 

Gijs, I know moving things around gets tricky. See below... possible to add another separator? 

```
Pin to overflow menu
Hide button when empty
Remove from toolbar
----------------------- 
Show panel when download begins  
----------------------- 
Bookmarks toolbar >
-----------------------
Customize toolbar...
```

Note: string does not change with state change. We are using the checkmark pattern to communicate state.

Back to Bug 1749998 Comment 37