eoger / tabcenter-redux

Vertical Tabs extension for Firefox
Mozilla Public License 2.0
379 stars 67 forks source link

Add warning to Reload All Tabs / Close Other Tab #338

Open noscript opened 6 years ago

noscript commented 6 years ago

By accident right clicking on the tab bar can initiate a destructive action, with no way to undo it. It would be great to incorporate some kind of confirmation dialog for actions that affect multiple tabs. IIRC Firefox has such for horizontal tabs.

Keith94 commented 6 years ago

Related: #93

Seems like it's possible to add with only JavaScript: https://github.com/piroor/treestyletab/issues/1752#ref-commit-dbd7414

haarp commented 6 years ago

It's possible to remove such dangerous menu entries with CSS.

/* Remove "Reload All Tabs", "Close Tabs Underneath" and "Close Other Tabs" */
.contextmenu li:nth-child(8), .contextmenu li:nth-child(9), .contextmenu li:nth-child(10) {
    display: none;
}

Not the same as a warning, but since I never needed these anyway, it works for me.

noscript commented 6 years ago

@haarp that's a good enough approach