Bug 403848 - Redundant warning on close for unmodified documents in multiple tabs view
Summary: Redundant warning on close for unmodified documents in multiple tabs view
Status: RESOLVED WORKSFORME
Alias: None
Product: okular
Classification: Applications
Component: general (show other bugs)
Version: 1.4.3
Platform: Other Linux
: NOR minor
Target Milestone: ---
Assignee: Okular developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-02-02 07:29 UTC by happy
Modified: 2019-03-07 04:33 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description happy 2019-02-02 07:29:48 UTC
SUMMARY
Mulitple tabs per window is nice, I hadn't used it before.
However, for unmodified documents it complains (only in multiple tab mode?) that you're about to close multiple tabs either on closing a window or quitting the application (eg: log out/shutdown session).

STEPS TO REPRODUCE
1. Enable multiple tabs per window
2. Open several PDFs (don't change them)
3. Log out (or quit Okular)

OBSERVED RESULT
Okular complains about closing multiple tabs.
This is a pain if you shut down the computer and walk away (or perhaps close the lid)... session log out hangs because of this unnecessary warning.

EXPECTED RESULT
Documents aren't modified by user (ie: no markup) so no complaint or checking necessary.

SOFTWARE/OS VERSIONS
Windows: 
MacOS: 
Linux/KDE Plasma: Kubuntu 18.10 Linux (x86_64) release 4.18.11-041811-generic
(available in About System)
KDE Plasma Version: 5.14.5
KDE Frameworks Version: 5.52.0
Qt Version: 5.11.1

ADDITIONAL INFORMATION
Minor (for me) because I don't often do markup and I can just uncheck "warn me if closing multiple tabs"; but if modifying documents was part of your work flow then this would be more severe (ie: false warnings obscuring real ones)
Comment 1 Yuri Chornoivan 2019-02-02 07:34:48 UTC
This warning can be useful because sometimes you can unintentionally close the window trying to close the current document only.
Comment 2 Michael Weghorn 2019-02-02 08:42:15 UTC
(In reply to happy from comment #0)
> ADDITIONAL INFORMATION
> Minor (for me) because I don't often do markup and I can just uncheck "warn
> me if closing multiple tabs"; but if modifying documents was part of your
> work flow then this would be more severe (ie: false warnings obscuring real
> ones)

Can you explain what exactly you mean by this?

A quick test for me showed that when unchecking "warn me if closing multiple tabs",

* no dialog appears when closing multiple unmodified documents (as you want it to be)
* a dialog "Do you want to save your changes to <FILE> or discard them?" still appears if any of the documents was modified (e.g. an annotation was added)

This is what I would have expected and what I'd consider to be the "correct" behaviour. Can you describe what should be different in your eyes?
Comment 3 happy 2019-02-03 01:11:50 UTC
(In reply to Yuri Chornoivan from comment #1)
> This warning can be useful because sometimes you can unintentionally close
> the window trying to close the current document only.

Yes, I totally agree, I do that all the time (it's the downside of tabs).

What the bug report is about is about is QUITTING Okular, particularly when done automatically on session log out.  I'm sorry I used the word "CLOSING".

I always use "Restore Previous Session" type logins; so I always (well, for most applications) get all my windows back anyway.
As I said, it's a problem if something hangs up the machine shutting down... particularly when there's no need.
Comment 4 happy 2019-02-03 01:16:43 UTC
> This is what I would have expected and what I'd consider to be the "correct"
> behaviour. Can you describe what should be different in your eyes?

Thank you for trying that, I didn't try markup.  See my reply to Yuri about clarification.

I guess I'm requesting a third behaviour on quitting the application?
IE: with the "warn me" behaviour unsuppressed, so that closing a window will generate warnings that _quitting_ the application won't ?
(eg: EXACTLY the same behaviour as Firefox)
Comment 5 Albert Astals Cid 2019-02-04 21:45:35 UTC
I'm going to suggest you close this bug and start a new one with a subject that clearly mentions logging out.
Comment 6 Nate Graham 2019-02-05 20:22:17 UTC
I'm not sure I see what the bug is. What Michael describes is exactly what I see too and what I would expect. If you don't like the "close multiple tabs" warning, you can just uncheck the checkbox and never see them again, but still see the warning when quitting/logging off when modified documents have not been saved.

Can you explain what it is that you expect to happen?
Comment 7 Bug Janitor Service 2019-02-20 04:33:07 UTC
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least
15 days. Please provide the requested information as soon as
possible and set the bug status as REPORTED. Due to regular bug
tracker maintenance, if the bug is still in NEEDSINFO status with
no change in 30 days the bug will be closed as RESOLVED > WORKSFORME
due to lack of needed information.

For more information about our bug triaging procedures please read the
wiki located here:
https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please
mark the bug as REPORTED so that the KDE team knows that the bug is
ready to be confirmed.

Thank you for helping us make KDE software even better for everyone!
Comment 8 Bug Janitor Service 2019-03-07 04:33:10 UTC
This bug has been in NEEDSINFO status with no change for at least
30 days. The bug is now closed as RESOLVED > WORKSFORME
due to lack of needed information.

For more information about our bug triaging procedures please read the
wiki located here:
https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

Thank you for helping us make KDE software even better for everyone!