Closed Bug 1340762 Opened 3 years ago Closed 3 years ago
Submenu to its own separate function like the other submenus of context menu in inspector .js
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:51.0) Gecko/20100101 Firefox/51.0 Build ID: 20170201180315 Expected results: Presently in /devtools/client/inspector.js, the attributes submenu and paste submenu are created in separate functions and those functions are called in _openMenu function inorder to create the submenus, but copy submenu is created within the _openMenu function.It would be nice to give copySubmenu its own separate function like the other two. It will reduce the size of the _openMenu function as well as will make the code more consistent.
Component: Untriaged → Developer Tools
Hardware: x86 → x86_64
Moved copySubmenu to a separate function
Moved copySubmenu to separate function
Assignee: nobody → djmdeveloper060796
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Priority: -- → P3
rebased and r+. Thank you for the patch Deepjyoti
Attachment #8838897 - Flags: review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/c0d805983cf3 Refactor copySubmenu into its own separate function in inspector.js; r=gl
You need to log in before you can comment on or make changes to this bug.