Moved to https://invent.kde.org/kde/okular/merge_requests/69 .

An MR is drafted at invent, see https://invent.kde.org/kde/okular/merge_requests/69 .

Fixed code styles

• Added "undo tab close"
• Added a test

In D25484#567258, @ngraham wrote:

Yes this helps the case that @pino brings up, but it handles a different use case. Most of the time a tab won't have unsaved changes because Okular is primarily a reading app. An "Undo last closed tab" feature is for these other cases.

In D25484#566490, @pino wrote:

...

In D25484#566483, @romangg wrote:

The solution to accidentally clicks is not disabling it by default and making it configurable, but providing the functionality to "restore" a tab after closing it accidentally. Dolphin and Chrome allow me to restore the last closed tab with Ctrl+Shift+T.

And this "solution" does not always work, for example when a page was the result of a form submission, redirection, or other kind of interactivity.

In D25484#567076, @aacid wrote:

...
Also please remember to use invent.kde.org and not phabricator for okular merge requests

In D25484#566993, @ngraham wrote:

@bdbai, to move this forward, do you think you could also make tab closing undoable?